Skip to content

Provide upgrade choices for stable and testing - #6

Open
anxolerd wants to merge 3 commits into
tailhook:masterfrom
anxolerd:feature/upgrade-choices
Open

Provide upgrade choices for stable and testing#6
anxolerd wants to merge 3 commits into
tailhook:masterfrom
anxolerd:feature/upgrade-choices

Conversation

@anxolerd

@anxolerd anxolerd commented Oct 1, 2017

Copy link
Copy Markdown

It seems weird that by default stable version of vagga is installed in vagga-box but all updates are for testing version of vagga. The aim of this PR is to fix that and allow upgrades between stable versions of vagga in vagga_box.

This PR introduces the following changes:

  • upgrade_vagga shell script can upgrade to either stable or testing version of vagga depending on command-line arguments passed
  • upgrade_vagga by default upgrades vagga to the latest stable version. The motivation for this is that by definition testing version may contain unknown and unexpected bugs, which should not be present (by definition) in the stable version.
  • vagga _box upgrade_vagga passes the first argument to the underlying shell script so the flags for upgrade_vagga can be used.

However still some help and clarification are needed. I have modified upgrade-vagga.sh but do not know how to rebuild image so my changes get into virtualbox. I suppose that some changes into code will also be required as the hash of the new image will be different from the hash of the current image.

Now upgrade-vagga.sh can upgrade either to stable or to testing versions
of vagga depending on arguments.
By default upgrade to stable version of vagga as testing version by
definition may contain unexpected bugs and be unstable.
Allow upgrade_vagga command to receive arguments which define upgrade
version: either testing or stable version of vagga. Argument is directly
passed to the underlying command in virtualbox.
Describe --testing flag for upgrade_vagga command in README.rst. Also
note that `vagga _box upgrade_vagga` will upgrade vagga to the latest
stable version by default.
@tailhook

tailhook commented Oct 1, 2017

Copy link
Copy Markdown
Owner

Looks good. But this PR will have to be postponed until I rebuild the virtualbox image (this isn't an automatic thing for many reasons).

@anxolerd

anxolerd commented Oct 1, 2017

Copy link
Copy Markdown
Author

Sure. By the way, how can I rebuild the image locally for my own machine?

@tailhook

tailhook commented Oct 2, 2017

Copy link
Copy Markdown
Owner

Use packer and image/vagga.json

@anxolerd

anxolerd commented Oct 2, 2017

Copy link
Copy Markdown
Author

Thanks

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