Skip to content

fix(tenantresources): nil pointer for additionalmetadata #1413

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

Merged
merged 2 commits into from
Apr 11, 2025

Conversation

Svarrogh1337
Copy link
Collaborator

Fixes #1387

Copy link

netlify bot commented Apr 7, 2025

Deploy Preview for capsule-documentation canceled.

Name Link
🔨 Latest commit 6e47c69
🔍 Latest deploy log https://app.netlify.com/sites/capsule-documentation/deploys/67f8d7d915dbc10008796f94

@Svarrogh1337 Svarrogh1337 changed the title Fix projectcapsule/capsule#1387 fix: nil pointer for additionalMetadata Apr 7, 2025
@Svarrogh1337 Svarrogh1337 force-pushed the fix-1387 branch 2 times, most recently from 24755d2 to 185c1bb Compare April 7, 2025 12:30
@Svarrogh1337 Svarrogh1337 changed the title fix: nil pointer for additionalMetadata fix: nil pointer for additionalMetadata Apr 7, 2025
Copy link
Member

@prometherion prometherion left a comment

Choose a reason for hiding this comment

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

Just a small review, but LGTM.

@Svarrogh1337 Svarrogh1337 changed the title fix: nil pointer for additionalMetadata fix(tenantresources): nil pointer for additionalMetadata Apr 7, 2025
@Svarrogh1337 Svarrogh1337 changed the title fix(tenantresources): nil pointer for additionalMetadata fix(tenantresources): nil pointer for additionalmetadata Apr 7, 2025
@Svarrogh1337
Copy link
Collaborator Author

@prometherion, did the changes satisfy your review?

Copy link
Member

@prometherion prometherion left a comment

Choose a reason for hiding this comment

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

Thanks for the heads up, LGTM! Let's wait the green ci

@Svarrogh1337
Copy link
Collaborator Author

@prometherion after the CI fix there are still problems with the CI unrelated to this PR - https://github.com/projectcapsule/capsule/actions/runs/14358268752
Perhaps it's the next thing I can check.

@oliverbaehler
Copy link
Collaborator

@Svarrogh1337 sure, OCI dependencies on github are so ****. I think we need to perform an helm login with the ci credentials and increase the timeiut for the helm operation to fix this

@Svarrogh1337
Copy link
Collaborator Author

@prometherion @oliverbaehler here we go. e2e was successful after merging my previous changes from main.

@prometherion
Copy link
Member

Amazing job, Hristo, love witnessing your efforts in getting Capsule fixed: LGTM!

@oliverbaehler oliverbaehler merged commit a440a59 into projectcapsule:main Apr 11, 2025
18 checks passed
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.

bug: nil pointer for additionalMetadata
3 participants