Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

use the distroless image for development with a cluster #41

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jgbernalp
Copy link
Contributor

This PR adjusts the binary name in the distroless images. Also adjusts the metrics default port to avoid collisions while testing locally, fixes: #40

@jgbernalp jgbernalp requested a review from Nexucis December 20, 2024 08:57
COPY --chown=nobody:nobody LICENSE /LICENSE
COPY --from=build-env --chown=nobody:nobody /etc/mime.types /etc/mime.types

EXPOSE 8080
ENTRYPOINT [ "/bin/operator" ]
ENTRYPOINT [ "/bin/manager" ]
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the location of the binary shouldn't be at root ? /manager (Based on the previous issue #38 )

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The deployment was using a command to point to the /manager but I removed it so it uses the default image entrypoint . Which for consistency is now /bin/manager

@Nexucis
Copy link
Member

Nexucis commented Dec 20, 2024

@jgbernalp to fix the CI, you need to update the file https://github.com/perses/perses-operator/blob/main/scripts/generate-goreleaser/.goreleaser.base.yaml#L5 to update the binary there

@jgbernalp jgbernalp force-pushed the use-distroless-for-local-setup branch from ffef9ea to 856c76c Compare December 20, 2024 09:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Perses build fails with updated instructions
2 participants