FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Updated vmware virtual hardware version in systemvmtemplate build script by Spaceman1984 · Pull Request #3921 · apache/cloudstack · GitHub

Repository navigation

Updated vmware virtual hardware version in systemvmtemplate build script - #3921

Merged
yadvr merged 2 commits into
apache:masterfrom
shapeblue:vmware-virtual-hardware-version
May 30, 2020
Merged

yadvr merged 2 commits into
apache:masterfrom
shapeblue:vmware-virtual-hardware-version

Conversation

Copy link
Copy Markdown
Contributor

Description

This Pull request updates the system VM template build script to use vmware hardware version 11 instead of 8.

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (improves an existing feature and functionality)
  • Cleanup (Code refactoring and cleanup, that may add test cases)

Screenshots (if appropriate):

How Has This Been Tested?

This was tested by running the build script before and after the change. No errors.

yadvr left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Remove the shar archive script, lgtm

yadvr commented Mar 2, 2020

Copy link
Copy Markdown
Member

ping @Spaceman1984 ^^

yadvr added this to the 4.15.0.0 milestone Mar 2, 2020

yadvr commented Mar 3, 2020

Copy link
Copy Markdown
Member

This needs to be manually tested by building a new systemvmtemplate and then run test matrix manually using backend jenkins. This can wait until 4.15.

DaanHoogland mentioned this pull request May 28, 2020
2 of 5 tasks

yadvr commented May 28, 2020

Copy link
Copy Markdown
Member

@DaanHoogland @harikrishna-patnala how about we merge this on master and continue work in the other PR? The change LGTM.

DaanHoogland left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

lgtm

yadvr merged commit 27e7637 into apache:master May 30, 2020
yadvr deleted the vmware-virtual-hardware-version branch May 30, 2020 11:42
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL