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

ci: add branch preparation CI changes #567

Merged

Conversation

Abhinandan-Purkait
Copy link
Member

Detailed workflow

  • We create the release/x.y branch on the repo from develop.
  • This concerned GitHub workflow kicks in.
  • The workflow extracts the x.y from the branch name i.e. release/x.y.
  • The workflow updates the chart version to x.y.0-prerelease and creates a PR against the release/x.y branch.
  • We are supposed to merge the PRs manually. Once the above PR is merged the PR merge workflow for the release/x.y kicks in.

This also adds the script that is capable of updating release and develop branches after tag is cut.

@codecov-commenter
Copy link

codecov-commenter commented Jul 19, 2024

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.67%. Comparing base (08ed197) to head (6666203).
Report is 1 commits behind head on develop.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #567   +/-   ##
========================================
  Coverage    96.67%   96.67%           
========================================
  Files            1        1           
  Lines          481      481           
========================================
  Hits           465      465           
  Misses          13       13           
  Partials         3        3           
Flag Coverage Δ
bddtests 96.67% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

.github/workflows/branch_preparation.yml Outdated Show resolved Hide resolved
.github/workflows/branch_preparation.yml Outdated Show resolved Hide resolved
.github/workflows/branch_preparation.yml Outdated Show resolved Hide resolved
scripts/update-chart-version.sh Outdated Show resolved Hide resolved
scripts/update-chart-version.sh Outdated Show resolved Hide resolved
@Abhinandan-Purkait Abhinandan-Purkait force-pushed the preparation_workflow branch 2 times, most recently from ef51b8d to 3641345 Compare July 22, 2024 05:33
.github/workflows/branch_preparation.yml Outdated Show resolved Hide resolved
scripts/test-update-chart-version.sh Outdated Show resolved Hide resolved
scripts/test-update-chart-version.sh Outdated Show resolved Hide resolved
@Abhinandan-Purkait Abhinandan-Purkait force-pushed the preparation_workflow branch 13 times, most recently from b201555 to 9ab8bd6 Compare July 25, 2024 09:47
@Abhinandan-Purkait Abhinandan-Purkait merged commit fcfad64 into openebs:develop Jul 25, 2024
7 checks passed
@Abhinandan-Purkait Abhinandan-Purkait deleted the preparation_workflow branch July 26, 2024 12:25
jnels124 pushed a commit to jnels124/zfs-localpv that referenced this pull request Jul 30, 2024
Abhinandan-Purkait added a commit that referenced this pull request Sep 10, 2024
Abhinandan-Purkait added a commit that referenced this pull request Sep 10, 2024
* ci: enhance pull_request workflow (#557)

* ci: enhance pull_request workflow

Signed-off-by: Abhinandan Purkait <[email protected]>

* chore: use ct via nix-shell,

Signed-off-by: Abhinandan Purkait <[email protected]>

* chore: add the csi-driver step

Signed-off-by: Abhinandan Purkait <[email protected]>

* fix: the release regex

Signed-off-by: Abhinandan Purkait <[email protected]>

* chore: remove test.test binary

Signed-off-by: Abhinandan Purkait <[email protected]>

* chore: add gitignore entry for .test files

Signed-off-by: Abhinandan Purkait <[email protected]>

* fix: remove redundant die method

Signed-off-by: Abhinandan Purkait <[email protected]>

* fix: set analytics to be disabled on ci

Signed-off-by: Abhinandan Purkait <[email protected]>

* fix: shebang, condition check, script input

Signed-off-by: Abhinandan Purkait <[email protected]>

* chore: move zfs env setup inside nix

Signed-off-by: Abhinandan Purkait <[email protected]>

* chore: get IMAGE_TAG from chart

Signed-off-by: Abhinandan Purkait <[email protected]>

* ci: disable the chart releaser workflow temporarily

Signed-off-by: Abhinandan Purkait <[email protected]>

* chore: don't use yq for extracting version in Makefile

Signed-off-by: Abhinandan Purkait <[email protected]>

---------

Signed-off-by: Abhinandan Purkait <[email protected]>

* chore: replace CRD with auto-generated copy (#564)

Signed-off-by: Niladri Halder <[email protected]>

* ci: enhance build.yml workflow

Signed-off-by: Abhinandan Purkait <[email protected]>

* ci: add branch preparation and release CI changes (#567)

Signed-off-by: Abhinandan Purkait <[email protected]>

* chore: cleanup unused scripts, update make manifests (#569)

* chore: cleanup unused scripts, update make manifets

Signed-off-by: Abhinandan Purkait <[email protected]>

* chore: generate operator from helm template

Signed-off-by: Abhinandan Purkait <[email protected]>

* ci: add manifests and codegen validation in pull request

Signed-off-by: Abhinandan Purkait <[email protected]>

* chore: add comments on the generate-manifests script

Signed-off-by: Abhinandan Purkait <[email protected]>

* chore: generate zfs-operator by disabling helm labels

Signed-off-by: Abhinandan Purkait <[email protected]>

---------

Signed-off-by: Abhinandan Purkait <[email protected]>

* chore: set version to prerelease

Signed-off-by: Abhinandan Purkait <[email protected]>

---------

Signed-off-by: Abhinandan Purkait <[email protected]>
Signed-off-by: Niladri Halder <[email protected]>
Co-authored-by: Niladri Halder <[email protected]>
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 this pull request may close these issues.

3 participants