-
Notifications
You must be signed in to change notification settings - Fork 4
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
PXP-7807 Audit-service tests #644
Conversation
run-tests.sh
Outdated
# they don't run for manifest repos because the audit-service might not | ||
# be deployed everywhere. | ||
# | ||
if [[ "$isGen3Release" != "true" && "$service" != "gen3-qa" && "$service" != "fence" && "$service" != "audit-service" && "$service" != "cloud-automation" ]]; then |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Don't we want to make sure we have CI with diff mix-and-match of versions in cdis-manifest PRs?
We should be able to figure out whether audit-service is configured against a given environment by checking the manifest versions block and/or the portal/gitops.json.
Just checking, if you prefer this approach, that's ok.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sure
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Not ready to merge - I had to revert my Fence config changes in all the Jenkins environments because of "Audit logs are enabled but audit-service is unreachable" errors in cdis-manifest PRs without audit-service. Looking for a solution |
@paulineribeyre , if you want to experiment further in the wild, you can just comment out the Of course, just a suggestion if you rather try iterative improvements / deliverables. |
@themarcelor thanks - i think i'm "almost there" 🤞 |
Jira Ticket: PXP-7807
New Features
Improvements