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

chore: cleanup unused scripts, update make manifests #569

Merged
merged 5 commits into from
Jul 31, 2024

Conversation

Abhinandan-Purkait
Copy link
Member

@Abhinandan-Purkait Abhinandan-Purkait commented Jul 22, 2024

Changes in the PR

  • Removes the traces of TRAVIS CI
  • Updates the generate-manifests to generate the CRDs and copy them to crds suchart.
  • Updates the generate-manifests to generate the operator yaml from helm template.
  • Adds validate-codegen jobs in the workflows, to catch missed kubegen and manifests
  • Adds installerType to helm values to change analytics installer type.
NOTE:
helm is the only recommended way of installation. Operator is kept so that existing users of operator can still use this. Although no support for upgrades will be provided for operator way of installation. 
Also, now generating the operator from helm template provides a way of merging back operator installs into helm.

@codecov-commenter
Copy link

codecov-commenter commented Jul 22, 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.37%. Comparing base (c389127) to head (b908623).
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     #569   +/-   ##
========================================
  Coverage    96.37%   96.37%           
========================================
  Files            1        1           
  Lines          496      496           
========================================
  Hits           478      478           
  Misses          14       14           
  Partials         4        4           
Flag Coverage Δ
bddtests 96.37% <ø> (ø)

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.

buildscripts/build.sh Outdated Show resolved Hide resolved
buildscripts/check-diff.sh Outdated Show resolved Hide resolved
deploy/helm/charts/templates/zfs-controller.yaml Outdated Show resolved Hide resolved
deploy/zfs-operator.yaml Outdated Show resolved Hide resolved
deploy/zfs-operator.yaml Show resolved Hide resolved
@Abhinandan-Purkait Abhinandan-Purkait force-pushed the cleanup_tidy branch 3 times, most recently from 9ffc657 to 3ccd34e Compare July 31, 2024 06:01
@Abhinandan-Purkait Abhinandan-Purkait changed the title chore: cleanup unused scripts, update make manifets chore: cleanup unused scripts, update make manifests Jul 31, 2024
@Abhinandan-Purkait Abhinandan-Purkait merged commit 3854ec2 into openebs:develop Jul 31, 2024
8 checks passed
Abhinandan-Purkait added a commit that referenced this pull request Sep 10, 2024
* 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]>
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.

5 participants