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

Red Hat Konflux update domain-proxy #2

Merged
merged 3 commits into from
Dec 23, 2024
Merged

Conversation

red-hat-konflux[bot]
Copy link
Contributor

@red-hat-konflux red-hat-konflux bot commented Dec 20, 2024

Pipelines as Code configuration proposal

To start the PipelineRun, add a new comment with content /ok-to-test

For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun

To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

https://issues.redhat.com/browse/JBS-74

Signed-off-by: red-hat-konflux <[email protected]>
RUN go mod tidy
RUN go build -o domainproxyserver cmd/server/main.go
RUN go build -o domainproxyclient cmd/client/main.go
ENV GOTOOLCHAIN=auto \
Copy link
Collaborator

Choose a reason for hiding this comment

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

What's the purpose of these two env variables?

Copy link
Collaborator

Choose a reason for hiding this comment

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

I've done some research and setting these two env variables appears to be redundant as they're inducing default behaviour?

Copy link
Collaborator

@tecarter94 tecarter94 Dec 23, 2024

Choose a reason for hiding this comment

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

Ah, I see that go mod tidy failed in a previous run, I suppose that might necessitate the env variables depending on build environment... Never failed when building domain proxy in JBS. I wonder if the top level JBS go config prevented that.

Copy link
Contributor

Choose a reason for hiding this comment

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

Yes its slightly bizarre that it didn't fail in JBS but those variables were used elsewhere in JBS so I suspect you're right and there was some subtle interaction that made it work.

@tecarter94 tecarter94 merged commit 7accbd6 into main Dec 23, 2024
3 checks passed
@tecarter94 tecarter94 deleted the appstudio-domain-proxy branch December 23, 2024 05:45
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.

2 participants