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

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

    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
                Reporter:
                oleg_nenashev Oleg Nenashev
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated: