Skip to content
This repository has been archived by the owner on Dec 3, 2021. It is now read-only.

Support option to specify curriculum version #120

Merged
merged 3 commits into from
Jul 30, 2019

Conversation

Mierdin
Copy link
Member

@Mierdin Mierdin commented Jul 27, 2019

Since we're now releasing the curriculum separately from the platform, we're also taking the opportunity to build and tag all curriculum images for each release. As a result, it is necessary to be able to specify which version of the curriculum Syringe is loading, so that it can be sure to request the correct tag for all endpoint versions.

This PR does this by adding an environment variable to Syringe called SYRINGE_CURRICULUM_VERSION, which defaults to latest, but in production, should be set to the exact tag used for curriculum releases, and associated curriculum image build tags.

Signed-off-by: Matt Oswalt <matt@keepingitclassless.net>
Mierdin added 2 commits July 26, 2019 17:28
Signed-off-by: Matt Oswalt <matt@keepingitclassless.net>
Signed-off-by: Matt Oswalt <matt@keepingitclassless.net>
@codecov
Copy link

codecov bot commented Jul 29, 2019

Codecov Report

Merging #120 into master will decrease coverage by 0.06%.
The diff coverage is 50%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #120      +/-   ##
==========================================
- Coverage   26.82%   26.75%   -0.07%     
==========================================
  Files          20       20              
  Lines        2032     2037       +5     
==========================================
  Hits          545      545              
- Misses       1437     1441       +4     
- Partials       50       51       +1
Impacted Files Coverage Δ
api/exp/lessons.go 3.05% <0%> (-0.05%) ⬇️
scheduler/pods.go 80.76% <100%> (-0.54%) ⬇️
config/config.go 58.75% <60%> (+0.08%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5ae5d63...fc92c4b. Read the comment docs.

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant