Skip to content

chore(setup-cluster.sh): remove prepare command#7055

Merged
mnencia merged 5 commits intomainfrom
dev/7050
Mar 5, 2025
Merged

chore(setup-cluster.sh): remove prepare command#7055
mnencia merged 5 commits intomainfrom
dev/7050

Conversation

@armru
Copy link
Member

@armru armru commented Mar 3, 2025

Closes #7050

@armru armru requested review from a team, NiccoloFei, jsilvela and litaocdl as code owners March 3, 2025 16:47
@github-actions
Copy link
Contributor

github-actions bot commented Mar 3, 2025

Build Error! No Linked Issue found. Please link an issue or mention it in the body using #<issue_id>

@cnpg-bot cnpg-bot added backport-requested ◀️ This pull request should be backported to all supported releases release-1.22 release-1.24 release-1.25 labels Mar 3, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Mar 3, 2025

❗ By default, the pull request is configured to backport to all release branches.

  • To stop backporting this pr, remove the label: backport-requested ◀️ or add the label 'do not backport'
  • To stop backporting this pr to a certain release branch, remove the specific branch label: release-x.y

@armru armru changed the title chore(setup-cluster.sh) remove prepare command chore(setup-cluster.sh): remove prepare command Mar 3, 2025
Copy link
Member

@mnencia mnencia left a comment

Choose a reason for hiding this comment

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

This is half of the required work. The "prepare" command is still called in .github/workflows/continuous-delivery.yml and needs to be replaced with the appropriate github action to install kind and kubectl.

@NiccoloFei
Copy link
Collaborator

NiccoloFei commented Mar 4, 2025

armru and others added 5 commits March 5, 2025 15:17
Closes #7050

Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
Signed-off-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
Signed-off-by: Niccolò Fei <niccolo.fei@enterprisedb.com>
Signed-off-by: Niccolò Fei <niccolo.fei@enterprisedb.com>
Signed-off-by: Niccolò Fei <niccolo.fei@enterprisedb.com>
@mnencia
Copy link
Member

mnencia commented Mar 5, 2025

/ok-to-merge

@cnpg-bot cnpg-bot added the ok to merge 👌 This PR can be merged label Mar 5, 2025
@mnencia mnencia merged commit 7e6af25 into main Mar 5, 2025
30 of 31 checks passed
@mnencia mnencia deleted the dev/7050 branch March 5, 2025 14:38
cnpg-bot pushed a commit that referenced this pull request Mar 5, 2025
The `hack/setup-cluster.sh` prepare subcommand was once essential
for E2E environment setup. Now, this is fully handled by the GitHub workflow,
which automatically installs the right versions of kind and kubectl.

Closes #7050

Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
Signed-off-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
Signed-off-by: Niccolò Fei <niccolo.fei@enterprisedb.com>
Co-authored-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
Co-authored-by: Niccolò Fei <niccolo.fei@enterprisedb.com>
(cherry picked from commit 7e6af25)
cnpg-bot pushed a commit that referenced this pull request Mar 5, 2025
The `hack/setup-cluster.sh` prepare subcommand was once essential
for E2E environment setup. Now, this is fully handled by the GitHub workflow,
which automatically installs the right versions of kind and kubectl.

Closes #7050

Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
Signed-off-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
Signed-off-by: Niccolò Fei <niccolo.fei@enterprisedb.com>
Co-authored-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
Co-authored-by: Niccolò Fei <niccolo.fei@enterprisedb.com>
(cherry picked from commit 7e6af25)
cnpg-bot pushed a commit that referenced this pull request Mar 5, 2025
The `hack/setup-cluster.sh` prepare subcommand was once essential
for E2E environment setup. Now, this is fully handled by the GitHub workflow,
which automatically installs the right versions of kind and kubectl.

Closes #7050

Signed-off-by: Armando Ruocco <armando.ruocco@enterprisedb.com>
Signed-off-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
Signed-off-by: Niccolò Fei <niccolo.fei@enterprisedb.com>
Co-authored-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
Co-authored-by: Niccolò Fei <niccolo.fei@enterprisedb.com>
(cherry picked from commit 7e6af25)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-requested ◀️ This pull request should be backported to all supported releases ok to merge 👌 This PR can be merged release-1.22 release-1.24 release-1.25

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[chore]: Remove prepare subcommand from hack/setup-cluster.sh

4 participants