removing deprecated preflight env in favor of PFLT_CERTIFICATION_COMPONENT_ID

[acornett21]

…

SUMMARY

Updating preflight tasks to use non-deprecated env variable, since this will be removed in an upcoming release.

• Relates: removing the deprecated certification-project-id flag redhat-openshift-ecosystem/openshift-preflight#1318

ISSUE TYPE

• Bug, Docs Fix or other nominal change
• New or Enhanced Feature

Tests

• TestBos2Workload: preflight-green - https://www.distributed-ci.io/jobs/914595a8-d763-4d7a-8ab8-2d3576fd1385

---

Test-hints: no-check

[dcibot]

from change #810:

• no check (not a code change)

[coderabbitai]

📝 Walkthrough

Walkthrough

Renamed environment variable PFLT_CERTIFICATION_PROJECT_ID to PFLT_CERTIFICATION_COMPONENT_ID in two preflight container task files; no other logic, control flow, or behavior changes.

Changes

Cohort / File(s)	Summary
Preflight environment variable rename roles/preflight/tasks/test_preflight_check_container_one_image.yml, roles/preflight/tasks/test_preflight_container_parallel.yml	Replaced environment variable name PFLT_CERTIFICATION_PROJECT_ID with PFLT_CERTIFICATION_COMPONENT_ID in preflight container task env blocks (affects container run conditionals). No other edits.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

• Homogeneous variable rename across two files; minimal verification required.

Suggested reviewers

• ramperher
• tonyskapunk
• manurodriguez

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	You can run @coderabbitai generate docstrings to improve docstring coverage.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title Check	✅ Passed	The PR title "removing deprecated preflight env in favor of PFLT_CERTIFICATION_COMPONENT_ID" clearly and specifically describes the main change in the changeset. Both modified files involve renaming the environment variable from PFLT_CERTIFICATION_PROJECT_ID to PFLT_CERTIFICATION_COMPONENT_ID, which is exactly what the title conveys. The title is concise, avoids vague language, and accurately captures the primary objective of the pull request.
Description Check	✅ Passed	The PR description directly relates to the changeset by explaining that preflight tasks are being updated to use the non-deprecated environment variable because the old one will be removed. It provides helpful context by referencing the related upstream issue and mentioning successful test results. The description meaningfully communicates the purpose and rationale behind the changes without being off-topic or overly vague.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b51dc15 and b8dd9c9.

📒 Files selected for processing (2)

• roles/preflight/tasks/test_preflight_check_container_one_image.yml (1 hunks)
• roles/preflight/tasks/test_preflight_container_parallel.yml (1 hunks)

🚧 Files skipped from review as they are similar to previous changes (2)

• roles/preflight/tasks/test_preflight_container_parallel.yml
• roles/preflight/tasks/test_preflight_check_container_one_image.yml

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)

• GitHub Check: dci/check
• GitHub Check: Ansible-lint Check
• GitHub Check: Sanity Check (stable-2.18)
• GitHub Check: Sanity Check (stable-2.9)

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[softwarefactory-project-zuul]

Build succeeded.
https://softwarefactory-project.io/zuul/t/local/buildset/5b2acd4c82f04deca47c196d7e756029

✔️ dci-rpm-build-el8 SUCCESS in 2m 51s
✔️ dci-rpm-build-el9 SUCCESS in 3m 11s

[tonyskapunk]

retest

[dcibot]

from change #810:

• SUCCESS https://www.distributed-ci.io/jobs/914595a8-d763-4d7a-8ab8-2d3576fd1385/jobStates

[tonyskapunk]

Thanks @acornett21
LGTM

[dcibot]

from change #810:

• no check (not a code change)

[softwarefactory-project-zuul]

Build succeeded.
https://softwarefactory-project.io/zuul/t/local/buildset/03f7bbcfd77b4a9294297c07c58aa224

✔️ dci-rpm-build-el8 SUCCESS in 2m 55s
✔️ dci-rpm-build-el9 SUCCESS in 3m 02s

[dcibot]

from change redhat-openshift-ecosystem/openshift-preflight#1318:

• SUCCESS https://www.distributed-ci.io/jobs/5b1d4ff6-1805-4ded-86c4-6a0e677d6555/jobStates

[dcibot]

from change redhat-openshift-ecosystem/openshift-preflight#1318:

• SUCCESS https://www.distributed-ci.io/jobs/21351828-f8ee-4057-9a01-537b290b2088/jobStates