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.

Sign up for GitHub

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

Jump to bottom

Fix: chart.digest is the same for all charts and not suited as unique id #1964

Merged
merged 1 commit into from Jan 19, 2021
Merged

Fix: chart.digest is the same for all charts and not suited as unique id #1964

merged 1 commit into from Jan 19, 2021

Conversation

ixrock
Copy link
Member

@ixrock ixrock commented Jan 15, 2021

Before:
image

After:
image

@ixrock ixrock added the bug Something isn't working label Jan 15, 2021
@ixrock ixrock requested a review from a team January 15, 2021 11:52
@ixrock ixrock self-assigned this Jan 15, 2021
@nevalla
Copy link
Contributor

nevalla commented Jan 15, 2021

BTW can't reproduce the issue on master branch.

image

@ixrock
Copy link
Member Author

ixrock commented Jan 15, 2021 •
edited

BTW can't reproduce the issue on master branch.

this is weird.. any ideas why it could happen? (checked again in master - still doesn't work as expected)

@Nokel81
Copy link
Collaborator

Nokel81 commented Jan 15, 2021

Screen Shot 2021-01-15 at 3 43 30 PM

This is what I see. It is mostly a "stable" repo problem but we should still fix it.

@Nokel81 Nokel81 self-requested a review January 15, 2021 20:44
@ixrock ixrock merged commit d8e088f into master Jan 19, 2021
@ixrock ixrock deleted the fix/helm-charts-listing branch January 19, 2021 12:15
@jakolehm jakolehm mentioned this pull request Jan 27, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nevalla nevalla nevalla approved these changes

@Nokel81 Nokel81 Nokel81 approved these changes

Assignees

@ixrock ixrock

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@ixrock @nevalla @Nokel81