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

Allow users to delete builds even if they are supposed to be kept

    • Icon: Improvement Improvement
    • Resolution: Fixed
    • Icon: Minor Minor
    • core
    • None

      Any Run with isKeepLog==true can not be deleted from UI. Allow such deletions but user must be made aware and explicitly confirm.

      Originally suggested by vjuranek https://github.com/jenkinsci/jenkins/pull/552#issuecomment-8220136.

          [JENKINS-26281] Allow users to delete builds even if they are supposed to be kept

          Oliver Gondža created issue -
          Oliver Gondža made changes -
          Description Original: Any {{Run}} with {{isKeepLog == true}} can not be deleted from UI. Allow such deletions but user must be made aware and explicitly confirm.

          Originally suggested by vjuranek https://github.com/jenkinsci/jenkins/pull/552#issuecomment-8220136.
          New: Any {{Run}} with {{isKeepLog==true}} can not be deleted from UI. Allow such deletions but user must be made aware and explicitly confirm.

          Originally suggested by vjuranek https://github.com/jenkinsci/jenkins/pull/552#issuecomment-8220136.
          Oliver Gondža made changes -
          Issue Type Original: Bug [ 1 ] New: Improvement [ 4 ]
          Oliver Gondža made changes -
          Remote Link New: This issue links to "PR 1511 (Web Link)" [ 12000 ]

          Code changed in jenkins
          User: Oliver Gondža
          Path:
          core/src/main/resources/hudson/model/Run/confirmDelete.jelly
          core/src/main/resources/hudson/model/Run/confirmDelete.properties
          core/src/main/resources/hudson/model/Run/confirmDelete_da.properties
          core/src/main/resources/hudson/model/Run/confirmDelete_de.properties
          core/src/main/resources/hudson/model/Run/confirmDelete_es.properties
          core/src/main/resources/hudson/model/Run/confirmDelete_fr.properties
          core/src/main/resources/hudson/model/Run/confirmDelete_ja.properties
          core/src/main/resources/hudson/model/Run/confirmDelete_nl.properties
          core/src/main/resources/hudson/model/Run/confirmDelete_pt_BR.properties
          core/src/main/resources/hudson/model/Run/confirmDelete_ru.properties
          core/src/main/resources/hudson/model/Run/confirmDelete_tr.properties
          core/src/main/resources/hudson/model/Run/confirmDelete_zh_TW.properties
          http://jenkins-ci.org/commit/jenkins/818ae76af82e8cb3f25cfbb83d42812c80809bab
          Log:
          [FIXED JENKINS-26281] Allow users to delete builds even if they are supposed to be kept

          This expects that any mechanism that keeps builds handles their disappearance gracefully.

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oliver Gondža Path: core/src/main/resources/hudson/model/Run/confirmDelete.jelly core/src/main/resources/hudson/model/Run/confirmDelete.properties core/src/main/resources/hudson/model/Run/confirmDelete_da.properties core/src/main/resources/hudson/model/Run/confirmDelete_de.properties core/src/main/resources/hudson/model/Run/confirmDelete_es.properties core/src/main/resources/hudson/model/Run/confirmDelete_fr.properties core/src/main/resources/hudson/model/Run/confirmDelete_ja.properties core/src/main/resources/hudson/model/Run/confirmDelete_nl.properties core/src/main/resources/hudson/model/Run/confirmDelete_pt_BR.properties core/src/main/resources/hudson/model/Run/confirmDelete_ru.properties core/src/main/resources/hudson/model/Run/confirmDelete_tr.properties core/src/main/resources/hudson/model/Run/confirmDelete_zh_TW.properties http://jenkins-ci.org/commit/jenkins/818ae76af82e8cb3f25cfbb83d42812c80809bab Log: [FIXED JENKINS-26281] Allow users to delete builds even if they are supposed to be kept This expects that any mechanism that keeps builds handles their disappearance gracefully.
          SCM/JIRA link daemon made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: Open [ 1 ] New: Resolved [ 5 ]

          Code changed in jenkins
          User: Oliver Gondža
          Path:
          core/src/main/resources/hudson/model/Run/confirmDelete.jelly
          core/src/main/resources/hudson/model/Run/confirmDelete.properties
          core/src/main/resources/hudson/model/Run/confirmDelete_da.properties
          core/src/main/resources/hudson/model/Run/confirmDelete_de.properties
          core/src/main/resources/hudson/model/Run/confirmDelete_es.properties
          core/src/main/resources/hudson/model/Run/confirmDelete_fr.properties
          core/src/main/resources/hudson/model/Run/confirmDelete_ja.properties
          core/src/main/resources/hudson/model/Run/confirmDelete_nl.properties
          core/src/main/resources/hudson/model/Run/confirmDelete_pt_BR.properties
          core/src/main/resources/hudson/model/Run/confirmDelete_ru.properties
          core/src/main/resources/hudson/model/Run/confirmDelete_tr.properties
          core/src/main/resources/hudson/model/Run/confirmDelete_zh_TW.properties
          http://jenkins-ci.org/commit/jenkins/50f940009b3eda003ff59ecdb21816b00be19339
          Log:
          Merge pull request #1511 from olivergondza/override-whyKeepLog

          JENKINS-26281 Allow users to delete builds even if they are supposed to be kept

          SCM/JIRA link daemon added a comment - Code changed in jenkins User: Oliver Gondža Path: core/src/main/resources/hudson/model/Run/confirmDelete.jelly core/src/main/resources/hudson/model/Run/confirmDelete.properties core/src/main/resources/hudson/model/Run/confirmDelete_da.properties core/src/main/resources/hudson/model/Run/confirmDelete_de.properties core/src/main/resources/hudson/model/Run/confirmDelete_es.properties core/src/main/resources/hudson/model/Run/confirmDelete_fr.properties core/src/main/resources/hudson/model/Run/confirmDelete_ja.properties core/src/main/resources/hudson/model/Run/confirmDelete_nl.properties core/src/main/resources/hudson/model/Run/confirmDelete_pt_BR.properties core/src/main/resources/hudson/model/Run/confirmDelete_ru.properties core/src/main/resources/hudson/model/Run/confirmDelete_tr.properties core/src/main/resources/hudson/model/Run/confirmDelete_zh_TW.properties http://jenkins-ci.org/commit/jenkins/50f940009b3eda003ff59ecdb21816b00be19339 Log: Merge pull request #1511 from olivergondza/override-whyKeepLog JENKINS-26281 Allow users to delete builds even if they are supposed to be kept

          dogfood added a comment -

          Integrated in jenkins_main_trunk #3925
          [FIXED JENKINS-26281] Allow users to delete builds even if they are supposed to be kept (Revision 818ae76af82e8cb3f25cfbb83d42812c80809bab)

          Result = SUCCESS
          ogondza : 818ae76af82e8cb3f25cfbb83d42812c80809bab
          Files :

          • core/src/main/resources/hudson/model/Run/confirmDelete_tr.properties
          • core/src/main/resources/hudson/model/Run/confirmDelete_ru.properties
          • core/src/main/resources/hudson/model/Run/confirmDelete_pt_BR.properties
          • core/src/main/resources/hudson/model/Run/confirmDelete_es.properties
          • core/src/main/resources/hudson/model/Run/confirmDelete.properties
          • core/src/main/resources/hudson/model/Run/confirmDelete_ja.properties
          • core/src/main/resources/hudson/model/Run/confirmDelete_fr.properties
          • core/src/main/resources/hudson/model/Run/confirmDelete.jelly
          • core/src/main/resources/hudson/model/Run/confirmDelete_zh_TW.properties
          • core/src/main/resources/hudson/model/Run/confirmDelete_da.properties
          • core/src/main/resources/hudson/model/Run/confirmDelete_nl.properties
          • core/src/main/resources/hudson/model/Run/confirmDelete_de.properties

          dogfood added a comment - Integrated in jenkins_main_trunk #3925 [FIXED JENKINS-26281] Allow users to delete builds even if they are supposed to be kept (Revision 818ae76af82e8cb3f25cfbb83d42812c80809bab) Result = SUCCESS ogondza : 818ae76af82e8cb3f25cfbb83d42812c80809bab Files : core/src/main/resources/hudson/model/Run/confirmDelete_tr.properties core/src/main/resources/hudson/model/Run/confirmDelete_ru.properties core/src/main/resources/hudson/model/Run/confirmDelete_pt_BR.properties core/src/main/resources/hudson/model/Run/confirmDelete_es.properties core/src/main/resources/hudson/model/Run/confirmDelete.properties core/src/main/resources/hudson/model/Run/confirmDelete_ja.properties core/src/main/resources/hudson/model/Run/confirmDelete_fr.properties core/src/main/resources/hudson/model/Run/confirmDelete.jelly core/src/main/resources/hudson/model/Run/confirmDelete_zh_TW.properties core/src/main/resources/hudson/model/Run/confirmDelete_da.properties core/src/main/resources/hudson/model/Run/confirmDelete_nl.properties core/src/main/resources/hudson/model/Run/confirmDelete_de.properties
          R. Tyler Croy made changes -
          Workflow Original: JNJira [ 160342 ] New: JNJira + In-Review [ 196391 ]

          Additional PR with further fixes send [1]. If applicable a warning about dependant builds is now displayed when user is prompted to confirm the delete action. However this warning does not prevent the deletion anymore and allows users to proceed by confirming the action.

          [1] https://github.com/jenkinsci/matrix-project-plugin/pull/39

          Jakub Cechacek added a comment - Additional PR with further fixes send [1] . If applicable a warning about dependant builds is now displayed when user is prompted to confirm the delete action. However this warning does not prevent the deletion anymore and allows users to proceed by confirming the action. [1] https://github.com/jenkinsci/matrix-project-plugin/pull/39

            olivergondza Oliver Gondža
            olivergondza Oliver Gondža
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: