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 committed Jul 29, 2024
1 parent 1772ae6 commit b5f7ddd
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 b5f7ddd

Please sign in to comment.