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

Uncap pbr #173

Merged
merged 1 commit into from May 30, 2018
Merged

Uncap pbr #173

merged 1 commit into from May 30, 2018

Conversation

mtreinish
Copy link
Owner

This commit uncaps the pbr version now that the windows issue reported
in: https://bugs.launchpad.net/pbr/+bug/1761134 is fixed.

@coveralls
Copy link

coveralls commented May 30, 2018

Coverage Status

Coverage remained the same at 64.785% when pulling 637ee6a on uncap-pip into 8a8db7c on master.

This commit uncaps the pbr version. The PBR windows issue reported
in: https://bugs.launchpad.net/pbr/+bug/1761134 no longer seems to
effect the windows CI jobs. Since this is the only method we have to
test functionality on windows this should be safe. If there are still
configurations that this doesn't work on, they'll have to cap the PBR
version downstream. The requirements.txt still sets a low minimum bound
to enable doing this.
@mtreinish mtreinish changed the title WIP: Uncap pbr Uncap pbr May 30, 2018
@mtreinish mtreinish merged commit f4d2394 into master May 30, 2018
@mtreinish mtreinish deleted the uncap-pip branch May 30, 2018 23:54
@mtreinish mtreinish mentioned this pull request May 31, 2018
mtreinish added a commit that referenced this pull request May 31, 2018
Revert "Uncap pbr"

Reverts #173

This was only a partial uncap as it was still capped in the setup.py for stestr. Meaning the passing tests were still using pbr<4.0.0. When you uncap the setup.py too, like in #174, things start failing again. The underlying issue hasn't been addressed yet so we can't uncap the requirement yet.
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.

None yet

2 participants