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

Creation/Remove view regression

    XMLWordPrintable

Details

    • Bug
    • Status: Closed (View Workflow)
    • Major
    • Resolution: Fixed
    • clearcase-plugin
    • None
    • Platform: all OS: all

    Description

      With the latest Clearcase plugin version in the trunk (0.8.2-SNAPSHOT - 03/01)
      "use update checkox" is checked but the snapshot view is removed and then
      created each time.

      Attachments

        Activity

          Code changed in hudson
          User: : sunfire
          Path:
          trunk/hudson/plugins/clearcase/src/main/java/hudson/plugins/clearcase/action/UcmSnapshotCheckoutAction.java
          http://fisheye4.cenqua.com/changelog/hudson/?cs=16717
          Log:
          JENKINS-3184

          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in hudson User: : sunfire Path: trunk/hudson/plugins/clearcase/src/main/java/hudson/plugins/clearcase/action/UcmSnapshotCheckoutAction.java http://fisheye4.cenqua.com/changelog/hudson/?cs=16717 Log: JENKINS-3184
          sunfire sunfire added a comment -

          Please test this....

          sunfire sunfire added a comment - Please test this....
          abayer Andrew Bayer added a comment -

          This is showing up with non-UCM snapshots as well, if the job's configured
          config spec uses different OS slashes than the OS the node is running. i.e., if
          I use Unix slashes in the config spec for the job and the node is Windows, the
          config spec's slashes get translated to Windows before setcs is called, but when
          comparing config specs to determine whether to rmview, we're comparing the live
          config spec against the untranslated config spec, so we're guaranteed it won't
          match.

          I'm fixing this now in SnapshotCheckoutAction and DynamicCheckoutAction by
          moving the creation of the translated string tempConfigSpec (which is currently
          used for setting the config spec) to the top of checkout(), and using
          tempConfigSpec in place of configSpec in the comparison.

          abayer Andrew Bayer added a comment - This is showing up with non-UCM snapshots as well, if the job's configured config spec uses different OS slashes than the OS the node is running. i.e., if I use Unix slashes in the config spec for the job and the node is Windows, the config spec's slashes get translated to Windows before setcs is called, but when comparing config specs to determine whether to rmview, we're comparing the live config spec against the untranslated config spec, so we're guaranteed it won't match. I'm fixing this now in SnapshotCheckoutAction and DynamicCheckoutAction by moving the creation of the translated string tempConfigSpec (which is currently used for setting the config spec) to the top of checkout(), and using tempConfigSpec in place of configSpec in the comparison.
          abayer Andrew Bayer added a comment -

          Marking as verified - for some reason, my changeset
          (http://fisheye4.atlassian.com/changelog/hudson/trunk/hudson/plugins/clearcase?cs=16940)
          didn't show up here automatically. Anyway, fixed in 0.8.2.

          abayer Andrew Bayer added a comment - Marking as verified - for some reason, my changeset ( http://fisheye4.atlassian.com/changelog/hudson/trunk/hudson/plugins/clearcase?cs=16940 ) didn't show up here automatically. Anyway, fixed in 0.8.2.
          abayer Andrew Bayer added a comment -

          Re-fixed for UCM views - we're still intending to rewrite the config spec/load
          rule comparison logic after 0.8.2, though.

          abayer Andrew Bayer added a comment - Re-fixed for UCM views - we're still intending to rewrite the config spec/load rule comparison logic after 0.8.2, though.

          People

            sunfire sunfire
            gbois Gregory Boissinot
            Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: