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

Build a Helm chart for the submission service #13

Closed
timberhill opened this issue May 21, 2023 · 3 comments · Fixed by #37
Closed

Build a Helm chart for the submission service #13

timberhill opened this issue May 21, 2023 · 3 comments · Fixed by #37
Assignees
@timberhill

Comments

@timberhill
Copy link
Contributor

We need a Helm chart that can deploy the submission service with a simple config.

Acceptance criteria:

  • The chart can deploy the submission service
  • You can set a service account for it to run as
  • Additional values like nodeSelector and attributes and resources can be set via values
@timberhill timberhill assigned timberhill and unassigned timberhill Jun 9, 2023
@timberhill timberhill self-assigned this Jun 26, 2023
@timberhill
Copy link
Contributor Author

See Helm chart for the comment service. This one should be similar (if possible) as they have similar functions - e.g., reading mongodb secrets.

@timberhill timberhill removed their assignment Jun 27, 2023
@timberhill timberhill mentioned this issue Jun 29, 2023
Closed
@timberhill timberhill assigned timberhill and unassigned timberhill Aug 11, 2023
@timberhill timberhill self-assigned this Sep 15, 2023
timberhill added a commit that referenced this issue Sep 15, 2023
@timberhill
#13 fixed autoscaling in comment service
b9e745e
timberhill added a commit that referenced this issue Sep 15, 2023
@timberhill
#13 added submission service helm chart
5b7ad86
@timberhill timberhill mentioned this issue Sep 15, 2023
Merged
2 tasks
timberhill added a commit that referenced this issue Sep 15, 2023
@timberhill
Merge pull request #32 from flam-flam/issue/13
3a7027d 
#13 add a submission service helm chart
timberhill added a commit that referenced this issue Sep 15, 2023
@timberhill
#13 comment service version bump
0fc33f5
timberhill added a commit that referenced this issue Sep 15, 2023
@timberhill
Merge pull request #33 from flam-flam/issue/13
8569bc6 
#13 comment service version bump
timberhill added a commit that referenced this issue Sep 15, 2023
@timberhill
#13 temporarily downgrade umbrella chart, until the dependency is rel…
93f99c7 
…eased
timberhill added a commit that referenced this issue Sep 15, 2023
@timberhill
Merge pull request #34 from flam-flam/issue/13
57db408 
#13 temporarily downgrade umbrella chart, until the dependency is rel…
timberhill added a commit that referenced this issue Sep 15, 2023
@timberhill
#13 temporarily downgrade umbrella chart, until the dependency is rel…
9a3ee7c 
…eased
timberhill added a commit that referenced this issue Sep 15, 2023
@timberhill
Merge pull request #35 from flam-flam/issue/13
5e8d0fd 
#13 temporarily downgrade umbrella chart, until the dependency is rel…
timberhill added a commit that referenced this issue Sep 15, 2023
@timberhill
#13 add submission service to umbrella
53ab75c
timberhill added a commit that referenced this issue Sep 15, 2023
@timberhill
Merge pull request #36 from flam-flam/issue/13
1986eed 
#13 add submission service to umbrella
@timberhill
Copy link
Contributor Author

Well, there is an issue with the environment variable names here. Submission service expects them to have a colon and k8s doesn't allow them: kubernetes/kubernetes#53201

@timberhill
Copy link
Contributor Author

Oh, apparently you can use __ instead of :. Still failing the liveness probes, but the env variable is passed on to the config -- tested by writing values to console.

timberhill added a commit that referenced this issue Sep 15, 2023
@timberhill
#13 fixed submission service helm chart
3e8516c
@timberhill timberhill mentioned this issue Sep 15, 2023
Merged
3 tasks
timberhill added a commit that referenced this issue Sep 15, 2023
@timberhill
Merge pull request #37 from flam-flam/issue/13
aa2f159 
#13 fixed submission service helm chart
@timberhill timberhill mentioned this issue Sep 15, 2023
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@timberhill timberhill

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

#13 fixed submission service helm chart flam-flam/helm-charts
1 participant
@timberhill