Skip to content

Conversation

ivan4th
Copy link
Contributor

@ivan4th ivan4th commented Oct 1, 2018

[ci skip]


This change is Reviewable

Copy link
Contributor

@jellonek jellonek left a comment

Choose a reason for hiding this comment

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

Reviewed 6 of 6 files at r1.
Reviewable status: 0 of 2 approvals obtained (waiting on @ivan4th)


examples/cirros-vm-persistent-rootfs-rbd.yaml, line 65 at r1 (raw file):

  containers:
  - name: cirros-vm
    #image: virtlet.cloud/cirros

That can be removed.


examples/README.md, line 86 at r1 (raw file):

for testing, you can use this command to start the cluster with
`BlockVolume` and Ceph support:

Maybe ```console ?
Same for rest of below code blocks. console or bash (they are a bit different)?


examples/README.md, line 221 at r1 (raw file):

demonstrates the use of persistent root filesystem. The most important part
is the `volumeDevices` section in the pod's container definition:

Copy link
Contributor

@jellonek jellonek left a comment

Choose a reason for hiding this comment

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

Reviewable status: 1 change requests, 0 of 2 approvals obtained (waiting on @ivan4th)


examples/README.md, line 221 at r1 (raw file):

Previously, jellonek (Piotr Skamruk) wrote…
</blockquote></details>

idk why reviewable messed that up, but there was info about ```yaml (yaml type for code block, if it again will try to chop it down)


<!-- Sent from Reviewable.io -->

@ivan4th ivan4th force-pushed the ivan4th/persistent-rootfs-docs branch from 82a3b77 to e60dfb3 Compare October 3, 2018 09:34
Copy link
Contributor Author

@ivan4th ivan4th left a comment

Choose a reason for hiding this comment

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

Reviewable status: 1 change requests, 0 of 2 approvals obtained (waiting on @jellonek and @ivan4th)


examples/cirros-vm-persistent-rootfs-rbd.yaml, line 65 at r1 (raw file):

Previously, jellonek (Piotr Skamruk) wrote…

That can be removed.

Done.


examples/README.md, line 86 at r1 (raw file):

Previously, jellonek (Piotr Skamruk) wrote…

Maybe ```console ?
Same for rest of below code blocks. console or bash (they are a bit different)?

Fixed. console is for console interaction snippets (with the prompt), bash is for shell snippets (without the prompt). Most snippets are bash for easier copy-paste.

@ivan4th ivan4th force-pushed the ivan4th/persistent-rootfs-docs branch from e60dfb3 to c54a172 Compare October 3, 2018 09:55
Copy link
Contributor

@jellonek jellonek left a comment

Choose a reason for hiding this comment

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

:lgtm:

Reviewed 2 of 2 files at r2, 1 of 1 files at r3.
Reviewable status: 1 change requests, 0 of 2 approvals obtained

Copy link
Contributor

@pigmej pigmej left a comment

Choose a reason for hiding this comment

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

Reviewed 4 of 6 files at r1, 1 of 2 files at r2, 1 of 1 files at r3.
Reviewable status: 1 of 2 approvals obtained

@pigmej pigmej merged commit 15bd4c0 into master Oct 3, 2018
@pigmej pigmej deleted the ivan4th/persistent-rootfs-docs branch October 3, 2018 11:04
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.

3 participants