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

Update index.yaml #1087

Merged
merged 1 commit into from
Nov 10, 2022
Merged

Update index.yaml #1087

merged 1 commit into from
Nov 10, 2022

Conversation

ChandlerSwift
Copy link

In the interim until #1075 is merged, the Helm package index only contains the most recent release (#1053) This commit backfills old releases into the index as a stop-gap measure until the next release, where this should (hopefully) happen automatically, once #1075 is merged.

@janorn
Copy link
Contributor

janorn commented Oct 19, 2022

I have a workaround in a repo. If you point your helm repo to https://ansible.github.io/awx-operator/index.yaml https://janorn.github.io/helmrepo-awx/index.yaml you will get all releases from the official repo.

@ChandlerSwift
Copy link
Author

ChandlerSwift commented Oct 20, 2022

It doesn't look like that's the case:

% curl https://ansible.github.io/awx-operator/index.yaml
apiVersion: v1
entries:
  awx-operator:
  - apiVersion: v2
    appVersion: 0.30.0
    created: "2022-10-03T21:00:35.413209726Z"
    description: A Helm chart for the AWX Operator
    digest: 9844a92b43af1b9fd988c6c77ce8ab2943f47d25aba77e9195f1cf6310d33373
    name: awx-operator
    type: application
    urls:
    - https://github.com/ansible/awx-operator/releases/download/0.30.0/awx-operator-0.30.0.tgz
    version: 0.30.0
generated: "2022-10-03T21:00:35.413221426Z"

Only the most recent version is shown. That's what this PR attempts to fix.

@janorn
Copy link
Contributor

janorn commented Oct 20, 2022

My mistake I gave the wrong url it's not in the fork.
Here it is the proper link https://janorn.github.io/helmrepo-awx/index.yaml

Signed-off-by: Chandler Swift <[email protected]>
@rooftopcellist
Copy link
Member

I have merged #1075, so this should be fixed automatically during the next release. As that might be a week or two, if you want to resolve the conflicts and add in 1.0.0 (if needed), I can merge this.

@ChandlerSwift
Copy link
Author

@rooftopcellist I've made those updates, thanks!

@rooftopcellist rooftopcellist merged commit b7f3148 into ansible:gh-pages Nov 10, 2022
@rooftopcellist
Copy link
Member

Thank you @ChandlerSwift !

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