PR#64: [PVHEALTH-132] Assert required vars at start of role - - Pagure.io

Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.

c3i-library

#64 [PVHEALTH-132] Assert required vars at start of role

Merged Apr 22, 2020 by mkovarik. Opened Apr 20, 2020 by sidpremkumar.

sidpremkumar/c3i-library assert-req-vars into master

Assert required vars at start of role

Sidhartha Premkumar • Apr 22, 2020

231fc8303c25da35dea851565bb1889d0422b590

Download 64.patch

sidpremkumar commented Apr 20, 2020

Assert that required variables are set at the start of the role

sidpremkumar commented Apr 20, 2020

@mkovarik Could I get a review on this

mkovarik commented Apr 21, 2020

c3i_component is in defaults so it is always defined, you should comment it in defaults.

mkovarik commented Apr 21, 2020

Also c3i_build_and_test_snippet and c3i_integration_test_snippet are required

rebased onto 9df0b84fe8bc34d9c6a75b1639aa4c96130e9aef

Apr 22, 2020

rebased onto 231fc8303c25da35dea851565bb1889d0422b590

Apr 22, 2020

Pull-Request has been merged by mkovarik

Apr 22, 2020

Metadata

Assignee

None

Tags

No Tags

Changes Summary 2

file changed

roles/c3i/defaults/main.yml

file changed

roles/c3i/tasks/main.yml

Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.