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

Use rally-openstack's upper constraints #18

Merged
merged 1 commit into from
Apr 15, 2024
Merged

Conversation

markgoddard
Copy link

docker-rally images built at present without upper constraints fail at the end of a rally/tempest verification with the following errors:

<class 'memoryview'> can't be decoded
Missing argument: --uuid

It's unclear what is causing the problem, but rebuilding the image with dependencies pinned to the last known working image (the 1.4-dev tag) results in a working image.

Looking at the rally-openstack source, the project maintains its own upper constraints file that it uses in testing. It makes sense for us to also use this to aim for a consistent and tested set of dependencies.

This change also adds Docker build args for the rally-openstack repo, version and upper constraints.

docker-rally images built at present without upper constraints fail at
the end of a rally/tempest verification with the following errors:

    <class 'memoryview'> can't be decoded
    Missing argument: --uuid

It's unclear what is causing the problem, but rebuilding the image with
dependencies pinned to the last known working image (the 1.4-dev tag)
results in a working image.

Looking at the rally-openstack source, the project maintains its own
upper constraints file that it uses in testing. It makes sense for us to
also use this to aim for a consistent and tested set of dependencies.

This change also adds Docker build args for the rally-openstack repo,
version and upper constraints.
@markgoddard markgoddard merged commit a0f41d7 into master Apr 15, 2024
1 check passed
@markgoddard markgoddard deleted the upper-constraints branch April 15, 2024 14:55
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