Skip to content

Conversation

samcharles93
Copy link
Contributor

@samcharles93 samcharles93 commented Dec 31, 2024

fixed changes to incus snapshot cli commands.

Author checklist (Completed by original Author)

  • Good fit for the Rocky Linux project? Title and Author Metatags inserted ?
  • If applicable, steps and instructions have been tested to work
  • Initial self-review to fix basic typos and grammar completed

Rocky Documentation checklist (Completed by Rocky team)

  • 1st Pass (Document is good fit for project and author checklist completed)
  • 2nd Pass (Technical Review - check for technical correctness)
  • 3rd Pass (Detailed Editorial Review and Peer Review)
  • Final approval (Final Review)

fixed changes to incus snapshot cli commands.
Copy link

Test results for 0c31d61:

Number of broken URLs: 4

URL,RESULT,FILENAME
 http://example.org/foo.txt,failed,books/web_services/052-load-balancer-proxies-varnish.md
 https://azuremarketplace.microsoft.com/en-us/marketplace/apps/resf.rockylinux-aarch64,failed,guides/cloud/migration-to-new-azure-images.md
 http://0.0.0.0:61208/,failed,guides/network/glances_network_monitoring.md
 https://your.domain.name/install.php,failed,guides/cms/chyrp_lite.md

@SergeCroise
Copy link
Contributor

Copy link
Contributor

@sspencerwire sspencerwire left a comment

Choose a reason for hiding this comment

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

Thanks very much @samcharles93

@sspencerwire
Copy link
Contributor

sspencerwire commented Dec 31, 2024

@sspencerwire see also https://docs.rockylinux.org/books/lxd_server/08-snapshots/#the-snapshot-process

With LXD at least, "create" is assumed if you do not actually specify it:

lxc list
+--------------+---------+-----------------------+------+-----------+-----------+
|     NAME     |  STATE  |         IPV4          | IPV6 |   TYPE    | SNAPSHOTS |
+--------------+---------+-----------------------+------+-----------+-----------+
| mkdocs       | RUNNING | 10.187.219.224 (eth0) |      | CONTAINER | 0         |
+--------------+---------+-----------------------+------+-----------+-----------+
| mkdocs-new   | RUNNING | 10.187.219.193 (eth0) |      | CONTAINER | 0         |
+--------------+---------+-----------------------+------+-----------+-----------+
| test-version | RUNNING | 10.187.219.18 (eth0)  |      | CONTAINER | 0         |
+--------------+---------+-----------------------+------+-----------+-----------+
$ lxc snapshot mkdocs-new mkdocs-snap1
lxc list
+--------------+---------+-----------------------+------+-----------+-----------+
|     NAME     |  STATE  |         IPV4          | IPV6 |   TYPE    | SNAPSHOTS |
+--------------+---------+-----------------------+------+-----------+-----------+
| mkdocs       | RUNNING | 10.187.219.224 (eth0) |      | CONTAINER | 0         |
+--------------+---------+-----------------------+------+-----------+-----------+
| mkdocs-new   | RUNNING | 10.187.219.193 (eth0) |      | CONTAINER | 1         |
+--------------+---------+-----------------------+------+-----------+-----------+
| test-version | RUNNING | 10.187.219.18 (eth0)  |      | CONTAINER | 0         |
+--------------+---------+-----------------------+------+-----------+-----------+

As you can see above, the mkdocs-new container now has a snapshot.

Further, for whatever reason (with my LXD instance anyway) using the create keyword, doesn't work at all, and gives a usage command that shows using the "create" keyword. If I use it, however, I get this instead:

$ lxc snapshot create mkdocs-new snap2

shows the command help dialog, with this included as an example:

Examples:
lxc snapshot create u1 snap0
Create a snapshot of "u1" called "snap0".

and then:

Error: Invalid number of arguments

So my instance of LXD doesn't support the same command structure.

@sspencerwire sspencerwire merged commit 0ac1f5d into rocky-linux:main Dec 31, 2024
3 checks passed
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