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

[backport v1.3] [BUG] Upgrade can be started while previous upgrade still in progress #6232

Open
harvesterhci-io-github-bot opened this issue Jul 29, 2024 · 5 comments
Assignees
Labels
kind/bug Issues that are defects reported by users or that we know have reached a real release not-require/test-plan Skip to create a e2e automation test issue reproduce/needed Reminder to add a reproduce label and to remove this one severity/needed Reminder to add a severity label and to remove this one
Milestone

Comments

@harvesterhci-io-github-bot
Copy link
Collaborator

backport the issue #6144

@harvesterhci-io-github-bot harvesterhci-io-github-bot added kind/bug Issues that are defects reported by users or that we know have reached a real release not-require/test-plan Skip to create a e2e automation test issue reproduce/needed Reminder to add a reproduce label and to remove this one severity/needed Reminder to add a severity label and to remove this one labels Jul 29, 2024
@harvesterhci-io-github-bot harvesterhci-io-github-bot added this to the v1.3.2 milestone Jul 29, 2024
@harvesterhci-io-github-bot
Copy link
Collaborator Author

Pre Ready-For-Testing Checklist

  • If labeled: require/HEP Has the Harvester Enhancement Proposal PR submitted?
    The HEP PR is at:

  • Where is the reproduce steps/test steps documented?
    The reproduce steps/test steps are at:

  • Is there a workaround for the issue? If so, where is it documented?
    The workaround is at:

  • Have the backend code been merged (harvester, harvester-installer, etc) (including backport-needed/*)?
    The PR is at:

    • Does the PR include the explanation for the fix or the feature?

    • Does the PR include deployment change (YAML/Chart)? If so, where are the PRs for both YAML file and Chart?
      The PR for the YAML change is at:
      The PR for the chart change is at:

  • If labeled: area/ui Has the UI issue filed or ready to be merged?
    The UI issue/PR is at:

  • If labeled: require/doc, require/knowledge-base Has the necessary document PR submitted or merged?
    The documentation/KB PR is at:

  • If NOT labeled: not-require/test-plan Has the e2e test plan been merged? Have QAs agreed on the automation test case? If only test case skeleton w/o implementation, have you created an implementation issue?

    • The automation skeleton PR is at:
    • The automation test case PR is at:
  • If the fix introduces the code for backward compatibility Has a separate issue been filed with the label release/obsolete-compatibility?
    The compatibility issue is filed at:

@TachunLin
Copy link

TachunLin commented Aug 9, 2024

Verified not fixed on v1.3-81d64ff-head (v1.3.2-head). Found one of the case would be failed.
Move to the implement for further investigation.

Result

$\color{green}{\textsf{PASS}}$ When there is a Harvester version upgrade already started, the upgrade trigger button will automatically disappeared. All the other version upgrade can't be trigger afterward. $~~$
  1. When we have multiple upgrade version in the list, (v1.3.2-head and v1.4.0-head)
    image

  2. After we trigger the v1.3.2-head upgrade, the upgrade button will automatically disappear

https://github.com/user-attachments/assets/cc194899-bc77-4df5-b8ce-218bf019f30a
$\color{red}{\textsf{Failed}}$ But when there are two versions of upgrade, when one upgrade failed, the upgrade button will show up again $~~$

image

image

Test Information

  • Test Environment: Three nodes harvester on remote kvm machines
  • Harvester version: v1.3-81d64ff-head

Verify Steps

Two upgrade version in the lists
  1. Use ipxe-examples to provision a three nodes Harvester
  2. Prepare the v1.3-head and v1.4-head iso image on the pxe_server
  3. Create the corresponding version.yaml file on Harvester node
  4. Create the v1.3.2-head and v1.4.0-head upgrade CR on Harvester node
  5. Check the upgrade button appear on the Host page
  6. Trigger the v1.3.2-head upgrade
  7. Check the upgrade button should be disappeared.

@bk201 bk201 modified the milestones: v1.3.2, v1.3.3 Aug 14, 2024
@a110605
Copy link

a110605 commented Aug 15, 2024

vokoscreenNG-2024-08-09_17-11-32.mp4

Hi @TachunLin , according to the fixed PR logic, if this is the latest upgrade and upgrade result is failed or success.

We will show yellow upgrade button again.

@a110605
Copy link

a110605 commented Aug 15, 2024

Hi @TachunLin , as team disucssion, the second behavior is expected.

@TachunLin
Copy link

Thank you Andy for checking the failure user behavior.
I could verify it again with the successfully upgrade scenario to double confirm the upgrade button would also display with the available upgrade version CR there.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Issues that are defects reported by users or that we know have reached a real release not-require/test-plan Skip to create a e2e automation test issue reproduce/needed Reminder to add a reproduce label and to remove this one severity/needed Reminder to add a severity label and to remove this one
Projects
None yet
Development

No branches or pull requests

5 participants