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

PXP-7807 Audit-service tests #644

Merged
merged 20 commits into from
Jul 14, 2021
Merged

Conversation

paulineribeyre
Copy link
Contributor

Jira Ticket: PXP-7807

New Features

  • Add tests for the audit-service integration with Fence: download and login events

Improvements

  • Fix bug in indexd tasks "addFileIndices" function causing error messages "file is not defined"
  • Refactor RAS client creation logic: remove code duplication, add error logs, and add a random string at the end of the client names to avoid "client already exists" issues
  • Refactor "smartWait" function so that it doesn't wait for nothing at the end of the loop

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
Copy link
Contributor

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.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

sure

Copy link
Contributor

@atharvar28 atharvar28 left a comment

Choose a reason for hiding this comment

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

👍

@paulineribeyre paulineribeyre marked this pull request as draft July 8, 2021 21:17
@paulineribeyre
Copy link
Contributor Author

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

@themarcelor
Copy link
Contributor

themarcelor commented Jul 14, 2021

@paulineribeyre , if you want to experiment further in the wild, you can just comment out the run-tests.sh logic and leave it as @donot '@audit', merge what you have so far and label specific PRs in gitops-qa and/pr cdis-manifest with "suites-apis-auditServiceTest" for further experimentation (or even label to select specific jenkins envs or even try that in ci-env-1 in qaplanetv2).

Of course, just a suggestion if you rather try iterative improvements / deliverables.

@paulineribeyre
Copy link
Contributor Author

@themarcelor thanks - i think i'm "almost there" 🤞

@paulineribeyre paulineribeyre marked this pull request as ready for review July 14, 2021 19:30
@paulineribeyre paulineribeyre merged commit e9177b0 into master Jul 14, 2021
@paulineribeyre paulineribeyre deleted the feat/audit-service-tests branch July 14, 2021 22:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants