Skip to content

Commit

Permalink
Update comment redundant data plane adoption 'old' job names
Browse files Browse the repository at this point in the history
We cannot remove the the 'old' data plane adoption jobs after the
job rename [1] because zuul checks all branches (e.g. ci-framework
has multiple branches referencing these jobs). This just updates the
comment to clarify that we will not be removing them.

[1] https://review.rdoproject.org/r/c/rdo-jobs/+/53788

Related: https://issues.redhat.com/browse/OSPRH-8452
  • Loading branch information
marios authored and openshift-merge-bot[bot] committed Jul 29, 2024
1 parent a382a3c commit 0d5f3f5
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions zuul.d/adoption.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -95,7 +95,7 @@
ip: 172.20.0.100
config_nm: false

# TODO(marios): remove this once we finish rename OSPRH-8452
# NOTE(marios): need to keep this old job because of zuul branches, see comments at OSPRH-8452
- job:
name: cifmw-data-plane-adoption-osp-17-to-extracted-crc
parent: adoption-standalone-to-crc-ceph-provider
Expand Down Expand Up @@ -166,7 +166,7 @@
- tests?\/functional
- zuul.d/molecule.*

# TODO(marios): remove this once we finish rename OSPRH-8452
# NOTE(marios): need to keep this old job because of zuul branches, see comments at OSPRH-8452
- job:
name: cifmw-data-plane-adoption-osp-17-to-extracted-crc-minimal-no-ceph
parent: adoption-standalone-to-crc-no-ceph-provider
Expand Down

0 comments on commit 0d5f3f5

Please sign in to comment.