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

    Coalesce not showing anymore

    Scheduled Pinned Locked Moved Solved Xen Orchestra
    13 Posts 5 Posters 1.0k Views 5 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.
    • olivierlambertO Offline
      olivierlambert Vates 🪐 Co-Founder CEO
      last edited by olivierlambert

      Let me add @julien-f in there

      edit: are you certain it should be the case? can you find a previous commit where it works?

      GheppyG 1 Reply Last reply Reply Quote 0
      • GheppyG Offline
        Gheppy @olivierlambert
        last edited by

        @olivierlambert I'm working on it, I'm looking for a commit that works

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

          Thanks!

          1 Reply Last reply Reply Quote 0
          • GheppyG Offline
            Gheppy
            last edited by

            this commit cb1223f72ead16db61b884a39b430b3be041e39d shows
            starting with this commit a9c123914945f72e55fb2d866e3cfccbfee3d3b1, no

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

              Okay so https://github.com/vatesfr/xen-orchestra/commit/a9c123914945f72e55fb2d866e3cfccbfee3d3b1 might have broke the VDI to coalesce display. @julien-f does it sound plausible? This commit is indeed playing with VDIs parents.

              0 MathieuRA committed to vatesfr/xen-orchestra
              feat(xo-server/xo-web/health): detect invalid vhd-parent VDIs (#6356)
              julien-fJ 1 Reply Last reply Reply Quote 0
              • julien-fJ Offline
                julien-f Vates 🪐 Co-Founder XO Team @olivierlambert
                last edited by

                @olivierlambert Yep, that's possible.

                @MathieuRA Can you take a look?

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

                  Thanks @Gheppy for the report!

                  1 Reply Last reply Reply Quote 0
                  • GheppyG Offline
                    Gheppy
                    last edited by

                    with pleasure

                    1 Reply Last reply Reply Quote 1
                    • MathieuRAM Offline
                      MathieuRA Vates 🪐 XO Team @julien-f
                      last edited by

                      @julien-f Yes, I will investigate

                      1 Reply Last reply Reply Quote 0
                      • MathieuRAM Offline
                        MathieuRA Vates 🪐 XO Team
                        last edited by MathieuRA

                        The fix has been pushed to the master branch.

                        E 1 Reply Last reply Reply Quote 1
                        • E Offline
                          errellion @MathieuRA
                          last edited by

                          @MathieuRA Confirmed, looks good! Thanks.

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

                            Yay! Thanks both @Gheppy for the report and @errellion for the feedback 🙂

                            1 Reply Last reply Reply Quote 0
                            • olivierlambertO olivierlambert marked this topic as a question on
                            • olivierlambertO olivierlambert has marked this topic as solved on
                            • First post
                              Last post