Skip to content

Upgrade: 1.1.2 to 1.2.0#381

Merged
guangbochen merged 3 commits intoharvester:mainfrom
bk201:wip-3839
Aug 22, 2023
Merged

Upgrade: 1.1.2 to 1.2.0#381
guangbochen merged 3 commits intoharvester:mainfrom
bk201:wip-3839

Conversation

@bk201
Copy link
Contributor

@bk201 bk201 commented Aug 9, 2023

  • Add helper script information.
  • Add known issues.

New known issues

Preview: https://64e4856d660bdd05f81e24ad--harvester-preview.netlify.app/dev/upgrade/v1-1-2-to-v1-2-0

Note: the zh translation is kept empty now to skip the building error (missing links).

Copy link
Member

@starbops starbops left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

Copy link
Contributor

@vickyhella vickyhella left a comment

Choose a reason for hiding this comment

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

Only some nits

@vickyhella vickyhella added this to the v1.2.0 milestone Aug 15, 2023

:::danger

Please do not upgrade a running cluster to v1.2.0 if your machine has an **Intel E810** NIC card. We saw some reports that the NIC card has a problem when added to a bonding device. Please check this issue for more info: https://github.com/harvester/harvester/issues/3860.
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this still a problem after we bumped the OS kernel, or it is still pending on QA?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed the danger admonitions, e810 issue is fixed:
harvester/harvester#3860 (comment)

bk201 and others added 3 commits August 22, 2023 17:47
- Add helper script information.
- Add known issues.

Signed-off-by: Kiefer Chang <kiefer.chang@suse.com>
Co-authored-by: vickyhella <vickyhella@hotmail.com>
Signed-off-by: Kiefer Chang <kiefer.chang@suse.com>
@harvester harvester deleted a comment from netlify bot Aug 22, 2023
@harvesterhci-io-github-bot
Copy link
Collaborator

Deploy Preview for harvester-preview ready!

Name Link
🔨 Latest commit f93aa96
🔍 Latest deploy log https://app.netlify.com/sites/harvester-preview/deploys/64e4856d660bdd05f81e24ad
😎 Deploy Preview https://64e4856d660bdd05f81e24ad--harvester-preview.netlify.app

@bk201 bk201 requested a review from guangbochen August 22, 2023 09:55
Copy link
Contributor

@guangbochen guangbochen left a comment

Choose a reason for hiding this comment

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

LGTM, thanks.

@guangbochen guangbochen merged commit e1b3ce2 into harvester:main Aug 22, 2023
vickyhella added a commit to vickyhella/harvester-docs that referenced this pull request Sep 4, 2023
* Upgrade: 1.1.2 to 1.2.0

- Add helper script information.
- Add known issues.

Signed-off-by: Kiefer Chang <kiefer.chang@suse.com>

* Apply suggestions from code review

Co-authored-by: vickyhella <vickyhella@hotmail.com>

* Remove E810 warning

Signed-off-by: Kiefer Chang <kiefer.chang@suse.com>

---------

Signed-off-by: Kiefer Chang <kiefer.chang@suse.com>
Co-authored-by: vickyhella <vickyhella@hotmail.com>
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