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

fix(ci/artifacthub-images): the artifacthub-ignore comments get lost, #831

Merged
merged 4 commits into from
Apr 12, 2024

Conversation

cwrau
Copy link
Member

@cwrau cwrau commented Mar 25, 2024

therefore the developer should just override the image via the artifacthub-values.yaml to artifacthub-ignore

This also forces the detected images to end the line, which conveniently throws out the charts/t8s-cluster/templates/management-cluster/clusterClass/openStackClusterTemplate/_openStackClusterTemplateSpec.yaml ignored image

@cwrau cwrau requested a review from a team as a code owner March 25, 2024 16:47
@cwrau cwrau requested a review from marvinWolff March 25, 2024 16:47
@cwrau cwrau enabled auto-merge (squash) March 25, 2024 16:47
@cwrau cwrau force-pushed the fix/ci-artifacthub-ignored-images branch from e777c23 to 193dfd4 Compare March 25, 2024 16:49
@cwrau cwrau force-pushed the fix/ci-artifacthub-ignored-images branch from 193dfd4 to f9297d2 Compare March 25, 2024 16:49
@cwrau cwrau changed the title fix(ci/artifacthub-images): the comments get lost, therefore fix(ci/artifacthub-images): the artifacthub-ignore comments get lost, Mar 25, 2024
marvinWolff
marvinWolff previously approved these changes Mar 28, 2024
@cwrau cwrau disabled auto-merge March 28, 2024 13:46
@cwrau cwrau force-pushed the fix/ci-artifacthub-ignored-images branch from b34e1ac to 3094734 Compare March 28, 2024 13:48
@teutonet-bot
Copy link
Contributor

🤖 I have diffed this beep boop

"/$namespace/$kind/$name.yaml" for normal resources
"/$namespace/HelmRelease/$name/$namespace/$kind/$name.yaml" for HelmReleases <- this is recursive
'null' means it's either cluster-scoped or it's in the default namespace for the HelmRelease

charts/t8s-cluster/ci/artifacthub-values.yaml

charts/t8s-cluster/ci/cidrs-values.yaml has no changes

charts/t8s-cluster/ci/injectedCertificateAuthorities-values.yaml

charts/t8s-cluster/ci/securityGroups-values.yaml

charts/t8s-cluster/ci/calico-values.yaml

charts/t8s-cluster/ci/bastion-values.yaml has no changes

charts/t8s-cluster/values.yaml

charts/t8s-cluster/ci/gpu-flavor-values.yaml

@cwrau cwrau force-pushed the fix/ci-artifacthub-ignored-images branch from 3094734 to c1b6132 Compare March 28, 2024 13:53
@cwrau cwrau enabled auto-merge (squash) March 28, 2024 13:56
cwrau added 4 commits April 11, 2024 09:31
therefore the developer should just override the image via the
artifacthub-values.yaml to `artifacthub-ignore`
which conveniently throws out the `charts/t8s-cluster/templates/management-cluster/clusterClass/openStackClusterTemplate/_openStackClusterTemplateSpec.yaml`
ignored image
@cwrau cwrau force-pushed the fix/ci-artifacthub-ignored-images branch from c1b6132 to 36af82c Compare April 11, 2024 07:31
@cwrau cwrau merged commit afacd7f into main Apr 12, 2024
6 checks passed
@cwrau cwrau deleted the fix/ci-artifacthub-ignored-images branch April 12, 2024 08:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants