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

MODHAADM-71 log pruning #118

Merged
merged 22 commits into from
Sep 18, 2024
Merged

MODHAADM-71 log pruning #118

merged 22 commits into from
Sep 18, 2024

Conversation

nielserik
Copy link
Collaborator

Add Okapi timer process that will purge past logs at a scheduled interval and with a configurable age limit for past jobs and logs.

nielserik added 22 commits July 25, 2024 15:45
  * add Okapi timer process for deleting old jobs, record failures and logs
  * add APIs for importing jobs, record failures and logs, i.e. from exports or from other FOLIO instances
  * add Okapi timer process for deleting old jobs, record failures and logs
  * add APIs for importing jobs, record failures and logs, i.e. from exports or from other FOLIO instances
  Reorganize Java classes (rename packages, classes, move classes)
  Update logging dependencies
  Add Okapi client for configuration look-up
  Preparations for unit tests (settable clock, APIs for populating jobs and logs with sample data)
Remove test resources directory
Expand test harness to cover local mod-harvester-admin APIs (purge old jobs) and FOLIO APIs (/configuration/entries) along existing legacy harvester interaction tests.
  After renaming the test suite from HarvesterAdminTestSuite to HarvesterAdminTestSuiteIT, the environment variables set in the configuration of the maven-surefire-plugin are no longer available in the module when running the unit tests by `mvn install` on the command line. Reverting the name to HarvesterAdminTestSuite, at least until it is clear why the env vars disappear. When running the tests in IDEA, the environment variables seems to be forwarded fine with either name for the unit test suite class.
  Adding profile for running tests against installed and running Harvester, activate with `-PharvesterTests`
  Default is to run unit tests that doesn't require Harvester
 - requires Harvester to be running (at localhost:8080)
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
11.4% Coverage on New Code (required ≥ 80%)
E Security Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

@nielserik nielserik merged commit c803e94 into master Sep 18, 2024
5 of 6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant