git-eca-rest-api merge requestshttps://gitlab.eclipse.org/eclipsefdn/it/api/git-eca-rest-api/-/merge_requests2023-09-26T15:36:14Zhttps://gitlab.eclipse.org/eclipsefdn/it/api/git-eca-rest-api/-/merge_requests/161Iss #147 - Update head SHA on retrieval of tracking2023-09-26T15:36:14ZMartin Lowemartin.lowe@eclipse-foundation.orgIss #147 - Update head SHA on retrieval of trackingAdditionally, add more null checking for the pull requestAdditionally, add more null checking for the pull requesthttps://gitlab.eclipse.org/eclipsefdn/it/api/git-eca-rest-api/-/merge_requests/160Iss #143 - Handle Status event by making PR in hook optional but checked2023-09-26T13:01:41ZMartin Lowemartin.lowe@eclipse-foundation.orgIss #143 - Handle Status event by making PR in hook optional but checkedResolves #143Resolves #143https://gitlab.eclipse.org/eclipsefdn/it/api/git-eca-rest-api/-/merge_requests/159Iss #145 - Add configurable frequency for GH revalidation, fix count2023-09-21T18:11:08ZMartin Lowemartin.lowe@eclipse-foundation.orgIss #145 - Add configurable frequency for GH revalidation, fix counthttps://gitlab.eclipse.org/eclipsefdn/it/api/git-eca-rest-api/-/merge_requests/158Add manual check of security provider presence in JDK for JWT helper2023-09-19T20:34:58ZMartin Lowemartin.lowe@eclipse-foundation.orgAdd manual check of security provider presence in JDK for JWT helperPreviously, we had an issue crop up where the required provider was
absent. This was unexpected and had an unexpected failure with strange
logging. This should help avoid the stack traces and give us clear
indicators of the error.Previously, we had an issue crop up where the required provider was
absent. This was unexpected and had an unexpected failure with strange
logging. This should help avoid the stack traces and give us clear
indicators of the error.https://gitlab.eclipse.org/eclipsefdn/it/api/git-eca-rest-api/-/merge_requests/157Add patch to fix limit use in loading GH install list2023-09-19T16:03:25ZMartin Lowemartin.lowe@eclipse-foundation.orgAdd patch to fix limit use in loading GH install listhttps://gitlab.eclipse.org/eclipsefdn/it/api/git-eca-rest-api/-/merge_requests/156Add error page for GH status validation page for bad requests2023-09-19T15:16:43ZMartin Lowemartin.lowe@eclipse-foundation.orgAdd error page for GH status validation page for bad requestshttps://gitlab.eclipse.org/eclipsefdn/it/api/git-eca-rest-api/-/merge_requests/155Iss #141 - Add DB-backed GH installation cache for better uptime2023-09-14T16:03:21ZMartin Lowemartin.lowe@eclipse-foundation.orgIss #141 - Add DB-backed GH installation cache for better uptimePreviously, we cached installation info live at server start, but this
had awful performance and due to GH API instability, was not reliable.
The new solution should be more stable and better handle disruptions.
This will initially be up...Previously, we cached installation info live at server start, but this
had awful performance and due to GH API instability, was not reliable.
The new solution should be more stable and better handle disruptions.
This will initially be updated hourly by a thread, and in the future
will have updates performed by the already configured webhook.https://gitlab.eclipse.org/eclipsefdn/it/api/git-eca-rest-api/-/merge_requests/154Iss #139 - Change when validation happens for GH status UI endpoint2023-09-11T18:49:36ZMartin Lowemartin.lowe@eclipse-foundation.orgIss #139 - Change when validation happens for GH status UI endpointPreviously, validation happened on every call to this endpoint. That led
to issues with some of our calls where things were being revalidated
when it wasn't needed. This would lead to failures to display, when the
data should be able to ...Previously, validation happened on every call to this endpoint. That led
to issues with some of our calls where things were being revalidated
when it wasn't needed. This would lead to failures to display, when the
data should be able to be presented as is.
Additionally, refactoring was done to split the validation status calls
to a separate service to keep the code cleaner and more legible.https://gitlab.eclipse.org/eclipsefdn/it/api/git-eca-rest-api/-/merge_requests/153Iss #137 - Remove magic numbers, split out large lambdas for code smell2023-08-24T20:03:18ZMartin Lowemartin.lowe@eclipse-foundation.orgIss #137 - Remove magic numbers, split out large lambdas for code smellResolves #137Resolves #137https://gitlab.eclipse.org/eclipsefdn/it/api/git-eca-rest-api/-/merge_requests/152Update to api-common v0.8, and fix tests that broke from the changes2023-08-23T15:46:57ZMartin Lowemartin.lowe@eclipse-foundation.orgUpdate to api-common v0.8, and fix tests that broke from the changesAdditionally, fix some smell that was associated with user-inputs used
raw in logs.Additionally, fix some smell that was associated with user-inputs used
raw in logs.https://gitlab.eclipse.org/eclipsefdn/it/api/git-eca-rest-api/-/merge_requests/151Iss #115 - Add a reprocessing queue for failed GH webhook calls2023-08-22T15:44:52ZMartin Lowemartin.lowe@eclipse-foundation.orgIss #115 - Add a reprocessing queue for failed GH webhook callsTo resolve an issue where some GH calls are timing out unexpectedly
during validation, a new scheduled task was created, along with some
updates to the DB models.
The new task uses the new revalidation flag on the tracking model to
look...To resolve an issue where some GH calls are timing out unexpectedly
during validation, a new scheduled task was created, along with some
updates to the DB models.
The new task uses the new revalidation flag on the tracking model to
look up requests to see if they need to be revalidated, and rerun
validation on the oldest set entry. When run, the last updated field
will be updated regardless of success of validation. This will allow us
to not get stuck on a single tracking request that is in a bad state
potentially.
The DB model changes add 2 new fields to webhook tracking entries,
allowing for tracking of if a scheduled revalidation needs to be run and
how many times a given webhook validation needed to be rerun. This
should give us both the flexibility to rerun as needed, manually flag
errant entries, as well as capture data on how often this happens.https://gitlab.eclipse.org/eclipsefdn/it/api/git-eca-rest-api/-/merge_requests/150feat: Improve code quality2023-08-16T12:59:48ZZachary Sabourinfeat: Improve code qualityRelates to https://gitlab.eclipse.org/eclipsefdn/it/api/git-eca-rest-api/-/issues/135Relates to https://gitlab.eclipse.org/eclipsefdn/it/api/git-eca-rest-api/-/issues/135https://gitlab.eclipse.org/eclipsefdn/it/api/git-eca-rest-api/-/merge_requests/149feat: Standardize configs2023-08-03T15:54:05ZZachary Sabourinfeat: Standardize configsRelates to https://gitlab.eclipse.org/eclipsefdn/it/api/git-eca-rest-api/-/issues/134
This MR is relying on the latest [common config changes](https://gitlab.eclipse.org/eclipsefdn/it/api/eclipsefdn-api-common/-/issues/61)Relates to https://gitlab.eclipse.org/eclipsefdn/it/api/git-eca-rest-api/-/issues/134
This MR is relying on the latest [common config changes](https://gitlab.eclipse.org/eclipsefdn/it/api/eclipsefdn-api-common/-/issues/61)https://gitlab.eclipse.org/eclipsefdn/it/api/git-eca-rest-api/-/merge_requests/148Update JVM dockerfile to resolve image instability2023-07-27T16:58:55ZMartin Lowemartin.lowe@eclipse-foundation.orgUpdate JVM dockerfile to resolve image instabilityhttps://gitlab.eclipse.org/eclipsefdn/it/api/git-eca-rest-api/-/merge_requests/147feat: Update Gitlab image to pull current version2023-07-27T13:10:31ZZachary Sabourinfeat: Update Gitlab image to pull current versionResolves https://gitlab.eclipse.org/eclipsefdn/it/api/git-eca-rest-api/-/issues/136Resolves https://gitlab.eclipse.org/eclipsefdn/it/api/git-eca-rest-api/-/issues/136https://gitlab.eclipse.org/eclipsefdn/it/api/git-eca-rest-api/-/merge_requests/146Add project PR context when available to the status page2023-06-28T14:24:15ZMartin Lowemartin.lowe@eclipse-foundation.orgAdd project PR context when available to the status pagehttps://gitlab.eclipse.org/eclipsefdn/it/api/git-eca-rest-api/-/merge_requests/145Update common lib to 0.7.6 stable from snapshot2023-06-19T14:57:09ZMartin Lowemartin.lowe@eclipse-foundation.orgUpdate common lib to 0.7.6 stable from snapshothttps://gitlab.eclipse.org/eclipsefdn/it/api/git-eca-rest-api/-/merge_requests/144Iss #120 - Remove some unneeded dependencies from the POM2023-06-19T14:05:10ZMartin Lowemartin.lowe@eclipse-foundation.orgIss #120 - Remove some unneeded dependencies from the POMhttps://gitlab.eclipse.org/eclipsefdn/it/api/git-eca-rest-api/-/merge_requests/143Fix commit author and committers to mark as nullable2023-06-15T20:09:40ZMartin Lowemartin.lowe@eclipse-foundation.orgFix commit author and committers to mark as nullableGithub commits won't always have values associated with authors and
committers, so we should mark them as nullable to prevent serialization
failures.Github commits won't always have values associated with authors and
committers, so we should mark them as nullable to prevent serialization
failures.https://gitlab.eclipse.org/eclipsefdn/it/api/git-eca-rest-api/-/merge_requests/142Fix nullable values for incoming Github commit data2023-06-15T13:38:20ZMartin Lowemartin.lowe@eclipse-foundation.orgFix nullable values for incoming Github commit dataMissing commit data caused cascading failures in the request that should
have resulted in just the specific commit being marked as invalid.Missing commit data caused cascading failures in the request that should
have resulted in just the specific commit being marked as invalid.