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

Add new Helm Chart OWNERS file #1547

Merged
merged 1 commit into from
Jul 17, 2024

Conversation

jfdenise
Copy link
Contributor

In preparation of the addition of the Helm Chart for EAP XP5, can you add the OWNERS for the eap-xp5 chart please?

Signed-off-by: Jean-Francois Denise [email protected]

Copy link
Contributor

🔓 Maintainers:

The OWNERS file contributed here has a chart name that is AVAILABLE!

The chart name 'redhat-eap-xp5' does not appear in our lockfile.

After reviewing this pull request, please re-run this workflow once more before merging.

This comment was auto-generated by GitHub Actions.

Copy link
Contributor

Thank you for submitting PR #1547 for Helm Chart Certification!

Outcome:

Failed

Detail

One or more errors were found with the pull request:

[INFO] OWNERS file changes require manual review by maintainers.

Task Insights

Here are the outcomes of tasks driving this result.

task outcome
PR Content Check failure
Run Chart Verifier skipped
Result Verification skipped
OpenShift Client Installation skipped

For information on the certification process see:

/metadata {"vendor_label": "", "chart_name": ""}

@jfdenise
Copy link
Contributor Author

I see that the workflow is failing, the chart tgz file will be added today in another PR. I hope all is fine.

Copy link
Collaborator

@komish komish left a comment

Choose a reason for hiding this comment

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

/lgtm

@komish komish merged commit 497fc75 into openshift-helm-charts:main Jul 17, 2024
10 of 11 checks passed
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.

2 participants