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

simple-app-build-local example builds the same image twice #243

Open
benmoss opened this issue May 9, 2022 · 2 comments
Open

simple-app-build-local example builds the same image twice #243

benmoss opened this issue May 9, 2022 · 2 comments
Labels
bug This issue describes a defect or unexpected behavior carvel accepted This issue should be considered for future work and that the triage process has been completed good first issue An issue that will be a good candidate for a new contributor priority/unprioritized-backlog Higher priority than priority/awaiting-more-evidence. Contributions are welcome.

Comments

@benmoss
Copy link

benmoss commented May 9, 2022

What steps did you take:

kbld -f examples/simple-app-build-local/build.yml
simple-app | starting build (using Docker): test/e2e/assets/simple-app -> kbld:rand-1652108946742255592-1387725203109-simple-app
simple-app | starting build (using Docker): test/e2e/assets/simple-app -> kbld:rand-1652108946743073128-10522560189175-simple-app
...
resolve | final: simple-app -> kbld:simple-app-sha256-d4ca5dac9eb3959e041a367997debe1d42a8a08d2a34a885d56b65910201a611
resolve | final: simple-app2 -> kbld:simple-app-sha256-d4ca5dac9eb3959e041a367997debe1d42a8a08d2a34a885d56b65910201a611

What happened:
It builds the same image twice

What did you expect:
It only builds the image once

Anything else you would like to add:
[Additional information that will assist in solving the issue.]

Environment:

  • kbld version (use kbld --version): v0.33.0

Vote on this request

This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.

👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"

We are also happy to receive and review Pull Requests if you want to help working on this issue.

@benmoss benmoss added bug This issue describes a defect or unexpected behavior carvel triage This issue has not yet been reviewed for validity labels May 9, 2022
@github-actions
Copy link

This issue is being marked as stale due to a long period of inactivity and will be closed in 5 days if there is no response.

@github-actions github-actions bot added the stale This issue has had no activity for a while and will be closed soon label Jun 19, 2022
@benmoss benmoss removed the stale This issue has had no activity for a while and will be closed soon label Jun 22, 2022
@joaopapereira joaopapereira added good first issue An issue that will be a good candidate for a new contributor carvel accepted This issue should be considered for future work and that the triage process has been completed priority/unprioritized-backlog Higher priority than priority/awaiting-more-evidence. Contributions are welcome. and removed carvel triage This issue has not yet been reviewed for validity labels Jun 23, 2022
@jignyasamishra
Copy link

Hello! @joaopapereira @praveenrewar . I recreated the above issue on my local system on two separate occassions and i got two different shasum's

resolve | final: simple-app -> kbld:simple-app-sha256-2d246faa9fec0e06584274d17d1336e114f2bcf3342fac699e6aa894288baee8
resolve | final: simple-app2 -> kbld:simple-app-sha256-c9e22332a5ec65dc311ff8bc41527ec16a5ef077abd4700d9fd0e12eaecdab34
resolve | final: simple-app -> kbld:simple-app-sha256-c9e22332a5ec65dc311ff8bc41527ec16a5ef077abd4700d9fd0e12eaecdab34
resolve | final: simple-app2 -> kbld:simple-app-sha256-c9e22332a5ec65dc311ff8bc41527ec16a5ef077abd4700d9fd0e12eaecdab34

the outputs are different.
I think here we should have two images but both should be different at every execution, any thoughts on why it's flaking?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug This issue describes a defect or unexpected behavior carvel accepted This issue should be considered for future work and that the triage process has been completed good first issue An issue that will be a good candidate for a new contributor priority/unprioritized-backlog Higher priority than priority/awaiting-more-evidence. Contributions are welcome.
Projects
Status: To Triage
Development

No branches or pull requests

3 participants