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

Invert pipeline-model-definition → docker-workflow dependency

    • docker-workflow 1.22; pipeline-model-definition 1.6.0

      Currently pipeline-model-definition has a dependency on docker-workflow to supply agent docker and agent dockerfile definitions. These should rather be defined in the docker-workflow plugin, which would then get a dependency on pipeline-model-extensions (unclear if that should be optional).

      This would match, for example, how agent kubernetes is defined in the kubernetes plugin, and how I propose (related to JENKINS-48050) something like agent dockerNode should be defined in the docker plugin, keeping the sensitive aspects of the behavior close to the plugin which manages that environment.

      It would also allow docker-workflow to be omitted from distributions which do not need it. In particular, Kubernetes-oriented distributions are unlikely to have any use for agent docker. (You might want to install docker-workflow so as to be able to use withDockerContainer with DinD, but in that case you would still be using agent kubernetes, and anyway the most common use case—building Docker images—is preferably handled via Kaniko.)

          [JENKINS-58732] Invert pipeline-model-definition → docker-workflow dependency

          Jesse Glick created issue -
          Jesse Glick made changes -
          Link New: This issue relates to JENKINS-48050 [ JENKINS-48050 ]
          Jesse Glick made changes -
          Remote Link New: This issue links to "CloudBees-internal issue (Web Link)" [ 23310 ]
          Jesse Glick made changes -
          Link New: This issue relates to JENKINS-49651 [ JENKINS-49651 ]
          Jesse Glick made changes -
          Assignee Original: Andrew Bayer [ abayer ] New: Jesse Glick [ jglick ]
          Jesse Glick made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          Jesse Glick made changes -
          Remote Link New: This issue links to "pipeline-model-definition #372 (Web Link)" [ 24347 ]
          Jesse Glick made changes -
          Remote Link New: This issue links to "pipeline-model-definition #373 (Web Link)" [ 24348 ]
          Jesse Glick made changes -
          Remote Link New: This issue links to "docker-workflow #202 (Web Link)" [ 24349 ]
          Jesse Glick made changes -
          Status Original: In Progress [ 3 ] New: In Review [ 10005 ]
          Jesse Glick made changes -
          Released As New: docker-workflow 1.22; pipeline-model-definition 1.5.2 (?)
          Resolution New: Fixed [ 1 ]
          Status Original: In Review [ 10005 ] New: Fixed but Unreleased [ 10203 ]

            jglick Jesse Glick
            jglick Jesse Glick
            Votes:
            1 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: