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 Makefile for CI jobs #3046

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

stephenfin
Copy link
Contributor

Start using the Makefile added in #2940 for our CI jobs. This requires #3005 to work correctly.

@EmilienM
Copy link
Contributor

EmilienM commented May 8, 2024

awesome! thanks

@github-actions github-actions bot added the semver:patch No API change label May 8, 2024
@coveralls
Copy link

coveralls commented May 8, 2024

Coverage Status

coverage: 78.696%. remained the same
when pulling 735bc6a on stephenfin:use-makefile-in-ci
into 571ae75 on gophercloud:master.

@github-actions github-actions bot removed the semver:patch No API change label May 15, 2024
@stephenfin stephenfin marked this pull request as ready for review May 15, 2024 14:44
@github-actions github-actions bot added the semver:patch No API change label May 15, 2024
@github-actions github-actions bot added semver:patch No API change and removed semver:patch No API change labels May 20, 2024
@github-actions github-actions bot added semver:patch No API change and removed semver:patch No API change labels May 21, 2024
@github-actions github-actions bot added semver:patch No API change and removed semver:patch No API change labels May 22, 2024
EmilienM
EmilienM previously approved these changes May 22, 2024
@stephenfin
Copy link
Contributor Author

stephenfin commented May 22, 2024

@EmilienM The failures of the functional-baremetal job on the master branch looks real and seems like we're not passing through OS_SYSTEM_SCOPE correctly. I'm trying to figure out why now (by increasing verbosity of the stackenv script, among other fixes).

EDIT: tbc, this is separate from the failures against stable/caracal, which are being addressed by #3052

@github-actions github-actions bot added semver:patch No API change and removed semver:patch No API change labels May 22, 2024
@EmilienM
Copy link
Contributor

@dtantsur FYI

@github-actions github-actions bot added semver:patch No API change and removed semver:patch No API change labels May 22, 2024
@github-actions github-actions bot added semver:patch No API change and removed semver:patch No API change labels May 22, 2024
@stephenfin
Copy link
Contributor Author

aaand of course the master job is passing now. Typical 🙄 There's a new failure but that seems to be an upper-constraint conflict

It's more common than the object storage service.

Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
We mangled this, probably since we were renaming the 'imageservice'
module to 'image' at the time.

Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
For use in CI.

Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
We can remove the 'acceptancetest' script in the process. Note that this
isn't a 1:1 match: the fwaasv2 test job now runs the entire networking
test suite and not just the fwaasv2-specific tests but seeing as this is
using OVS rather than OVN that doesn't seem like such a bad thing.

Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
Signed-off-by: Stephen Finucane <stephenfin@redhat.com>
@github-actions github-actions bot added semver:patch No API change and removed semver:patch No API change labels May 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver:patch No API change
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants