Skip to content

Use the correct libvirt pool instead of hardcoded default#2147

Merged
supakeen merged 1 commit into
osbuild:mainfrom
FrostyX:upload-libvirt-pool
Jan 22, 2026
Merged

Use the correct libvirt pool instead of hardcoded default#2147
supakeen merged 1 commit into
osbuild:mainfrom
FrostyX:upload-libvirt-pool

Conversation

@FrostyX

@FrostyX FrostyX commented Jan 20, 2026

Copy link
Copy Markdown
Contributor

We always used the "default"

$ image-builder upload -to libvirt --libvirt-pool images ...
...
error: Failed to find storage pool: virError(Code=49, Domain=18, Message='Storage pool not found: no storage pool with matching name 'default'')

We always used the "default"

    $ image-builder upload -to libvirt --libvirt-pool images ...
    ...
    error: Failed to find storage pool: virError(Code=49, Domain=18, Message='Storage pool not found: no storage pool with matching name 'default'')
@FrostyX
FrostyX requested a review from a team as a code owner January 20, 2026 10:19
@FrostyX
FrostyX requested review from croissanne, lzap and thozza January 20, 2026 10:19
@supakeen

Copy link
Copy Markdown
Member

Test might be nice if you have some time :)

@FrostyX

FrostyX commented Jan 20, 2026

Copy link
Copy Markdown
Contributor Author

Not sure if I can whip out some test in a timely manner :-/
Many things going on currently, so I was happy to do at least this quick fix.

@supakeen
supakeen enabled auto-merge January 20, 2026 16:09
@supakeen
supakeen added this pull request to the merge queue Jan 22, 2026
Merged via the queue into osbuild:main with commit d0bf3a9 Jan 22, 2026
25 checks passed
achilleas-k pushed a commit to achilleas-k/image-builder that referenced this pull request Jun 17, 2026
This bumps the osbuild/images to the version that uses depsolver V2
API.

Also bump the minimum required osbuild version and pin it in the
Schutzfile.

Changes with 0.235.0

----------------
  - Align the bootc distro package with the generic one (osbuild#2136)
    - Author: Achilleas Koutsou, Reviewers: Lukáš Zapletal, Tomáš Hozza
  - Ensure pinned osbuild version is used consistently in CI and build command. (HMS-10070) (osbuild#2145)
    - Author: Tomáš Hozza, Reviewers: Achilleas Koutsou, Simon de Vlieger
  - GHA: bump used golangci-lint version to 2.8 (HMS-10069) (osbuild#2144)
    - Author: Tomáš Hozza, Reviewers: Achilleas Koutsou, Lukáš Zapletal
  - Update osbuild dependency commit ID (osbuild#2141)
    - Author: SchutzBot, Reviewers: Achilleas Koutsou, Tomáš Hozza
  - Use the correct libvirt pool instead of hardcoded default (osbuild#2147)
    - Author: Jakub Kadlčík, Reviewers: Achilleas Koutsou, Simon de Vlieger
  - boot-image: add make_check_host_config to boot-aws (osbuild#2143)
    - Author: Lukáš Zapletal, Reviewers: Simon de Vlieger, Tomáš Hozza
  - osbuild: add force option to org.osbuild.groups and always enable [HMS-9825] (osbuild#2146)
    - Author: Achilleas Koutsou, Reviewers: Brian C. Lane, Lukáš Zapletal, Tomáš Hozza

— Somewhere on the Internet, 2026-01-26

---

Changes with 0.236.0

----------------
  - Add V2 depsolver API support and make it the default (HMS-10029) (osbuild#2139)
    - Author: Tomáš Hozza, Reviewers: Lukáš Zapletal, Simon de Vlieger

— Somewhere on the Internet, 2026-01-26

Signed-off-by: Tomáš Hozza <thozza@redhat.com>
achilleas-k pushed a commit to achilleas-k/image-builder that referenced this pull request Jun 17, 2026
This bumps the osbuild/images to the version that uses depsolver V2
API.

Also bump the minimum required osbuild version and pin it in the
Schutzfile.

Changes with 0.235.0

----------------
  - Align the bootc distro package with the generic one (osbuild#2136)
    - Author: Achilleas Koutsou, Reviewers: Lukáš Zapletal, Tomáš Hozza
  - Ensure pinned osbuild version is used consistently in CI and build command. (HMS-10070) (osbuild#2145)
    - Author: Tomáš Hozza, Reviewers: Achilleas Koutsou, Simon de Vlieger
  - GHA: bump used golangci-lint version to 2.8 (HMS-10069) (osbuild#2144)
    - Author: Tomáš Hozza, Reviewers: Achilleas Koutsou, Lukáš Zapletal
  - Update osbuild dependency commit ID (osbuild#2141)
    - Author: SchutzBot, Reviewers: Achilleas Koutsou, Tomáš Hozza
  - Use the correct libvirt pool instead of hardcoded default (osbuild#2147)
    - Author: Jakub Kadlčík, Reviewers: Achilleas Koutsou, Simon de Vlieger
  - boot-image: add make_check_host_config to boot-aws (osbuild#2143)
    - Author: Lukáš Zapletal, Reviewers: Simon de Vlieger, Tomáš Hozza
  - osbuild: add force option to org.osbuild.groups and always enable [HMS-9825] (osbuild#2146)
    - Author: Achilleas Koutsou, Reviewers: Brian C. Lane, Lukáš Zapletal, Tomáš Hozza

— Somewhere on the Internet, 2026-01-26

---

Changes with 0.236.0

----------------
  - Add V2 depsolver API support and make it the default (HMS-10029) (osbuild#2139)
    - Author: Tomáš Hozza, Reviewers: Lukáš Zapletal, Simon de Vlieger

— Somewhere on the Internet, 2026-01-26

Signed-off-by: Tomáš Hozza <thozza@redhat.com>
achilleas-k pushed a commit to achilleas-k/image-builder that referenced this pull request Jun 18, 2026
This bumps the osbuild/images to the version that uses depsolver V2
API.

Also bump the minimum required osbuild version and pin it in the
Schutzfile.

Changes with 0.235.0

----------------
  - Align the bootc distro package with the generic one (osbuild#2136)
    - Author: Achilleas Koutsou, Reviewers: Lukáš Zapletal, Tomáš Hozza
  - Ensure pinned osbuild version is used consistently in CI and build command. (HMS-10070) (osbuild#2145)
    - Author: Tomáš Hozza, Reviewers: Achilleas Koutsou, Simon de Vlieger
  - GHA: bump used golangci-lint version to 2.8 (HMS-10069) (osbuild#2144)
    - Author: Tomáš Hozza, Reviewers: Achilleas Koutsou, Lukáš Zapletal
  - Update osbuild dependency commit ID (osbuild#2141)
    - Author: SchutzBot, Reviewers: Achilleas Koutsou, Tomáš Hozza
  - Use the correct libvirt pool instead of hardcoded default (osbuild#2147)
    - Author: Jakub Kadlčík, Reviewers: Achilleas Koutsou, Simon de Vlieger
  - boot-image: add make_check_host_config to boot-aws (osbuild#2143)
    - Author: Lukáš Zapletal, Reviewers: Simon de Vlieger, Tomáš Hozza
  - osbuild: add force option to org.osbuild.groups and always enable [HMS-9825] (osbuild#2146)
    - Author: Achilleas Koutsou, Reviewers: Brian C. Lane, Lukáš Zapletal, Tomáš Hozza

— Somewhere on the Internet, 2026-01-26

---

Changes with 0.236.0

----------------
  - Add V2 depsolver API support and make it the default (HMS-10029) (osbuild#2139)
    - Author: Tomáš Hozza, Reviewers: Lukáš Zapletal, Simon de Vlieger

— Somewhere on the Internet, 2026-01-26

Signed-off-by: Tomáš Hozza <thozza@redhat.com>
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