Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

update ezxenstore.opam dependencies [epoll] #700

Merged
merged 2 commits into from
Sep 20, 2024

Conversation

edwintorok
Copy link
Contributor

Equivalent commit in XAPI is in this PR:
xapi-project/xen-api@0a8fc6e

This is fallout from the epoll merge which introduced this additional dependency, and although it didn't cause a failure in the XAPI CI, it did one here.

@edwintorok edwintorok changed the title update ezxenstore.opam dependencies update ezxenstore.opam dependencies [epoll] Sep 19, 2024
Signed-off-by: Edwin Török <edwin.torok@cloud.com>
It was missing a dependency on rrd-transport.

Signed-off-by: Edwin Török <edwin.torok@cloud.com>
@edwintorok
Copy link
Contributor Author

There also appears to be a missing dependency between xapi-rrdd and rrd-transport that caused a failure in my other PR (4.14.2 failed, 4.14.1 passed, so its a race depending what order packages get installed).
A bit strange as this hasn't change with epoll, anyway equivalent fix on XAPI here: xapi-project/xen-api@85ffeb4 and pushed a commit to add the dependency here.

@psafont psafont merged commit d3bf1b7 into xapi-project:master Sep 20, 2024
3 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants