XCP-ng
    • Categories
    • Recent
    • Tags
    • Popular
    • Users
    • Groups
    • Register
    • Login

    Manual snapshots retention

    Scheduled Pinned Locked Moved Solved Management
    45 Posts 5 Posters 640 Views 4 Watching
    Loading More Posts
    • Oldest to Newest
    • Newest to Oldest
    • Most Votes
    Reply
    • Reply as topic
    Log in to reply
    This topic has been deleted. Only users with topic management privileges can see it.
    • M Offline
      manilx @olivierlambert
      last edited by

      @olivierlambert 30 days seems OK to me.

      1 Reply Last reply Reply Quote 0
      • Tristis OrisT Offline
        Tristis Oris Top contributor @olivierlambert
        last edited by

        @olivierlambert well, 30 days is a super maximum for any real needs to restore anything.

        1 Reply Last reply Reply Quote 0
        • olivierlambertO Offline
          olivierlambert Vates 🪐 Co-Founder CEO
          last edited by

          Done: https://github.com/vatesfr/xen-orchestra/issues/8258

          olivierlambert created this issue in vatesfr/xen-orchestra

          closed Healthview: display snapshots older than 30 days #8258

          D 1 Reply Last reply Reply Quote 2
          • D Offline
            DustinB @olivierlambert
            last edited by

            @olivierlambert I added a response in the hopes that it clarifies the end-goal.

            F 1 Reply Last reply Reply Quote 0
            • F Offline
              flakpyro @DustinB
              last edited by

              @olivierlambert I wonder if it would be beneficial to show all snapshots older than 30 days not just ones not created by an automated process. For example what happens if a XO backup job runs, creates a snapshot but the process is interrupted and the job fails. Will the next time the job runs clean up the previous failed jobs snapshot or is there a chance a snapshot could be left behind?

              Tristis OrisT D 2 Replies Last reply Reply Quote 0
              • Tristis OrisT Offline
                Tristis Oris Top contributor @flakpyro
                last edited by

                @flakpyro true, sometimes that happens and they will never be deleted.
                I'm don't remember, but maybe such snaspshots not visible as orphaned.

                1 Reply Last reply Reply Quote 0
                • D Offline
                  DustinB @flakpyro
                  last edited by

                  @flakpyro said in Manual snapshots retention:

                  @olivierlambert I wonder if it would be beneficial to show all snapshots older than 30 days not just ones not created by an automated process. For example what happens if a XO backup job runs, creates a snapshot but the process is interrupted and the job fails. Will the next time the job runs clean up the previous failed jobs snapshot or is there a chance a snapshot could be left behind?

                  Garbage collection should be handling this...

                  1 Reply Last reply Reply Quote 0
                  • olivierlambertO Offline
                    olivierlambert Vates 🪐 Co-Founder CEO
                    last edited by

                    In theory there's no such thing as XO checks. Now for the feature request, I prefer the idea of showing manual leftovers and leave the admin to decide what to do with it 🙂

                    1 Reply Last reply Reply Quote 2
                    • Tristis OrisT Offline
                      Tristis Oris Top contributor
                      last edited by

                      you say here that this feature is implemented, but i don't see new menu. ec782

                      1 Reply Last reply Reply Quote 0
                      • olivierlambertO Offline
                        olivierlambert Vates 🪐 Co-Founder CEO
                        last edited by

                        I think the dev working on it had personal issues few days before the release, so at worst it will be there in a patch release or in 5.106. There's so much things in the pipe, it's possible sometimes they aren't shipped on D-day. But it will be there finally 🙂

                        Tristis OrisT 1 Reply Last reply Reply Quote 1
                        • Tristis OrisT Offline
                          Tristis Oris Top contributor @olivierlambert
                          last edited by

                          @olivierlambert haha, i'm watching you)

                          1 Reply Last reply Reply Quote 0
                          • Tristis OrisT Offline
                            Tristis Oris Top contributor
                            last edited by

                            Merged. Works as intented, thank you.

                            1 Reply Last reply Reply Quote 1
                            • Tristis OrisT Tristis Oris marked this topic as a question on
                            • Tristis OrisT Tristis Oris has marked this topic as solved on
                            • Tristis OrisT Offline
                              Tristis Oris Top contributor
                              last edited by

                              i found something to remove. Need to change text here, it too scary.

                              9a4fbfac-eb61-4dc9-9273-c4a38779c7b2-image.png

                              M 1 Reply Last reply Reply Quote 0
                              • M Offline
                                manilx @Tristis Oris
                                last edited by

                                @Tristis-Oris I had the SAME thing today. The message scared me for 1min before I hit go.......

                                1 Reply Last reply Reply Quote 0
                                • olivierlambertO Offline
                                  olivierlambert Vates 🪐 Co-Founder CEO
                                  last edited by olivierlambert

                                  Hi,

                                  I'm not sure to understand the issue. Removing those old snapshots makes you nervous because on the message telling you if you remove it the data associated with it will be remove? Or maybe it's just a phrasing thing? Like all "VM SNAPSHOTS" will be removed instead of VM only?

                                  Tristis OrisT 1 Reply Last reply Reply Quote 0
                                  • Tristis OrisT Offline
                                    Tristis Oris Top contributor @olivierlambert
                                    last edited by

                                    @olivierlambert "delete vm" "ALL vm disks will be removed" that wrong at least.
                                    Yes, need to change it to snapshots.

                                    1 Reply Last reply Reply Quote 0
                                    • olivierlambertO Offline
                                      olivierlambert Vates 🪐 Co-Founder CEO
                                      last edited by

                                      Why it's wrong? All the VM disks for this VM will be removed.

                                      Remember: there's no "snapshot" object, it's only a VM with "is a snapshot: true".

                                      Tristis OrisT M 2 Replies Last reply Reply Quote 0
                                      • Tristis OrisT Offline
                                        Tristis Oris Top contributor @olivierlambert
                                        last edited by

                                        @olivierlambert But on my side VM it's VM. Snapshot it's something else)
                                        And not "all", "selected" sounds better.

                                        1 Reply Last reply Reply Quote 0
                                        • M Offline
                                          manilx @olivierlambert
                                          last edited by

                                          @olivierlambert !Delete VMs ->Delete VM Snapshots

                                          delete 7 vms -> delete 7 VM snapshots

                                          I you don't know as it is it scares you. Did me!

                                          1 Reply Last reply Reply Quote 0
                                          • olivierlambertO Offline
                                            olivierlambert Vates 🪐 Co-Founder CEO
                                            last edited by olivierlambert

                                            Yes, I get it. But that explains why the message is generic for VM removal (because it's the same that's a VM with snapshot true or false).

                                            I agree this could improved, but I'm not sure this worth the effort vs moving forward on XO 6/XO Lite 🤔

                                            Tristis OrisT M 2 Replies Last reply Reply Quote 1
                                            • First post
                                              Last post