-
Notifications
You must be signed in to change notification settings - Fork 592
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
chore(orc8r): upload vagrant box with vagrant cli #13555
chore(orc8r): upload vagrant box with vagrant cli #13555
Conversation
Thanks for opening a PR! 💯
Howto
More infoPlease take a moment to read through the Magma project's
If this is your first Magma PR, also consider reading
|
✔️ The Semantic PR check ended with status success. See instructions on formatting your commit and pull request titles. |
7c48e44
to
5483346
Compare
"$VERSION" \ | ||
"$BOX_PROVIDER" \ | ||
"$BOX_FILE" \ | ||
--release |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this line will automatically release the file directly. Maybe we want to drop that, but that was also the old logic which was just broken.
|
||
BOX_FILE=$1 | ||
|
||
USER=magmacore | ||
BOX=$(basename $BOX_FILE | cut -d_ -f1-2 | cut -d. -f1) | ||
VERSION="1.1.$(date +"%Y%m%d")" | ||
VERSION="1.2.$(date +"%Y%m%d")" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I thought we already bumped the version
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
And this was changed on master too.
Signed-off-by: Maximilian Huber <maximilian.huber@tngtech.com>
My changes, including tests: |
This might need a rebase, I think the broken dp integ test is fixed on master. |
I will rebase |
a392a73
to
b7fa204
Compare
- Moved `--release` option to the beginning of `vagrant cloud publish`, suggested by `vagrant cloud publish --help` - Added a missing backslash - Tested script on personal vagrant account Signed-off-by: Moritz Huebner <moritz.huebner@tngtech.com>
b7fa204
to
01d0897
Compare
* chore(orc8r): upload vagrant box with vagrant cli Signed-off-by: Maximilian Huber <maximilian.huber@tngtech.com> * chore(orc8r): Tidied vagrant cli usage - Moved `--release` option to the beginning of `vagrant cloud publish`, suggested by `vagrant cloud publish --help` - Added a missing backslash - Tested script on personal vagrant account Signed-off-by: Maximilian Huber <maximilian.huber@tngtech.com> Signed-off-by: Moritz Huebner <moritz.huebner@tngtech.com> Co-authored-by: Moritz Huebner <moritz.huebner@tngtech.com>
Needs someone to take care of it and to finish this PR.