refactor(docker): improve Dockerfiles for authentication, graphql, subscriber, and upgrade-agents#5468
Conversation
…bscriber, and upgrade-agents - Remove unnecessary RUN go env step - Replace ADD with COPY for local context - Remove ENV GOOS/GOARCH, pass them directly in build command - Ensure LABEL maintainer is present in all stages Signed-off-by: ADISH <adishjain1107@gmail.com>
|
Hey @1107-adishjain Some checks are failing. Could you please check once? |
Sure let me check once and fix it. |
…d subscriber Signed-off-by: ADISH <adishjain1107@gmail.com>
|
Hi @PriteshKiri , I have made the required fixes for the failing Docker CI checks in this PR. Updated Go builder image to a patched version in authentication, graphql, and subscriber Dockerfiles. Could you please re-run the checks? If everything passes, we can proceed with merge to master. |
Description:
This PR improves the Dockerfiles for the authentication, graphql, subscriber, and upgrade-agents services:
RUN go envstepADDwithCOPYfor local contextENV GOOS/GOARCH, pass them directly in build commandLABEL maintaineris present in all stagesProposed changes
This PR refactors the Dockerfiles for core services to improve consistency, follow best practices, and reduce unnecessary steps.
No functional changes to the services themselves.
Types of changes
What types of changes does your code introduce to Litmus? Put an
xin the boxes that applyChecklist
Put an
xin the boxes that apply.Dependency
Please add the links to the dependent PRs that need to be merged before this (if any):
None
Special notes for your reviewer:
N/A