• Icon: Bug Bug
    • Resolution: Unresolved
    • Icon: Minor Minor
    • None
    • Jenkins 2.19.4, fresh Pipeline installation as of 2016-12-08

      When configuring a Shared Library, after selecting "Legacy SCM" or "Modern SCM" the SCM options are listed with a radio selection UI element.

      Unfortunately there is no item in that radio selection that is selected by default, meaning if the user doesn't notice nothing has been actually selected in the radio selection, might Save the configuration and be very confused with their Shared Libraries don't work.

          [JENKINS-40323] No SCM selected when adding a shared library

          James Dumay added a comment -

          hrmpw we should revisit this and give it some design love.

          James Dumay added a comment - hrmpw we should revisit this and give it some design love.

          Patrick Wolf added a comment -

          Is this true for modern SCM, too?

          Should error on saving if they don't select something. I wouldn't want to choose a default for users. There's too much configuration needed to have a real default.

          Patrick Wolf added a comment - Is this true for modern SCM, too? Should error on saving if they don't select something. I wouldn't want to choose a default for users. There's too much configuration needed to have a real default.

          James Dumay added a comment -

          What is "modern scm" vs "legacy scm"?

          James Dumay added a comment - What is "modern scm" vs "legacy scm"?

          R. Tyler Croy added a comment -

          Yes the issue exists when selecting either "SCM" option. And unfortunately the configuration page will happily save (but not persist any configuration of course) without a validation error.

          What is "modern scm" vs "legacy scm"?

          This is some implementation detail about whether a full clone is needed to get the library code or not. Something like that, has nothing to do with the SCM and everything with the SCM plugin from my recollection

          R. Tyler Croy added a comment - Yes the issue exists when selecting either "SCM" option. And unfortunately the configuration page will happily save (but not persist any configuration of course) without a validation error. What is "modern scm" vs "legacy scm"? This is some implementation detail about whether a full clone is needed to get the library code or not. Something like that, has nothing to do with the SCM and everything with the SCM plugin from my recollection

          Patrick Wolf added a comment -

          It's based on usage the of scm-api-plugin as far as I know but I'm not 100% sure how it is related. jglick or stephenconnolly know more about it than anyone.

          I think we need a more descriptive term for them but I'm not sure what we could call them.

          Patrick Wolf added a comment - It's based on usage the of scm-api-plugin as far as I know but I'm not 100% sure how it is related. jglick or stephenconnolly know more about it than anyone. I think we need a more descriptive term for them but I'm not sure what we could call them.

          Jesse Glick added a comment -

          No good fix without JENKINS-19584.

          Jesse Glick added a comment - No good fix without JENKINS-19584 .

          Jesse Glick added a comment -

          FWIW “legacy” vs. “modern” simply refers to whether the SCM plugin is implementing the recommended APIs, or whether we need to fall back to a workaround compatible with older SCM plugins.

          Jesse Glick added a comment - FWIW “legacy” vs. “modern” simply refers to whether the SCM plugin is implementing the recommended APIs, or whether we need to fall back to a workaround compatible with older SCM plugins.

          James Dumay added a comment -

          jglick whats the difference as far as the users concerned? Any behavioural difference or is it just a implementation detail?

          James Dumay added a comment - jglick whats the difference as far as the users concerned? Any behavioural difference or is it just a implementation detail?

          Jesse Glick added a comment -

          jamesdumay yes, legacy means

          • you must use the ${library.NAME.version} hack
          • there is unlikely to be support for @abc1234 and generally selecting anything other than a branch head

          Jesse Glick added a comment - jamesdumay yes, legacy means you must use the ${library.NAME.version } hack there is unlikely to be support for @abc1234 and generally selecting anything other than a branch head

            cloudbees CloudBees Inc.
            rtyler R. Tyler Croy
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated: