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

vault lots of files #83

Open
wants to merge 61 commits into
base: tests
Choose a base branch
from
Open

vault lots of files #83

wants to merge 61 commits into from

Conversation

mmulholla
Copy link
Owner

No description provided.

redhat-mercury-bot and others added 30 commits May 18, 2021 09:51
The API_SERVER GitHub secret should be created with a base64 encoded
string (Warning: That means that's not really a secret -- the value will
be visible in the log)

To generate base64 encoded string for a URL:

  echo -n "https://api.ocpappsvc-osd.zn6c.p1.openshiftapps.com:6443" | base64
These cluster level rules are the minimum needed for the chart-verifier
to be able to validate the hashicorp/vault chart in our current 4.7 repo.
Adding logic to compute digest and compare with digest passed
in report. The digest will be stored at the same level as urls
field.
* workflows: add redhat and community workflow logic

As part of supporting different vendor types for openshift helm
charts, this adds the github actions for redhat and community
submission. For design details please refer to: docs/workflow-build.md.

Signed-off-by: Allen Bai <[email protected]>

* Update and factor code according to reviews

Signed-off-by: Allen Bai <[email protected]>
Ref. #200

Currently, chart-testing failure leads to empty value for
certifiedOpenShiftVersions.
There is an open issue about it here:
redhat-certification/chart-verifier#157

This fix will prioritize report verification above the
certifiedOpenShiftVersions.  This way user will see the report error
first.
…openshift-helm-charts#242)

This fix handles multi-line reason strings more nicely by removing use
of xargs and concatenating strings spanning multiple lines. Note that
this script will be completely replaced by the incoming report command
of chart-verifier.

Closes: https://issues.redhat.com/browse/HELM-211
Signed-off-by: Allen Bai <[email protected]>
@mmulholla mmulholla force-pushed the Vault0114NoOwners branch from fbf9a2b to c140645 Compare July 21, 2021 02:14
@github-actions
Copy link

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

An error was found with the Pull Request:
[ERROR] PR should only modify chart related files: charts/partners/test-org1/vault/OWNERS

/metadata {"vendor_label": "test-org1", "chart_name": "vault"}

@github-actions
Copy link

Thank you for submitting pull request #83 for Helm Chart Certification!

There were one or more errors while building and verifying your pull request.
To see the console output with the error messages, click the "Details"
link next to "CI / Chart Certification" job status towards the end of this page.


/metadata {"vendor_label": "test-org1", "chart_name": "vault"}

For support, connect with our Technology Partner Success Desk.

@github-actions
Copy link

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

An error was found with the Pull Request:
[ERROR] PR should only modify chart related files: charts/partners/test-org1/vault/OWNERS

/metadata {"vendor_label": "test-org1", "chart_name": "vault"}

@github-actions
Copy link

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

An error was found with the Pull Request:
[ERROR] PR should only modify chart related files: charts/partners/test-org1/vault/OWNERS

/metadata {"vendor_label": "test-org1", "chart_name": "vault"}

@github-actions
Copy link

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

An error was found with the Pull Request:
[ERROR] PR should only modify chart related files: charts/partners/test-org1/vault/OWNERS

/metadata {"vendor_label": "test-org1", "chart_name": "vault"}

@github-actions
Copy link

Thank you for submitting pull request #83 for Helm Chart Certification!

There were one or more errors while building and verifying your pull request.
To see the console output with the error messages, click the "Details"
link next to "CI / Chart Certification" job status towards the end of this page.


/metadata {"vendor_label": "test-org1", "chart_name": "vault"}

For support, connect with our Technology Partner Success Desk.

@github-actions
Copy link

Thank you for submitting pull request #83 for Helm Chart Certification!

There were one or more errors while building and verifying your pull request.
To see the console output with the error messages, click the "Details"
link next to "CI / Chart Certification" job status towards the end of this page.


/metadata {"vendor_label": "test-org1", "chart_name": "vault"}

For support, connect with our Technology Partner Success Desk.

@github-actions
Copy link

Thank you for submitting pull request #83 for Helm Chart Certification!

There were one or more errors while building and verifying your pull request.
To see the console output with the error messages, click the "Details"
link next to "CI / Chart Certification" job status towards the end of this page.


/metadata {"vendor_label": "test-org1", "chart_name": "vault"}

For support, connect with our Technology Partner Success Desk.

mmulholla pushed a commit that referenced this pull request Jun 7, 2022
)

* Provide the landing page content for the OpenShift Helm Charts site

The README.md under the main branch will contained a lot of details
about how to participate in the certification program. The site
README.md changed here will provide a summary and other general
information about this repository.

* edit the intro and added few informations

* Adding directory structure and program description.

Added directory structure for partners and mentioned the community
path. Also added generic description for the certification program.

* Adding new oc command to install helm repo.

Also adding the repo yaml file with the custom resource definition.

* Adding charts directory on the file structure section

* Remove community path for now, since is not supported for GA

* Taking care of review comments

Co-authored-by: David Peraza <[email protected]>
Co-authored-by: Stévan Le Meur <[email protected]>
mmulholla added a commit that referenced this pull request Jul 20, 2022
[WIP] Reusing steps through conftest.py file
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.

4 participants