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

feat: ability to upgrade the cluster via the new binary #684

Merged
merged 96 commits into from
Jul 16, 2024

Conversation

emosbaugh
Copy link
Member

No description provided.

Copy link

github-actions bot commented Jun 10, 2024

This PR has been released (on staging) and is available for download with a embedded-cluster-smoke-test-staging-app license ID.

Online Installer:

curl "https://staging.replicated.app/embedded/embedded-cluster-smoke-test-staging-app/ci/appver-dev-5a8edc1" -H "Authorization: $EC_SMOKE_TEST_LICENSE_ID" -o embedded-cluster-smoke-test-staging-app-ci.tgz

Airgap Installer (may take a few minutes before the airgap bundle is built):

curl "https://staging.replicated.app/embedded/embedded-cluster-smoke-test-staging-app/ci-airgap/appver-dev-5a8edc1?airgap=true" -H "Authorization: $EC_SMOKE_TEST_LICENSE_ID" -o embedded-cluster-smoke-test-staging-app-ci.tgz

Happy debugging!

@emosbaugh emosbaugh force-pushed the emosbaugh/sc-105498/eco-ability-to-upgrade-the-cluster branch 2 times, most recently from c777c7a to 6e755df Compare June 14, 2024 20:19
@emosbaugh emosbaugh changed the title feat: ability to upgrade the cluster (DRAFT DO NOT MERGE) feat: ability to upgrade the cluster Jun 14, 2024
@emosbaugh emosbaugh force-pushed the emosbaugh/sc-105498/eco-ability-to-upgrade-the-cluster branch 3 times, most recently from 5f911e5 to cb074e7 Compare June 27, 2024 20:35
@emosbaugh emosbaugh force-pushed the emosbaugh/sc-105498/eco-ability-to-upgrade-the-cluster branch 3 times, most recently from 1737f3d to a47ffba Compare July 8, 2024 22:35
@sgalsaleh sgalsaleh force-pushed the emosbaugh/sc-105498/eco-ability-to-upgrade-the-cluster branch from a47ffba to 7eb4543 Compare July 8, 2024 22:57
@sgalsaleh sgalsaleh marked this pull request as ready for review July 15, 2024 15:36
@sgalsaleh sgalsaleh changed the title feat: ability to upgrade the cluster feat: ability to upgrade the cluster via the new binary Jul 16, 2024
@@ -30,15 +30,15 @@ jobs:

- name: Build and push local-artifact-mirror image
run: |
export SHORT_SHA=dev-$(git rev-parse --short=7 HEAD)
export SHORT_SHA=dev-${GITHUB_SHA::7}
Copy link
Member

Choose a reason for hiding this comment

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

this should match what's in the pull-request.yaml version of things

Copy link
Member

Choose a reason for hiding this comment

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

can't use the same thing here though as github.event.pull_request.head.sha doesn't apply here, no?

Copy link
Member

Choose a reason for hiding this comment

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

i guess we could use GITHUB_SHA in pull-request.yaml?

Copy link
Member

@laverya laverya left a comment

Choose a reason for hiding this comment

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

Looks good besides the SHORT_SHA mismatch between pull-request.yaml and release-dev.yaml (we should really combine those soon)

@sgalsaleh sgalsaleh merged commit b6f2e17 into main Jul 16, 2024
41 checks passed
@sgalsaleh sgalsaleh deleted the emosbaugh/sc-105498/eco-ability-to-upgrade-the-cluster branch July 16, 2024 22:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants