Skip to content
This repository has been archived by the owner on Feb 22, 2022. It is now read-only.

GH Actions to automate gh-pages #23921

Merged
merged 3 commits into from
Oct 19, 2020

Conversation

scottrigby
Copy link
Member

Relates to #23850 (explanatory blog post forthcoming).

  • Charts are auto-synced to gh-pages hosted chart repos.
  • Chart repo READMEs are auto-synced to gh-pages on any changes to these files in master. This will allow changes to gh-pages READMEs using the normal PR process (including access) on master.

See https://github.com/scottrigby/chart-repo-gh-pages for a vanilla working example.

Notes:

  • There have been some packages published between when Add stable package history #23913 and Add incubator package history #23911 were merged and now. Filling in the package history gap will be.a follow-up PR once this one is merged.
  • This PR doesn't remove test/repo-sync.sh, since for now we want to continue publishing both to GCR and gh-pages. @viglesiasce this is where our timeline/plan comes in. Can be addressed in follow-up PRs.
  • Some of these run steps could be moved into local actions, but it didn't seem as though it would help much here, and I hope the shell is clear all in one place.

@scottrigby scottrigby self-assigned this Oct 19, 2020
@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Oct 19, 2020
Copy link
Member

@cpanato cpanato left a comment

Choose a reason for hiding this comment

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

/lgtm
/hold for more reviews

@k8s-ci-robot k8s-ci-robot added do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. lgtm Indicates that a PR is ready to be merged. labels Oct 19, 2020
Copy link
Member

@unguiculus unguiculus left a comment

Choose a reason for hiding this comment

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

Great job 🎉

.github/workflows/sync-repos.yaml Outdated Show resolved Hide resolved
@k8s-ci-robot k8s-ci-robot removed the lgtm Indicates that a PR is ready to be merged. label Oct 19, 2020
@helm-bot helm-bot added Contribution Allowed If the contributor has signed the DCO or the CNCF CLA (prior to the move to a DCO). size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Oct 19, 2020
@bgrant0607
Copy link

Ref helm/community#114

@unguiculus
Copy link
Member

/lgtm
/hold for more reviews

@k8s-ci-robot k8s-ci-robot added the lgtm Indicates that a PR is ready to be merged. label Oct 19, 2020
@paulczar
Copy link
Collaborator

/unhold

@k8s-ci-robot k8s-ci-robot removed the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Oct 19, 2020
@paulczar
Copy link
Collaborator

/lgtm

@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cpanato, paulczar, scottrigby, unguiculus

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:
  • OWNERS [cpanato,paulczar,scottrigby,unguiculus]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. Contribution Allowed If the contributor has signed the DCO or the CNCF CLA (prior to the move to a DCO). lgtm Indicates that a PR is ready to be merged. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants