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

Add audit middleware test class for ironic-api #1223

Closed
wants to merge 2 commits into from
Closed

Add audit middleware test class for ironic-api #1223

wants to merge 2 commits into from

Conversation

MylesJP
Copy link
Contributor

@MylesJP MylesJP commented Jun 17, 2024

New test class for extension of Keystone Audit Middleware tests for charm Ironic-API. This extension is required because Ironic-API does not use an api-paste.ini file and the existing tests are not relevant.

Copy link
Member

@freyes freyes left a comment

Choose a reason for hiding this comment

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

What's the associate review.opendev.org patch that's exercising this PR?, the one that has the "Func-Test-Pr: https://..." header.

Copy link
Member

@freyes freyes left a comment

Choose a reason for hiding this comment

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

Since you will need to backport this PR in the future, it's better if you squash these two commits.

Ref: https://gist.github.com/lpranam/4ae996b0a4bc37448dc80356efbca7fa

@MylesJP
Copy link
Contributor Author

MylesJP commented Jun 19, 2024

What's the associate review.opendev.org patch that's exercising this PR?, the one that has the "Func-Test-Pr: https://..." header.

https://review.opendev.org/c/openstack/charm-ironic-api/+/922144

The CI is currently being hung up on the jammy-caracal build with ironic-api but passes the jammy-bobcat build.

MylesJP and others added 2 commits June 19, 2024 10:49
Added general class for testing keystone audit middleware
functionality in charms. Tests correct rendering of
api-paste.ini file. Includes class that skips tests for
charms that do not use an api-paste.ini file.

Example usage with charm Heat:

tests/tests.yaml

tests:
  - zaza.openstack.charm_tests.audit.tests.KeystoneAuditMiddlewareTest

tests_options:
  audit-middleware:
    service: heat
@MylesJP MylesJP closed this Jun 19, 2024
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