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.

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

Kubernetes jobs.yml template ignores RUN_FORUM #525

Closed
fghaas opened this issue Nov 17, 2021 · 0 comments · Fixed by #526
Closed

Kubernetes jobs.yml template ignores RUN_FORUM #525

fghaas opened this issue Nov 17, 2021 · 0 comments · Fixed by #526

Comments

@fghaas
Copy link
Contributor

fghaas commented Nov 17, 2021

Bug description
When running tutor k8s we can disable the forum deployment and service by setting RUN_FORUM to false, but the corresponding job is added to jobs.yml unconditionally.

How to reproduce

  • In a fresh Tutor environment, set RUN_FORUM: False in tutor/config.yml
  • Run tutor k8s quickstart
  • Observe that there is no mention of forum in $TUTOR_ROOT/env/k8s/deployments.yml or services.yml, but there is a job named forum-job in jobs.yml.

Environment

  • Tutor 12.1.6

(The fix for this should be trivial; PR incoming.)

fghaas added a commit to fghaas/tutor that referenced this issue Nov 17, 2021
When running "tutor k8s" we can disable the forum deployment and
service by setting RUN_FORUM to false, but the corresponding job is
added to jobs.yml unconditionally.

Add a conditional to templates/k8s/jobs.yml so that the forum job
definition is only included when RUN_FORUM is true.

Fixes overhangio#525.
fghaas added a commit to fghaas/tutor that referenced this issue Nov 17, 2021
When running "tutor k8s" we can disable the forum deployment and
service by setting RUN_FORUM to false, but the corresponding job is
added to jobs.yml unconditionally.

Add a conditional to tutor/templates/k8s/jobs.yml so that the forum
job definition is only included when RUN_FORUM is true.

Fixes overhangio#525.
fghaas added a commit to fghaas/tutor that referenced this issue Nov 17, 2021
When running "tutor k8s" we can disable the forum deployment and
service by setting RUN_FORUM to false, but the corresponding job is
added to jobs.yml unconditionally.

Add a conditional to tutor/templates/k8s/jobs.yml so that the forum
job definition is only included when RUN_FORUM is true.

Fixes overhangio#525.
fghaas added a commit to fghaas/tutor that referenced this issue Nov 18, 2021
When running "tutor k8s" we can disable the forum deployment and
service by setting RUN_FORUM to false, but the corresponding job is
added to jobs.yml unconditionally.

Add a conditional to tutor/templates/k8s/jobs.yml so that the forum
job definition is only included when RUN_FORUM is true.

Fixes overhangio#525.
regisb pushed a commit that referenced this issue Nov 18, 2021
When running "tutor k8s" we can disable the forum deployment and
service by setting RUN_FORUM to false, but the corresponding job is
added to jobs.yml unconditionally.

Add a conditional to tutor/templates/k8s/jobs.yml so that the forum
job definition is only included when RUN_FORUM is true.

Fixes #525.
regisb added a commit that referenced this issue Nov 18, 2021
- [Security] Timed exam security fix [29347](https://github.com/edx/edx-platform/pull/29347).
- [Feature] Add [tutor-richie](https://github.com/overhangio/tutor-richie) to the plugins that are bundled with the tutor binary.
- [Improvement] Make `tutor plugins list` print plugins sorted by name.
- [Improvement] Ignore Python plugins which cannot be loaded.
- [Bugfix] When configured with `RUN_FORUM: false`, omit forum-related [Jobs](https://kubernetes.io/docs/concepts/workloads/controllers/job/) from the manifests that `tutor k8s` generates. (#525)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant