RFR: Backport security level should be inherited from main issue

Kevin Rushforth kcr at openjdk.java.net
Thu Apr 8 21:00:41 UTC 2021


On Thu, 8 Apr 2021 20:30:39 GMT, Erik Joelsson <erikj at openjdk.org> wrote:

> This is a change Robin prepared, but we need to deploy it ASAP to match the new behavior in the backport plugin after the Jira upgrade. The security level of a backport should simply be based on the security level of the primary bug.

Looks good to me (although I am not a Skara project Reviewer).

bots/notify/src/test/java/org/openjdk/skara/bots/notify/issue/IssueNotifierTests.java line 1069:

> 1067:             if (level != null) {
> 1068:                 assertEquals(level.asString(), backport.properties().get("security").asString());
> 1069:             }

Maybe also worth also checking that if level is null, the returned property is empty? This could be done as a follow-up given the need to get this in right away.

-------------

Marked as reviewed by kcr (no project role).

PR: https://git.openjdk.java.net/skara/pull/1110


More information about the skara-dev mailing list