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

feat: [CI-16330]: Adding default OutputFile as DRONE_OUTPUT. #139

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

DevanshMathur19
Copy link

@DevanshMathur19 DevanshMathur19 commented Feb 24, 2025

Execution Links for Kaniko-ecr:
Cloud: https://app.harness.io/ng/account/gCoPSwHxS7ipOgx2iA9tOQ/all/orgs/default/projects/devanshci/pipelines/kanikoecr/executions/dWsqGal1Qku6LxiYZuXRDA/pipeline?storeType=INLINE&step=5xolfedFTiul47eQWLXsVQ&stage=6kD1Q5xEQISQLi__2CuLiw&childStage=&stageExecId=

K8:https://qa.harness.io/ng/account/h61p38AZSV6MzEkpWWBtew/all/orgs/default/projects/Devansh_Test/pipelines/kanikoecrtest/executions/nM7k6O0fQwOIlgTqNrccZg/pipeline?step=Ls8YdXqJRLqiNcHZgTvPPg&stage=2W7tg7EsSyyt3_4hutHkdA&childStage=&stageExecId=

execution link for kaniko base image in plugin step(Only the 2nd parallel step): https://qa.harness.io/ng/account/h61p38AZSV6MzEkpWWBtew/all/orgs/default/projects/D[…]oD-opcEg&stage=Q1EOchtCTD6DmImmvMyQHQ&childStage=&stageExecId=

Screenshot 2025-02-24 at 2 54 06 PM

•Tested Kaniko base image in the plugin step, and it provided the digest in output variables.
•Unable to test in the OOTB step because images were still being pulled through Buildx, even after updating them via API.
•Attempted to disable a feature flag in Harness0 to stop Buildx from pulling images, but it continued to do so.
•Tried testing locally but encountered an error previously found by Abhay; attempts to fix it were unsuccessful.
•Since the images worked in the plugin step, they should be backward compatible and not cause any issues.

Copy link
Member

@Ompragash Ompragash left a comment

Choose a reason for hiding this comment

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

@DevanshMathur19 can you attach a link to the test execution where the updated change is tested and the digest is exported as output var?

Copy link
Member

@Ompragash Ompragash left a comment

Choose a reason for hiding this comment

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

build the test image - and update it via api so that it is used in OOTB step. Use K8s flow with both docker and ecr steps and verify the digest output var

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