Skip to content

feat: add additional tempest tests #1513

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Feb 18, 2025
Merged

feat: add additional tempest tests #1513

merged 2 commits into from
Feb 18, 2025

Conversation

jackhodgkiss
Copy link
Contributor

This commit adds additional tempest tests for the following services:

  • barbican
  • cinder
  • cloudkitty
  • designate
  • glance
  • ironic
  • keystone
  • magnum
  • manila
  • neutron
  • octavia

@jackhodgkiss jackhodgkiss added the enhancement New feature or request label Feb 14, 2025
@jackhodgkiss jackhodgkiss self-assigned this Feb 14, 2025
@Alex-Welsh
Copy link
Member

  1. Can we have a list that includes everything?
  2. What's the next step to get this out of draft?
  3. Can you add a release note?

@jackhodgkiss jackhodgkiss mentioned this pull request Feb 17, 2025
Draft
@jackhodgkiss
Copy link
Contributor Author

  1. Can we have a list that includes everything?

  2. What's the next step to get this out of draft?

  3. Can you add a release note?

  1. Not sure what the motivation would be running just 30 Octavia tests takes three hours.
  2. We need to finalise the rally-docker image and update .automation to use the new image
  3. Will do once .automation has been updated.

@jackhodgkiss
feat: add additional tempest tests
cbe0cde 
This commit adds additional tempest tests for the following services:
  - barbican
  - cinder
  - cloudkitty
  - designate
  - glance
  - ironic
  - keystone
  - magnum
  - manila
  - neutron
  - octavia

It also upgrades `kayobe-automation` in order to use the latest
`docker-rally` image.
@jackhodgkiss jackhodgkiss force-pushed the add-more-tempest-tests branch from e9e05d5 to cbe0cde Compare February 17, 2025 15:22
@jackhodgkiss jackhodgkiss marked this pull request as ready for review February 17, 2025 15:25
@jackhodgkiss jackhodgkiss requested a review from a team as a code owner February 17, 2025 15:25
jovial
jovial previously approved these changes Feb 17, 2025
View reviewed changes
Copy link
Contributor

@jovial jovial left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. We can always tune the lists later.

Alex-Welsh
Alex-Welsh reviewed Feb 18, 2025
View reviewed changes
@Alex-Welsh Alex-Welsh enabled auto-merge (squash) February 18, 2025 09:47
@jackhodgkiss jackhodgkiss merged commit cd0d44e into stackhpc/2024.1 Feb 18, 2025
14 checks passed
@jackhodgkiss jackhodgkiss deleted the add-more-tempest-tests branch February 18, 2025 11:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Alex-Welsh Alex-Welsh Alex-Welsh approved these changes

@jovial jovial jovial left review comments

Assignees

@jackhodgkiss jackhodgkiss

Labels
enhancement New feature or request kayobe-automation size: xl
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jackhodgkiss @Alex-Welsh @jovial