feat(ec): ability to upgrade the cluster #4678

emosbaugh · 2024-06-12T13:31:13Z

What this PR does / why we need it:

Related replicatedhq/embedded-cluster-operator#218

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Steps to reproduce

Does this PR introduce a user-facing change?

Does this PR require documentation?

pkg/embeddedcluster/upgrade.go

sgalsaleh · 2024-06-21T19:17:26Z

pkg/embeddedcluster/upgrade.go

+		CheckRedirect: func(req *http.Request, via []*http.Request) error {
+			if strings.Contains(req.URL.Path, "/embedded-cluster-public-files/") {
+				// Workaround as cloudflare complains about unsupported auth method
+				req.Header.Del("Authorization")


it's probably good practice to always remove the auth header on redirect. idk why Go doesn't do that automatically. can probably just always delete the auth header without checking the path.

im fixing this in the proxy https://github.com/replicatedhq/cloudflare-workers/pull/348

sgalsaleh · 2024-06-21T19:23:36Z

pkg/embeddedcluster/upgrade.go

+}
+
+func embeddedRegistryImageAddress(registryDomain string, srcImage string) (string, error) {
+	imagePath, err := image.PathToRegistryECImage(srcImage, registryDomain)


the second parameter is the registry namespace, not the domain. these registry settings should be passed by the deploy function from kots.

* airgap preflight reporting for upgrade service * create SubmitAppInfo func for upgrade service * use UpgradeServiceParams where possible

* ability to deploy an available update

* Airgap upgrade service --------- Co-authored-by: Craig O'Donnell <[email protected]>

…ndpoint (#4718)

* temporarily disable EKS 1.30 testing * Pin ARC charts versions (cherry picked from commit 20853b8) --------- Co-authored-by: Salah Al Saleh <[email protected]>

* Bump github.com/replicatedhq/troubleshoot from 0.92.2 to 0.93.2 Bumps [github.com/replicatedhq/troubleshoot](https://github.com/replicatedhq/troubleshoot) from 0.92.2 to 0.93.2. - [Release notes](https://github.com/replicatedhq/troubleshoot/releases) - [Commits](replicatedhq/troubleshoot@v0.92.2...v0.93.2) --- updated-dependencies: - dependency-name: github.com/replicatedhq/troubleshoot dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> * go mod tidy * fix --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Salah Al Saleh <[email protected]>

Bumps the security group with 2 updates: [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) and [github.com/schemahero/schemahero](https://github.com/schemahero/schemahero). Updates `github.com/aws/aws-sdk-go` from 1.54.2 to 1.54.6 - [Release notes](https://github.com/aws/aws-sdk-go/releases) - [Commits](aws/aws-sdk-go@v1.54.2...v1.54.6) Updates `github.com/schemahero/schemahero` from 0.17.8 to 0.17.9 - [Release notes](https://github.com/schemahero/schemahero/releases) - [Commits](schemahero/schemahero@v0.17.8...v0.17.9) --- updated-dependencies: - dependency-name: github.com/aws/aws-sdk-go dependency-type: direct:production update-type: version-update:semver-patch dependency-group: security - dependency-name: github.com/schemahero/schemahero dependency-type: direct:production update-type: version-update:semver-patch dependency-group: security ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Salah Al Saleh <[email protected]>

Bumps the security group in /web with 6 updates: | Package | From | To | | --- | --- | --- | | [monaco-editor](https://github.com/microsoft/monaco-editor) | `0.49.0` | `0.50.0` | | [@types/node](https://github.com/DefinitelyTyped/DefinitelyTyped/tree/HEAD/types/node) | `20.14.2` | `20.14.7` | | [eslint-plugin-jsx-a11y](https://github.com/jsx-eslint/eslint-plugin-jsx-a11y) | `6.8.0` | `6.9.0` | | [eslint-plugin-react](https://github.com/jsx-eslint/eslint-plugin-react) | `7.34.2` | `7.34.3` | | [sass](https://github.com/sass/dart-sass) | `1.77.5` | `1.77.6` | | [webpack](https://github.com/webpack/webpack) | `5.92.0` | `5.92.1` | Updates `monaco-editor` from 0.49.0 to 0.50.0 - [Release notes](https://github.com/microsoft/monaco-editor/releases) - [Changelog](https://github.com/microsoft/monaco-editor/blob/main/CHANGELOG.md) - [Commits](microsoft/monaco-editor@v0.49.0...v0.50.0) Updates `@types/node` from 20.14.2 to 20.14.7 - [Release notes](https://github.com/DefinitelyTyped/DefinitelyTyped/releases) - [Commits](https://github.com/DefinitelyTyped/DefinitelyTyped/commits/HEAD/types/node) Updates `eslint-plugin-jsx-a11y` from 6.8.0 to 6.9.0 - [Release notes](https://github.com/jsx-eslint/eslint-plugin-jsx-a11y/releases) - [Changelog](https://github.com/jsx-eslint/eslint-plugin-jsx-a11y/blob/main/CHANGELOG.md) - [Commits](jsx-eslint/eslint-plugin-jsx-a11y@v6.8.0...v6.9.0) Updates `eslint-plugin-react` from 7.34.2 to 7.34.3 - [Release notes](https://github.com/jsx-eslint/eslint-plugin-react/releases) - [Changelog](https://github.com/jsx-eslint/eslint-plugin-react/blob/master/CHANGELOG.md) - [Commits](jsx-eslint/eslint-plugin-react@v7.34.2...v7.34.3) Updates `sass` from 1.77.5 to 1.77.6 - [Release notes](https://github.com/sass/dart-sass/releases) - [Changelog](https://github.com/sass/dart-sass/blob/main/CHANGELOG.md) - [Commits](sass/dart-sass@1.77.5...1.77.6) Updates `webpack` from 5.92.0 to 5.92.1 - [Release notes](https://github.com/webpack/webpack/releases) - [Commits](webpack/webpack@v5.92.0...v5.92.1) --- updated-dependencies: - dependency-name: monaco-editor dependency-type: direct:production update-type: version-update:semver-minor dependency-group: security - dependency-name: "@types/node" dependency-type: direct:development update-type: version-update:semver-patch dependency-group: security - dependency-name: eslint-plugin-jsx-a11y dependency-type: direct:development update-type: version-update:semver-minor dependency-group: security - dependency-name: eslint-plugin-react dependency-type: direct:development update-type: version-update:semver-patch dependency-group: security - dependency-name: sass dependency-type: direct:development update-type: version-update:semver-patch dependency-group: security - dependency-name: webpack dependency-type: direct:development update-type: version-update:semver-patch dependency-group: security ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Salah Al Saleh <[email protected]>

Co-authored-by: dydx <[email protected]>

emosbaugh added the type::feature New feature or request label Jun 12, 2024

emosbaugh mentioned this pull request Jun 17, 2024

feat: ability to upgrade the cluster replicatedhq/embedded-cluster-operator#218

Merged

sgalsaleh reviewed Jun 17, 2024

View reviewed changes

pkg/embeddedcluster/upgrade.go Show resolved Hide resolved

sgalsaleh reviewed Jun 18, 2024

View reviewed changes

pkg/embeddedcluster/upgrade.go Outdated Show resolved Hide resolved

sgalsaleh reviewed Jun 18, 2024

View reviewed changes

pkg/embeddedcluster/upgrade.go Outdated Show resolved Hide resolved

emosbaugh force-pushed the emosbaugh/sc-105498/eco-ability-to-upgrade-the-cluster branch from 46b72f6 to 474fb43 Compare June 20, 2024 21:28

sgalsaleh reviewed Jun 21, 2024

View reviewed changes

sgalsaleh and others added 9 commits June 23, 2024 16:05

KOTS upgrader

064d1b5

upgrade service preflights api (#4693)

a15da88

upgrade service airgap preflight reporting (#4698)

31c9116

* airgap preflight reporting for upgrade service * create SubmitAppInfo func for upgrade service * use UpgradeServiceParams where possible

Deploy available version (#4704)

eae0d65

* ability to deploy an available update

Embedded Cluster Upgrade UI: Show upgrade wizard (#4705)

40f562c

Airgap upgrade service (#4713)

b3fbb79

* Airgap upgrade service --------- Co-authored-by: Craig O'Donnell <[email protected]>

remove airgap update on deploy

cf58be9

wait for current ec install

922e1e7

EC upgrade improvements: Close Modal after deploying, fix preflight e…

d304e4c

…ndpoint (#4718)

emosbaugh force-pushed the emosbaugh/sc-105498/eco-ability-to-upgrade-the-cluster branch from b80bf40 to 5aef836 Compare June 26, 2024 17:19

Craig O'Donnell and others added 9 commits June 26, 2024 10:19

temporarily disable EKS 1.30 testing (#4714)

6ce001f

* temporarily disable EKS 1.30 testing * Pin ARC charts versions (cherry picked from commit 20853b8) --------- Co-authored-by: Salah Al Saleh <[email protected]>

Update arm64-arc-runner-set.yaml to 100G disk (#4717)

e599e6b

Bump github.com/hashicorp/go-getter in the go_modules group (#4715)

8648d5d

Update KOTS image dependency tags (#4710)

9928079

Co-authored-by: dydx <[email protected]>

feat(ec): ability to upgrade the cluster

913c6b8

compile

04776d0

emosbaugh force-pushed the emosbaugh/sc-105498/eco-ability-to-upgrade-the-cluster branch from 5aef836 to 04776d0 Compare June 26, 2024 17:30

emosbaugh closed this Jun 26, 2024

emosbaugh mentioned this pull request Jun 26, 2024

feat(ec): ability to upgrade the cluster #4720

Merged

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat(ec): ability to upgrade the cluster #4678

feat(ec): ability to upgrade the cluster #4678

emosbaugh commented Jun 12, 2024

sgalsaleh Jun 21, 2024

emosbaugh Jun 21, 2024 •

edited

Loading

sgalsaleh Jun 21, 2024

feat(ec): ability to upgrade the cluster #4678

feat(ec): ability to upgrade the cluster #4678

Conversation

emosbaugh commented Jun 12, 2024

What this PR does / why we need it:

Which issue(s) this PR fixes:

Special notes for your reviewer:

Steps to reproduce

Does this PR introduce a user-facing change?

Does this PR require documentation?

sgalsaleh Jun 21, 2024

Choose a reason for hiding this comment

emosbaugh Jun 21, 2024 • edited Loading

Choose a reason for hiding this comment

sgalsaleh Jun 21, 2024

Choose a reason for hiding this comment

emosbaugh Jun 21, 2024 •

edited

Loading