Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-37546

When dynamic plugin load fails, PermissionGroups are not being cleaned up

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Open (View Workflow)
    • Priority: Major
    • Resolution: Unresolved
    • Component/s: core
    • Labels:
    • Environment:
      jenkins-2.16
    • Similar Issues:

      Description

      Noticed it during JENKINS-37545.

      Reproduction steps:
      1) Manually deploy a plugin with detached dependencies and permission groups (e.g. Ownership 0.5)
      2) The loading should fail...
      3) Install missing dependencies
      4) Deploy the plugin again
      5) Change the log

      Expected behavior:
      The plugin installation succeeds

      Actual behavior:
      Plugin installation fails due to the missing permission group

      Workaround:
      Restart the instance

        Attachments

          Issue Links

            Activity

            Hide
            jglick Jesse Glick added a comment -

            Probably should be fixed implicitly by a rewrite of permission registration, as mentioned in JENKINS-17200.

            Show
            jglick Jesse Glick added a comment - Probably should be fixed implicitly by a rewrite of permission registration, as mentioned in JENKINS-17200 .
            Hide
            jvz Matt Sicker added a comment -

            In my current proposal around permissions, this error wouldn't happen as permissions are loaded lazily. Before that, I also had a ConcurrentMap for registering permissions, so even that was using putIfAbsent to avoid duplicating permissions in the UI.

            Show
            jvz Matt Sicker added a comment - In my  current proposal around permissions, this error wouldn't happen as permissions are loaded lazily. Before that, I also had a ConcurrentMap for registering permissions, so even that was using putIfAbsent to avoid duplicating permissions in the UI.

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              oleg_nenashev Oleg Nenashev
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated: