-
Notifications
You must be signed in to change notification settings - Fork 5
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
fix(RHTAPWATCH-819): Resolve EC label errors #14
Conversation
Dockerfile
Outdated
LABEL description="Konflux Project Controller" | ||
LABEL com.redhat.component="Konflux Project Controller" | ||
LABEL io.k8s.description="Konflux Project Controller" | ||
LABEL io.k8s.display-name="o11y-project-controller" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This has nothing to do with o11y. Please use "konflux" here.
@@ -1,5 +1,5 @@ | |||
# Build the manager binary | |||
FROM golang:1.21 AS builder | |||
FROM registry.access.redhat.com/ubi9/go-toolset:1.20.10-2 AS builder |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need Go 1.21.x
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't have redhat Go 1.21.x image yet. The latest one 1.20.10-3.
Please rebase to see if the latest code builds |
/retest |
Add missing labels which should be present and update images used in Dockerfile for the integration tests to pass. Signed-off-by: Homaja Marisetty <hmariset@redhat.com> Signed-off-by: Barak Korren <bkorren@redhat.com>
Add missing labels which should be present in Dockerfile for the integration tests to pass.