Split build_yaml part 1

[matejak]

Moved rule and platform functionality to separate files.

Only minimal modifications were made to the existing code - the PR is basically only about moving code around and about adjusting imports.

[openshift-ci[bot]]

Skipping CI for Draft Pull Request. If you want CI signal for your change, please convert it to an actual PR. You can still manually trigger a test run with /test all

[pep8speaks]

Hello @matejak! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

• In the file ssg/build_yaml.py:

Line 42:100: E501 line too long (104 > 99 characters) Line 1298:29: W292 no newline at end of file

• In the file ssg/entities/common.py:

Line 53:100: E501 line too long (105 > 99 characters) Line 101:1: W293 blank line contains whitespace Line 245:36: W292 no newline at end of file

• In the file ssg/entities/platform.py:

Line 94:20: W292 no newline at end of file

• In the file ssg/entities/rule.py:

Line 122:100: E501 line too long (105 > 99 characters) Line 130:9: E303 too many blank lines (2) Line 295:100: E501 line too long (108 > 99 characters) Line 298:100: E501 line too long (116 > 99 characters) Line 539:24: W292 no newline at end of file

[codeclimate[bot]]

Code Climate has analyzed commit 0f40caa9 and detected 80 issues on this pull request.

Here's the issue category breakdown:

Category	Count
Complexity	16
Duplication	8
Style	56

Note: there are 2 critical issues.

View more on Code Climate.

[github-actions[bot]]

Start a new ephemeral environment with changes proposed in this pull request:

[openshift-ci[bot]]

@matejak: PR needs rebase.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[jan-cerny]

@matejak there is a risk of getting more conflicts in future if somebody else would modify the same file

[openshift-ci[bot]]

@matejak: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name	Commit	Details	Required	Rerun command
ci/prow/e2e-aws-ocp4-stig-node	0f40caa9a0079ef417c77c65f460783652cdc566	link	true	/test e2e-aws-ocp4-stig-node
ci/prow/e2e-aws-ocp4-stig	0f40caa9a0079ef417c77c65f460783652cdc566	link	true	/test e2e-aws-ocp4-stig

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[jan-cerny]

@matejak any plans?

[matejak]

I will probably return to it after the 0.1.63 upstream release.

[jan-cerny]

@matejak there is a risk of getting more conflicts in future if somebody else would modify the same file

[matejak]

I will start with something less intrusive than moving rules - for instance, profiles are more suitable.