Skip to content
This repository has been archived by the owner on Jan 5, 2023. It is now read-only.

fix devel molecule branch from master to main #29

Merged
merged 1 commit into from
Nov 2, 2021
Merged

fix devel molecule branch from master to main #29

merged 1 commit into from
Nov 2, 2021

Conversation

jseguillon
Copy link
Contributor

@jseguillon jseguillon commented Oct 13, 2021 •
edited

Should fix broken CI

@jseguillon
Copy link
Contributor Author

Ok on my fork https://github.com/jseguillon/molecule-libvirt/actions/runs/1337371874

@goldyfruit goldyfruit added the enhancement This issue/PR relates to a feature request. label Oct 13, 2021
@goldyfruit goldyfruit self-assigned this Oct 13, 2021
@jseguillon
Copy link
Contributor Author

Hi @goldyfruit
Could you please run again the CI ? Thanks.

@ssbarnea
Copy link
Member

recheck

@goldyfruit
Copy link
Collaborator

The CI complains about Fedora 34 and Python 3.8

2021-10-25 10:06:09.777546 \| TASK [Ensure python3.8 is present]
655 | 2021-10-25 10:06:12.913587 \| fedora-34 \| ERROR
656 | 2021-10-25 10:06:12.913859 \| fedora-34 \| {
657 | 2021-10-25 10:06:12.913899 \| fedora-34 \|   "failures": [
658 | 2021-10-25 10:06:12.913927 \| fedora-34 \|     "No package python38-devel available."
659 | 2021-10-25 10:06:12.913952 \| fedora-34 \|   ],
660 | 2021-10-25 10:06:12.913977 \| fedora-34 \|   "msg": "Failed to install some of the specified packages",
661 | 2021-10-25 10:06:12.914001 \| fedora-34 \|   "rc": 1,
662 | 2021-10-25 10:06:12.914025 \| fedora-34 \|   "results": []
663 | 2021-10-25 10:06:12.914049 \| fedora-34 \| }

@jseguillon
Copy link
Contributor Author

Too bad. It seems not related to changes from this pr since #30 has same problem.
And I have no idea how I could fix this :/

goldyfruit added a commit to goldyfruit/molecule-libvirt that referenced this pull request Oct 25, 2021
@goldyfruit
[zuul] Use Python 3.9 with Fedora 34
c2bb035 
Zuul complains about python38-devel package not available.
Default Python version on Fedora 34 is Python 3.9.

This should fix issue ansible-community#29.
@goldyfruit goldyfruit mentioned this pull request Oct 25, 2021
Merged
@goldyfruit
Copy link
Collaborator

Too bad. It seems not related to changes from this pr since #30 has same problem. And I have no idea how I could fix this :/

Let see if #31 will fix the issue.

@goldyfruit
Copy link
Collaborator

recheck

@ssbarnea ssbarnea added bug This issue/PR relates to a bug. and removed enhancement This issue/PR relates to a feature request. labels Nov 2, 2021
@ssbarnea ssbarnea merged commit 4e82bdf into ansible-community:main Nov 2, 2021
@goldyfruit
Copy link
Collaborator

Yeah :) @ssbarnea thanks

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ssbarnea ssbarnea ssbarnea approved these changes

@jamesregis jamesregis Awaiting requested review from jamesregis jamesregis is a code owner

Assignees

@goldyfruit goldyfruit

Labels
bug This issue/PR relates to a bug.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@jseguillon @ssbarnea @goldyfruit