Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Modify requirements content #256

Merged
merged 2 commits into from
Feb 21, 2024
Merged

Conversation

rjrpaz
Copy link
Contributor

@rjrpaz rjrpaz commented Feb 21, 2024

No description provided.

@pierre
Copy link
Member

pierre commented Feb 21, 2024

@rjrpaz Could you take a look at the docker_ci job errors? Wondering if the Docker build needs to be updated too?

kpm_ci job failure is tracked by #254

@rjrpaz
Copy link
Contributor Author

rjrpaz commented Feb 21, 2024

  • docker_ci error is related to the same thing about trying to update the ansible collection using https. I fixed it but the pipeline tries to download the repo using a branch that does not exist in the original repo (my user was not able to do new branches in the original repo so I had to a fork and create the new repo there). If you run this pipeline from the original repo and the branch exists, that step should work.
  • kpm_ci failed because of a ruby gems update trying to install a higher version. I also updated that but now the pipeline fails as part of unit testing. Your team members will probably know better about these errors.

@pierre pierre merged commit e3e2c96 into killbill:master Feb 21, 2024
2 of 5 checks passed
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.

2 participants