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

Generated vic-machine create command has wrong argument for volume store #251

Closed
zjs opened this issue Dec 15, 2017 · 10 comments
Closed

Generated vic-machine create command has wrong argument for volume store #251

zjs opened this issue Dec 15, 2017 · 10 comments
Assignees
Labels
component/plugin/h5c The plugin for the vSphere HTML5 client kind/defect Behavior that is inconsistent with what's intended source/customer Reported by a customer, directly or via an intermediary team/lifecycle

Comments

@zjs
Copy link
Member

zjs commented Dec 15, 2017

VIC version:

1.3.0 OVA RC 4 (UI RC6)

Steps to reproduce:

Use the creation wizard to specify a VCH which has a default volume store (or, perhaps, any volume store).

Actual behavior:

The generated CLI command includes a --volume-stores option.

This will result in an error when executed: Incorrect Usage: flag provided but not defined: -volume-stores.

Expected behavior:

The generated CLI command should include a --volume-store option.

@zjs zjs added component/plugin/h5c The plugin for the vSphere HTML5 client status/needs-estimation The issue needs to be estimated by the team team/lifecycle triage/proposed-1.3.1 labels Dec 15, 2017
@mdubya66 mdubya66 added the impact/doc/note Requires creation of or changes to an official release note label Dec 15, 2017
@cristianfalcone cristianfalcone self-assigned this Dec 15, 2017
@cristianfalcone cristianfalcone removed the status/needs-estimation The issue needs to be estimated by the team label Dec 15, 2017
@jak-atx jak-atx added the kind/defect Behavior that is inconsistent with what's intended label Dec 19, 2017
jak-atx pushed a commit that referenced this issue Dec 19, 2017
Fix volume store CLI option name in Create wizard (#251)
@zjs zjs reopened this Jan 5, 2018
@AngieCris
Copy link
Contributor

AngieCris commented Jan 5, 2018

Reopen: generated CLI command still has the wrong volume store flag
(cherry-pick PR is missing)

@AngieCris
Copy link
Contributor

@stuclem Can you please document that the generated CLI command has the wrong volume store flag? The workaround is to change --volume-stores to --volume-store. Thanks!

@jooskim
Copy link
Collaborator

jooskim commented Jan 5, 2018

@stuclem To give you more context on this, commit 10863aa was not cherry-picked onto our releases/1.3.0 branch and thus got omitted in v1.3.0 GA.

@stuclem
Copy link
Contributor

stuclem commented Jan 10, 2018

Release note:

  • Generated vic-machine command includes incorrect --volume-stores option. #251
    In the Summary page of the Create VCH wizard, if you configured a volume store, the generated vic-machine command includes the option --volume-stores. This is should be --volume-store. Attempting to run the generated command results in the error Incorrect Usage: flag provided but not defined: -volume-stores.

    Workaround: Manually edit the generated command to correct the option name to --volume-store.

@jooskim is this OK?

@jooskim
Copy link
Collaborator

jooskim commented Jan 10, 2018

@stuclem Looks good! :shipit:

@zjs
Copy link
Member Author

zjs commented Jan 10, 2018

This is should be

I think there's an extraneous "is" here.

@mdubya66
Copy link
Contributor

mdubya66 commented Jan 10, 2018

Let's get this merged by Jan 17. so it makes it into 1.3.1

@mdubya66 mdubya66 reopened this Jan 10, 2018
@stuclem
Copy link
Contributor

stuclem commented Jan 11, 2018

Fixed the extraneous "is" in the RN.

@stuclem stuclem removed the impact/doc/note Requires creation of or changes to an official release note label Jan 11, 2018
@jooskim jooskim added this to the Sprint 24 Lifecycle milestone Jan 11, 2018
@andrewtchin
Copy link
Contributor

just needs cherry pick pr

jak-atx pushed a commit that referenced this issue Jan 16, 2018
Fix volume store CLI option name in Create wizard (#251)
@jak-atx
Copy link
Contributor

jak-atx commented Jan 16, 2018

Merged to master. Cherry-pick PR #303

@jak-atx jak-atx closed this as completed Jan 16, 2018
jak-atx added a commit that referenced this issue Jan 17, 2018
Fix volume store CLI option name in Create wizard (#251)
@mdubya66 mdubya66 added the source/customer Reported by a customer, directly or via an intermediary label Feb 4, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component/plugin/h5c The plugin for the vSphere HTML5 client kind/defect Behavior that is inconsistent with what's intended source/customer Reported by a customer, directly or via an intermediary team/lifecycle
Projects
None yet
Development

No branches or pull requests

8 participants