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

master promotion #164

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Conversation

mrjackv
Copy link
Contributor

@mrjackv mrjackv commented Nov 4, 2024

@mrjackv mrjackv force-pushed the feature/master-promotion branch 2 times, most recently from 50fb3fe to 2544129 Compare November 19, 2024 15:00
.orchestra/ci/ci-run.sh Outdated Show resolved Hide resolved
.orchestra/ci/ci-regression.sh Outdated Show resolved Hide resolved
.orchestra/ci/parts/pipeline-create.sh Outdated Show resolved Hide resolved
@mrjackv mrjackv force-pushed the feature/master-promotion branch 8 times, most recently from 1b444cc to ea2f20a Compare November 26, 2024 15:29
Inline parts of the s3cmd config which are not auth-specific or
bucket-specific.
@mrjackv mrjackv force-pushed the feature/master-promotion branch from ea2f20a to 16ffd58 Compare November 26, 2024 15:46
@mrjackv mrjackv force-pushed the feature/master-promotion branch 4 times, most recently from 05e5c53 to afabd7d Compare November 29, 2024 15:31
Add additional information when failing in the mass-testing-regression
component.
Sync the `.gitlab-ci.yml` back from the gitlab CI repo.
Introduce the `ci-regression.sh` script, which will be used for
regression testing and promoting the `develop` branch to `master`.
@mrjackv mrjackv force-pushed the feature/master-promotion branch from afabd7d to 23ee577 Compare December 2, 2024 09:21
@aleclearmind
Copy link
Contributor

I think we need a brief README.md summarizing what happens when you push something. I'm losing track of all the things we do.

# 1:
# File containing the list of changed files in the binary archives, each line
# is a different file path
# COMPONENT_TARGET_BRANCH: target branch the CI has been triggered against
Copy link
Contributor

Choose a reason for hiding this comment

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

Blank line

@@ -7,6 +7,11 @@ set -euo pipefail
#
# Mandatory parameters:
#
# 1:
Copy link
Contributor

Choose a reason for hiding this comment

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

./binary-archives-hook.sh CHANGED_FILES

CHANGED_FILES
   File containing the list of changed files in the binary archives, each line
   is a different file path

Environment variables:

  COMPONENT_TARGET_BRANCH: target branch the CI has been triggered against

[...]

Same elsewhere, the environment variables are list under "parameters", which is confusing.

Also, please document in the header what this script does. It only explains when it runs.

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.

2 participants