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

Separate persistent disks #960

Merged
6 commits merged into from Oct 31, 2017
Merged

Separate persistent disks #960

6 commits merged into from Oct 31, 2017

Conversation

ghost
Copy link

@ghost ghost commented Oct 19, 2017

Fixes #743, consistent with storage design proposals in #954.

Also includes a hot fix for failing fast in the cache script if a curl request returns an error.
Adds the -it flag to building the docker container locally.

@ghost ghost changed the title [WIP] Separate persisten disks [WIP] Separate persistent disks Oct 19, 2017
@ghost ghost requested review from andrewtchin and anchal-agrawal October 19, 2017 15:42

task "VMDK Sizes"
du -h bin/*
du -h bin/*
Copy link
Contributor

Choose a reason for hiding this comment

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

is this used to set the size in the ovf at the end? i think that's not happening correctly on master

Copy link
Author

Choose a reason for hiding this comment

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

No, it just prints out the value. ovf vmdk file reference sizes are definitely not correct in master, and haven't been for a long time.

@@ -38,7 +38,11 @@ for _, s in enumerate(dataMap["services"]):
dataMap["services"][s]["restart"] = "on-failure"
if "volumes" in dataMap["services"][s]:
for kvol, vol in enumerate(dataMap["services"][s]["volumes"]):
if vol.startswith( '/data' ):
if vol.startswith( '/data/database' ):
Copy link
Contributor

Choose a reason for hiding this comment

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

can you put a comment to note that this is fixing up the compose file?

Copy link
Author

Choose a reason for hiding this comment

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

sure.

@@ -1,8 +1,8 @@
[Unit]
Description=Demo VCH Installer Wizard Service
Documentation=https://github.com/vmware/vic
After=engine_installer_startup.service data.mount
Requires=data.mount
After=engine_installer_startup.service .targetvic-mounts.target
Copy link
Contributor

Choose a reason for hiding this comment

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

is this a typo?

Copy link
Author

Choose a reason for hiding this comment

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

yes, good catch!

@andrewtchin
Copy link
Contributor

andrewtchin commented Oct 20, 2017

reminder to ping stuart to update docs and system requirements for this
and document that logs will be lost during upgrade, user should back them up if they need them

@ghost ghost changed the title [WIP] Separate persistent disks Separate persistent disks Oct 24, 2017
@andrewtchin
Copy link
Contributor

Remove this since it's not needed for upgrade anymore https://github.com/vmware/vic-product/blob/master/installer/build/cache.sh#L31

@andrewtchin
Copy link
Contributor

Also if logs will be lost during the upgrade to new disks, print a warning in the upgrade script with a confirm y/n input

@@ -0,0 +1,44 @@
#!/usr/bin/env bash
Copy link
Contributor

Choose a reason for hiding this comment

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

nice

@ghost ghost merged commit 8146345 into vmware:master Oct 31, 2017
@ghost ghost deleted the feature/separate-data-disks branch October 31, 2017 14:15
This pull request was closed.
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.

Separate persistent data to individual disks on the vSphere Integrated Containers Appliance
2 participants