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

apply refactorization of coda k8s/helm infra constructs #6111

Merged
merged 29 commits into from
Sep 30, 2020
Merged

apply refactorization of coda k8s/helm infra constructs #6111

merged 29 commits into from
Sep 30, 2020

Conversation

O1ahmad
Copy link
Contributor

@O1ahmad O1ahmad commented Sep 22, 2020 •
edited
Loading

Per #5641, this change implements Coda helm chart placement and versioning refactorizations including:

  • migration of infra src from coda-automation to coda repo (in order to be co-located with corresponding application src)
  • individual versioning + metadata setting of Coda component Helm charts
  • creation of single bots Helm chart to be utilized by o1-bot types (e.g. faucet, echo, user-agent)
  • addition of Buildkite Helm release verification and overall management jobs

Testing: Buildkite CI

Checklist:

  • Document code purpose, how to use it
    • Mention expected invariants, implicit constraints
  • Tests were added for the new behavior
    • Document test purpose, significance of failures
    • Test names should reflect their purpose
  • All tests pass (CI will check this if you didn't)

Closes #5641

@O1ahmad O1ahmad requested review from a team as code owners September 22, 2020 22:08
@O1ahmad O1ahmad self-assigned this Sep 22, 2020
@O1ahmad O1ahmad changed the title [WIP] apply initial refactorization of coda k8s/helm infra constructs [WIP] apply refactorization of coda k8s/helm infra constructs Sep 22, 2020
@O1ahmad O1ahmad added the ci-build-me Add this label to trigger a circle+buildkite build for this branch label Sep 24, 2020
@O1ahmad O1ahmad changed the title [WIP] apply refactorization of coda k8s/helm infra constructs apply refactorization of coda k8s/helm infra constructs Sep 25, 2020
@netlify
Copy link

netlify bot commented Sep 29, 2020 •
edited
Loading

Deploy preview for mina-staging ready!

Built with commit 76f3c7e

https://deploy-preview-6111--mina-staging.netlify.app

@netlify
Copy link

netlify bot commented Sep 29, 2020

Preview:

Built with commit 7e384ad

https://deploy-preview-6111--o1website2.netlify.app

@O1ahmad
apply initial refactorization of coda k8s/helm infra constructs
e9a4dba
@O1ahmad
add initial example README and more updates
327a11a
@O1ahmad
fix helm linting errors
4ed84b2
@O1ahmad
add node_port to buildkite exporter service and refactor values
bff35dd
@O1ahmad
port buildkite exporter changes from coda-automation
605d295
@O1ahmad
update buildkite-exporter README and refactor listening port var
0a52ad6
@O1ahmad
add volume/volumeMount template vars to bot helm chart + README
679fe64
@O1ahmad
README updates
37d0dc0
@O1ahmad
more README updates
2ca6471
@O1ahmad
add helm lint script (looks for Chart.yaml in DIR of changed files)
ee19b43
@O1ahmad
update leaderboard helm README
60000e1
@O1ahmad
add dry-run and initial chart version bump checks and fix existing er…
f68b1ea 
…rors
@O1ahmad
update helm install logic in toolchain dockerfile
83fb257
@O1ahmad
add ports var value to o1-bot chart
ca281eb
@O1ahmad
execute helm lint steps from agent
9c35f81
@O1ahmad
change helm lint dry-run to operate on default namespace
0e0122f
@O1ahmad
add dir labeling to helm lint operation announce
52eab87
@O1ahmad
reduce helm lint agent target sizing to Small
84891c0
@O1ahmad
add helm release script
530086a
@O1ahmad
add helm release dependency on linting step
c5c2ae0
@O1ahmad O1ahmad force-pushed the helm_refactor branch 3 times, most recently from 32d2a5b to 2e968fd Compare September 30, 2020 21:12
@O1ahmad O1ahmad merged commit d600777 into develop Sep 30, 2020
@O1ahmad O1ahmad deleted the helm_refactor branch September 30, 2020 21:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@lk86 lk86 lk86 approved these changes

Assignees

@O1ahmad O1ahmad

Labels
ci-build-me Add this label to trigger a circle+buildkite build for this branch
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Refactor Helm/Terraform Versioning
2 participants
@O1ahmad @lk86