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

roles: cluster_upgrade: Add progress tracking/reporting #415

Merged
merged 5 commits into from
Jan 24, 2022

Conversation

sjd78
Copy link
Member

@sjd78 sjd78 commented Jan 12, 2022

Since cluster upgrade is a very long running process, adding progress tracking allows users and admins to easily track status. Progress tracking is done by posting well formatted events.

Note: roles/cluster_upgrade/tasks/upgrade.yml was converted to a block. This allows calculating var the tasks for a host upgrade needs without using set_fact style global facts. The only annoyance is having to put them at the bottom of the file below all of the tasks.

Bug-Url: https://bugzilla.redhat.com/2040474
Signed-off-by: Scott J Dickerson sdickers@redhat.com

Since cluster upgrade is a very long running process, adding progress
tracking allows users and admins to easily track status.  Progress
tracking is done by posting well formatted events.

Bug-Url: https://bugzilla.redhat.com/2040474
Signed-off-by: Scott J Dickerson <sdickers@redhat.com>
@mwperina mwperina requested a review from mnecas January 19, 2022 15:31
roles/cluster_upgrade/tasks/upgrade.yml Outdated Show resolved Hide resolved
roles/cluster_upgrade/tasks/main.yml Show resolved Hide resolved
roles/cluster_upgrade/tasks/main.yml Show resolved Hide resolved
  - It is not used by the cluster upgrade wizard in ovirt's
    admin portal.

  - The VMs are not stopped anywhere within the role, so this task
    doesn't really do anything useful.
@sjd78 sjd78 requested a review from mnecas January 20, 2022 16:58
Copy link
Member

@mnecas mnecas left a comment

Choose a reason for hiding this comment

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

LGTM

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.

2 participants