Skip to content

Removing nns-docker partner #1527

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

Conversation

devopsnns
Copy link
Contributor

@devopsnns devopsnns commented Jul 2, 2024

Removing nns-docker partner since it does not represent our real entity name and was derived from somewhere, need to name change it to nonamesec in a different PR

Copy link
Contributor

github-actions bot commented Jul 2, 2024

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

Outcome:

Failed

Detail

One or more errors were found with the pull request:

[ERROR] PR includes one or more files not related to charts, e.g., charts/partners/nns-docker/nonamesec/OWNERS

[ERROR] OWNERS file should never be set directly by partners. See certification docs.

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": "nns-docker", "chart_name": "nonamesec-ubi"}

@devopsnns devopsnns changed the title Removing nns-docker partner Removing "nns-docker" partner since it does not represent our real entity name and was derived from somewhere, need to name change it to nonamesec in a different PR Jul 2, 2024
@devopsnns devopsnns changed the title Removing "nns-docker" partner since it does not represent our real entity name and was derived from somewhere, need to name change it to nonamesec in a different PR Removing nns-docker partner Jul 2, 2024
Copy link
Contributor

github-actions bot commented Jul 2, 2024

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

Outcome:

Failed

Detail

One or more errors were found with the pull request:

[ERROR] PR includes one or more files not related to charts, e.g., charts/partners/nns-docker/nonamesec/OWNERS

[ERROR] OWNERS file should never be set directly by partners. See certification docs.

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": "nns-docker", "chart_name": "nonamesec-ubi"}

2 similar comments
Copy link
Contributor

github-actions bot commented Jul 2, 2024

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

Outcome:

Failed

Detail

One or more errors were found with the pull request:

[ERROR] PR includes one or more files not related to charts, e.g., charts/partners/nns-docker/nonamesec/OWNERS

[ERROR] OWNERS file should never be set directly by partners. See certification docs.

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": "nns-docker", "chart_name": "nonamesec-ubi"}

Copy link
Contributor

github-actions bot commented Jul 2, 2024

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

Outcome:

Failed

Detail

One or more errors were found with the pull request:

[ERROR] PR includes one or more files not related to charts, e.g., charts/partners/nns-docker/nonamesec/OWNERS

[ERROR] OWNERS file should never be set directly by partners. See certification docs.

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": "nns-docker", "chart_name": "nonamesec-ubi"}

@mgoerens
Copy link
Contributor

mgoerens commented Jul 3, 2024

Closing in favor of #1535

@mgoerens mgoerens closed this Jul 3, 2024
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.

2 participants