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

Fix CI tests #335

Merged
merged 4 commits into from
Feb 8, 2022
Merged

Fix CI tests #335

merged 4 commits into from
Feb 8, 2022

Conversation

Zlopez
Copy link
Collaborator

@Zlopez Zlopez commented Feb 1, 2022

This PR fixes CI failures that are currently happening.

@softwarefactory-project-zuul
Copy link

Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci

@softwarefactory-project-zuul
Copy link

Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci

@softwarefactory-project-zuul
Copy link

Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci

@Zlopez
Copy link
Collaborator Author

Zlopez commented Feb 1, 2022

This issue is strange, I can't reproduce it locally at all. Even the patch error happening earlier wasn't happening on my machine.

@softwarefactory-project-zuul
Copy link

Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci

@softwarefactory-project-zuul
Copy link

Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci

The tox tests skipped any argument provided by user after tox arguments
in testenv target. This commit is fixing this behaviour.

Signed-off-by: Michal Konečný <mkonecny@redhat.com>
Signed-off-by: Michal Konečný <mkonecny@redhat.com>
The python script for creating DB schema image needs models to be
loaded, this needs to have configuration in place. We just copy the
configuration from project root to docs/ folder to be able to generate
the image.

Signed-off-by: Michal Konečný <mkonecny@redhat.com>
@softwarefactory-project-zuul
Copy link

Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci

@Zlopez
Copy link
Collaborator Author

Zlopez commented Feb 8, 2022

recheck

@softwarefactory-project-zuul
Copy link

Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci

@softwarefactory-project-zuul
Copy link

Build failed. More information on how to proceed and troubleshoot errors available at https://fedoraproject.org/wiki/Zuul-based-ci

This change is needed because the label used for the job was changed.

Signed-off-by: Michal Konečný <mkonecny@redhat.com>
@softwarefactory-project-zuul
Copy link

Build succeeded.

@Zlopez Zlopez merged commit 5c6bda9 into develop Feb 8, 2022
@Zlopez Zlopez deleted the zlopez/tests branch February 8, 2022 12:25
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.

1 participant