Fix manual runs of CI integrations with multiple configurations.

Review Request #12836 — Created Feb. 21, 2023 and submitted

Information

rbintegrations
release-3.x

Reviewers

When a review request has multiple configurations for the same
integration, manual runs behave incorrectly. They check for a matching
configuration, and may get a configuration that doesn't actually match
the one triggering the run. In this case, some other configuration may
end up running.

Review Board 5.0.3 adds support for tracking configurations along with
StatusUpdates and passing them to the signal handler for manual runs.
This change takes advantage of that to use the stored configuration.

Since prior versions of 5.0.x can upgrade to this release of
rbintegrations, this change also backports some of the logic, manually
storing the configuration association and looking it up if not provided
to the signal handler.

Unit tests pass.

Manually tested conditions with and without the new logic in both
Review Board and in rbintegrations.

This change will be tested in production by a customer affected by this
issue.

Summary ID
Fix manual runs of CI integrations with multiple configurations.
When a review request has multiple configurations for the same integration, manual runs behave incorrectly. They check for a matching configuration, and may get a configuration that doesn't actually match the one triggering the run. In this case, some other configuration may end up running. Review Board 5.0.3 adds support for tracking configurations along with `StatusUpdate`s and passing them to the signal handler for manual runs. This change takes advantage of that to use the stored configuration. Since prior versions of 5.0.x can upgrade to this release of rbintegrations, this change also backports some of the logic, manually storing the configuration association and looking it up if not provided to the signal handler.
9415bd106f745e101989b115ebfa8955d390d01a
david
  1. Ship It!
  2. 
      
chipx86
Review request changed
Status:
Completed
Change Summary:
Pushed to release-3.x (05e82cc)