Skip to content

Commit

Permalink
Molecule: Different fix for new Cython
Browse files Browse the repository at this point in the history
The first attempt (ansible#1492) was unfortunately wrong in two ways. We need
to pin the dependency in inside the container, not as a molecule
dependency. Also it seems like the Cython pin needs to happen *earlier*
but an easy solution is to just pin pyyaml to an old version instead for
now until yaml/pyyaml#601 is resolved.

Signed-off-by: Rick Elrod <rick@elrod.me>
  • Loading branch information
relrod committed Jul 17, 2023
1 parent 45f7545 commit 546181b
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 1 deletion.
1 change: 1 addition & 0 deletions molecule/kind/converge.yml
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@
- urllib3==1.26.15
- docker
- docker-compose
- pyyaml==5.3.1
state: present

- name: Build operator image
Expand Down
1 change: 0 additions & 1 deletion molecule/requirements.txt
Original file line number Diff line number Diff line change
Expand Up @@ -6,4 +6,3 @@ openshift!=0.13.0
jmespath
ansible-core
ansible-compat<4 # https://github.com/ansible-community/molecule/issues/3903
Cython<3 # https://github.com/yaml/pyyaml/issues/601

0 comments on commit 546181b

Please sign in to comment.