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

feat(deploy): add helm chart #138

Merged
merged 33 commits into from
Aug 3, 2023
Merged

feat(deploy): add helm chart #138

merged 33 commits into from
Aug 3, 2023

Conversation

corrieriluca
Copy link
Member

Closes #137

@corrieriluca corrieriluca self-assigned this Jul 10, 2023
@corrieriluca corrieriluca force-pushed the deploy/helm-chart branch 2 times, most recently from 34002e1 to d2b443e Compare July 12, 2023 09:20
@codecov-commenter
Copy link

codecov-commenter commented Jul 12, 2023 •
edited
Loading

Codecov Report

Merging #138 (9539ea9) into main (6f45c0b) will increase coverage by 0.03%.
The diff coverage is 58.33%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@            Coverage Diff             @@
##             main     #138      +/-   ##
==========================================
+ Coverage   69.67%   69.71%   +0.03%     
==========================================
  Files          23       23              
  Lines        1560     1565       +5     
==========================================
+ Hits         1087     1091       +4     
- Misses        411      412       +1     
  Partials       62       62
Files Changed Coverage Δ
...nal/controllers/terraformpullrequest/controller.go 34.92% <0.00%> (ø)
internal/burrito/config/config.go 61.76% <20.00%> (-0.93%) ⬇️
internal/controllers/terraformlayer/pod.go 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@corrieriluca
chore(deploy): split redis URL in host + port
023808d
@corrieriluca
fix(cli): make Redis flags global
21ab2ce
@corrieriluca
fix(redis): rename port field
4ee9b2a
@corrieriluca
fix(runner): try log layer
c424ddf
@corrieriluca
fix: trigger ci
7fd33d1
@corrieriluca
fix(runner): fix get layer
1ca3d0a
@corrieriluca
fix(runner): fix logs layer/repo
ebe723e
@corrieriluca
chore(deploy): fix RBAC for burrito-runner + ssh known hosts
681f79f
@corrieriluca
chore(deploy): add TODO on CM reloading
de821ad
@corrieriluca
chore(deploy): helm: add checksums for configmap
7ca7b15
@corrieriluca
chore(deploy): helm: use appVersion for image tag
cee2f0e
@corrieriluca
ci(helm): add CI/CD for Helm chart
d089fe8
@corrieriluca
ci(helm): test Helm rendering
ebf0b35
@corrieriluca
ci(helm): make Helm render mandatory
5c963dd
@corrieriluca
fix(helm): appVersion references
026365f
@corrieriluca
ci(helm): trigger on Helm chart files for PR
fe75390
@corrieriluca
docs(burrito): fix some typos and markdown lint
98716b1
@corrieriluca
chore(manifests): update references to burrito namespace
af60980
@corrieriluca
fix(cmd): add --namespaces flag + update doc
8bf2e5b
@spoukke spoukke changed the title chore(deploy): Add Helm Chart feat(deploy): add helm chart Aug 2, 2023
spoukke
spoukke previously requested changes Aug 2, 2023
View reviewed changes
.github/workflows/helm.yaml Outdated Show resolved Hide resolved
.github/workflows/helm.yaml Outdated Show resolved Hide resolved
deploy/charts/burrito/templates/runner.yaml Outdated Show resolved Hide resolved
deploy/charts/burrito/values.yaml Outdated Show resolved Hide resolved
@corrieriluca corrieriluca mentioned this pull request Aug 2, 2023
Merged
@Alan-pad Alan-pad dismissed spoukke’s stale review August 3, 2023 10:12

Bug with Github :(

@Alan-pad Alan-pad merged commit 8f98413 into main Aug 3, 2023
@corrieriluca corrieriluca deleted the deploy/helm-chart branch November 4, 2023 16:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Alan-pad Alan-pad Alan-pad approved these changes

@spoukke spoukke Awaiting requested review from spoukke

Assignees

@corrieriluca corrieriluca

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Deploy] Helm chart
4 participants
@corrieriluca @codecov-commenter @spoukke @Alan-pad