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

✨(jobs) add support for pre/post deployment jobs #214

Merged
merged 2 commits into from
Dec 21, 2018

Conversation

jmaupetit
Copy link
Contributor

Purpose

Some jobs require to be run before deploying an application, hence we need a way to filter jobs and execute them before or after creating deployments.

Proposal

Filtering is based on the job_type label, pre-deployment jobs must be labelled with job_type: "pre" and post-deployment jobs must be labelled with job_type: "post". Jobs without job_type label will be considered as post-deployment jobs.

For now the only pre-deployment job is edxapp's internationalization job.

Some jobs require to be run before deploying an application, hence we
introduced a way to filter jobs and execute them before or after
creating deployments.

Filtering is based on the "job_type" label, pre-deployment jobs must be
labelled with job_type="pre" and post-deployment jobs must be labelled
with job_type="post".
Copy link
Contributor

@sampaccoud sampaccoud left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Awesome

@jmaupetit
Copy link
Contributor Author

Once openfun/openedx-docker#84 has been merged, we need to make a new release of openedx-docker (e.g. hawthorn.1-2.3.0) and add this tag as the default edxapp_image_tag, so that we ensure recent developments are not broken (due to missing dependencies).

@jmaupetit jmaupetit force-pushed the add-pre-post-jobs branch 2 times, most recently from 131d7dc to eeb0e60 Compare December 20, 2018 17:45
Recent changes in vault variable namespacing had not been propagated in
internationalization-related objects where the TX_TOKEN variable is
expected to be defined.
@jmaupetit jmaupetit removed the WIP label Dec 21, 2018
@jmaupetit jmaupetit merged commit c558c93 into master Dec 21, 2018
@jmaupetit jmaupetit deleted the add-pre-post-jobs branch December 21, 2018 11:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants