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

[PR #827/c8a33c71 backport][stable-5] Fix helm tests #828

Conversation

patchback[bot]
Copy link

@patchback patchback bot commented Dec 13, 2024

This is a backport of PR #827 as merged into main (c8a33c7).

SUMMARY

Some of the charts we've used for testing are no longer available at the old helm repository urls, as they've been moved to oci registries. This updates those charts.

In the longer term, we should find a better way to handle these kinds of test fixtures, probably by switching to local charts as much as possible.

ISSUE TYPE
  • Bugfix Pull Request
  • Docs Pull Request
  • Feature Pull Request
  • New Module Pull Request
COMPONENT NAME
ADDITIONAL INFORMATION

SUMMARY

Some of the charts we've used for testing are no longer available at the old helm repository urls, as they've been moved to oci registries. This updates those charts.
In the longer term, we should find a better way to handle these kinds of test fixtures, probably by switching to local charts as much as possible.

ISSUE TYPE

Bugfix Pull Request
Docs Pull Request
Feature Pull Request
New Module Pull Request

COMPONENT NAME

ADDITIONAL INFORMATION

Reviewed-by: Helen Bailey <[email protected]>
Reviewed-by: Yuriy Novostavskiy
(cherry picked from commit c8a33c7)
@patchback patchback bot mentioned this pull request Dec 13, 2024
Copy link

@abikouo abikouo added mergeit test-all-the-targets In order to tell to the integration test splitter to test all targets labels Dec 16, 2024
Copy link

Build succeeded (gate pipeline).
https://ansible.softwarefactory-project.io/zuul/buildset/e53331a0c260406bb617c6466bb9beab

✔️ ansible-galaxy-importer SUCCESS in 5m 31s
✔️ build-ansible-collection SUCCESS in 5m 37s

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit e1f52dd into stable-5 Dec 16, 2024
77 of 85 checks passed
@yurnov yurnov mentioned this pull request Dec 26, 2024
30 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mergeit test-all-the-targets In order to tell to the integration test splitter to test all targets
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants