Uploaded image for project: 'OpenShift Pipelines'
  1. OpenShift Pipelines
  2. SRVKP-4057

Support to set priorityClassName and other configs for affinity-assistant pods using affinity-assistant-pod-template

XMLWordPrintable

    • Support to set priorityClassName and other configs for affinity-assistant pods using affinity-assistant-pod-template
    • False
    • None
    • False
    • To Do

      Description of problem:

      While trying to set  priorityClassName for affinity-assistant pods using the field `default-affinity-assistant-pod-template`, the field is not added to the pod yaml.

      Prerequisites (if any, like setup, operators/versions):

      Steps to Reproduce

       Edited TektonConfig CR to include below field:

      pipeline:
          await-sidecar-readiness: true
          default-affinity-assistant-pod-template: |
            nodeSelector: <-------------
              test: pipelines
            priorityClassName: high-priority <------------
          default-service-account: pipeline
          disable-affinity-assistant: false 

       

      Actual results:

      affinity-assistant pod yaml picked up `nodeSelector` but not the `priorityClassName`:

      spec:
        nodeSelector: <-------------
          test: pipelines
        restartPolicy: Always
        serviceAccountName: default
        imagePullSecrets:
          - name: default-dockercfg-kxdr2
        priority: 0 <------------------------
        schedulerName: default-scheduler
        enableServiceLinks: true 

       

      Expected results:

      priorityClassName should get merged to the affinity-assistant pod yaml.

      Reproducibility (Always/Intermittent/Only Once):

      Acceptance criteria: 

       

      Definition of Done:

      Build Details:

      Additional info (Such as Logs, Screenshots, etc):

       

       *

              ppunamiy Puneet Punamiya
              rhn-support-jyarora Jyotsana Arora
              Puneet Punamiya Puneet Punamiya
              Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

                Created:
                Updated: