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

update share test, production manila audit map, add debug for tox #94

Merged
merged 4 commits into from
Aug 7, 2024

Conversation

notque
Copy link
Contributor

@notque notque commented Aug 6, 2024

No description provided.

@notque
Copy link
Contributor Author

notque commented Aug 7, 2024

For unknown reasons, the create_notifier() in test_api_request_no_messaging used "oslo_messaging.Notifier" instead of the "_LogNotifier" that should be originally used.

This patch fixes this issue by changing "use_oslo_messaging" config to false

@notque notque self-assigned this Aug 7, 2024
@notque notque requested a review from rajivmucheli August 7, 2024 03:33
@notque notque merged commit fe80ece into master Aug 7, 2024
1 check passed
@notque notque deleted the read_list_error branch August 7, 2024 06:27
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