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.

Sign up for GitHub

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

Jump to bottom

scrub mozilla-it refs, setup GHA for CI/CD of Common Voice helm charts #1

Merged
merged 3 commits into from
Nov 23, 2021
Merged

scrub mozilla-it refs, setup GHA for CI/CD of Common Voice helm charts #1

merged 3 commits into from
Nov 23, 2021

Conversation

cmharlow
Copy link
Contributor

@cmharlow cmharlow commented Oct 25, 2021 •
edited
Loading

Jira ticket: https://mozilla-hub.atlassian.net/browse/SE-2206

What this PR does:

  • Migrates common voice & sentence collector helm charts (with git history) from mozilla-it/helm-charts GitHub repository to here
  • updates CI & CD to use GitHub actions & run https://github.com/helm/chart-testing for linting & e2e tests on charts, GitHub Pages - Helm publication steps for publishing helm charts via GitHub Actions

Manually testing the GitHub Pages / Helm Chart publication now.

@cmharlow cmharlow force-pushed the SE-2206 branch 14 times, most recently from 7746563 to 0852e1d Compare October 25, 2021 22:13
@cmharlow cmharlow changed the title scrub readme of mozilla-it refs scrub mozilla-it refs, setup GHA for CI/CD of Common Voice helm charts Oct 25, 2021
@cmharlow cmharlow marked this pull request as ready for review October 25, 2021 22:20
@cmharlow cmharlow mentioned this pull request Oct 25, 2021
Merged
@jencohoon
Copy link
Contributor

@cmharlow - not sure if we need to / should make jenny a blocking reviewer on these mostly infra PRs? Jenny is 1/2 time on CV at NVDIA
If we're going to need these tomorrow, the it should probably just be me jzinner

phirework
phirework approved these changes Oct 27, 2021
View reviewed changes
.github/workflows/release.yml Show resolved Hide resolved
@cmharlow cmharlow merged commit 02f88de into main Nov 23, 2021
@cmharlow cmharlow deleted the SE-2206 branch November 23, 2021 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jencohoon jencohoon jencohoon approved these changes

@phirework phirework phirework approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@cmharlow @jencohoon @phirework