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

Examples: fix show_summary and add check_OCP_support #211

Merged
merged 2 commits into from
Jun 13, 2020

Conversation

dougsland
Copy link
Contributor

@dougsland dougsland commented Jun 12, 2020

  • fix show_summary
  • parse engine version and check OCP support

Signed-off-by: Douglas Schilling Landgraf [email protected]

Benefits

  • Fix show_summary example
  • Add new example: check OCP support

Possible Drawbacks

None

Verification Process

Locally tested

Applicable Issues

Show users how the bits works.

Currently, the oVirt Engine version is not showed correctly. This
patch adds .MustFullVersion().

Signed-off-by: Douglas Schilling Landgraf <[email protected]>
@dougsland
Copy link
Contributor Author

@dougsland
Copy link
Contributor Author

Before the patch:

$ go run ./show_summary.go 
Version: &{{<nil>} 0xc000297178 <nil> <nil> 0xc0002ab090 <nil> 0xc0002971f8 0xc000297228 <nil> 0xc000297258}
Hosts: 2
StorageDomain: 3
Users: 1
Vms: 1

After the patch:

$ go run ./show_summary.go 
Version: 4.3.10.4-1.el7
Hosts: 2
StorageDomain: 3
Users: 1
Vms: 1

Add example how to parse engine versions and check
if OCP deploy is supported.

Signed-off-by: Douglas Schilling Landgraf <[email protected]>
@dougsland dougsland changed the title examples: fix show_summary to show version correctly Examples: fix show_summary and add check_OCP_support Jun 12, 2020
Copy link

@apahim apahim left a comment

Choose a reason for hiding this comment

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

lgtm

@imjoey
Copy link
Member

imjoey commented Jun 13, 2020

Looks good to me. Let's merge it. Thank you very much @dougsland .

@imjoey imjoey merged commit 33b95ab into oVirt:master Jun 13, 2020
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.

3 participants