Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Initial tests CI PoC #17

Merged
merged 1 commit into from
Nov 19, 2024
Merged

Initial tests CI PoC #17

merged 1 commit into from
Nov 19, 2024

Conversation

kokesak
Copy link
Contributor

@kokesak kokesak commented Oct 31, 2024

Introduce the tests jobs to the packit CI.

Proposal: At the moment it is split between two jobs. One that will be used for simple sanity, functional tests that will run on container (which is much faster to provision). This is run automatically. The second run will be on the VM which sill be used for more complex e2e testing, this can be triggered by specifying the label. For example:

/packit test --labels e2e

Copy link

Failed to load packit config file:

Cannot parse package config. ValidationError({'jobs': {2: {'packages': defaultdict(<class 'dict'>, {'command-line-assistant': {'value': {'targets': ["Expected 'list[str]' or 'dict[str,dict]', got [{'epel-9-x86_64': None, 'distros': ['RHEL-9-Nightly']}, {'epel-10-x86_64': None, 'distro': ['RHEL-10-Nightly']}] (type <class 'list'>)."]}}})}}})

For more info, please check out the documentation or contact the Packit team. You can also use our CLI command validate-config or our pre-commit hooks for validation of the configuration.

2 similar comments
Copy link

Failed to load packit config file:

Cannot parse package config. ValidationError({'jobs': {2: {'packages': defaultdict(<class 'dict'>, {'command-line-assistant': {'value': {'targets': ["Expected 'list[str]' or 'dict[str,dict]', got [{'epel-9-x86_64': None, 'distros': ['RHEL-9-Nightly']}, {'epel-10-x86_64': None, 'distro': ['RHEL-10-Nightly']}] (type <class 'list'>)."]}}})}}})

For more info, please check out the documentation or contact the Packit team. You can also use our CLI command validate-config or our pre-commit hooks for validation of the configuration.

Copy link

Failed to load packit config file:

Cannot parse package config. ValidationError({'jobs': {2: {'packages': defaultdict(<class 'dict'>, {'command-line-assistant': {'value': {'targets': ["Expected 'list[str]' or 'dict[str,dict]', got [{'epel-9-x86_64': None, 'distros': ['RHEL-9-Nightly']}, {'epel-10-x86_64': None, 'distro': ['RHEL-10-Nightly']}] (type <class 'list'>)."]}}})}}})

For more info, please check out the documentation or contact the Packit team. You can also use our CLI command validate-config or our pre-commit hooks for validation of the configuration.

@kokesak kokesak force-pushed the ci-poc branch 4 times, most recently from 8df90bf to b5ee9c9 Compare October 31, 2024 12:37
@kokesak
Copy link
Contributor Author

kokesak commented Oct 31, 2024

/packit test

@kokesak
Copy link
Contributor Author

kokesak commented Oct 31, 2024

/packit build

@kokesak
Copy link
Contributor Author

kokesak commented Oct 31, 2024

/packit build

.packit.yaml Outdated Show resolved Hide resolved
.packit.yaml Outdated Show resolved Hide resolved
.packit.yaml Show resolved Hide resolved
.packit.yaml Outdated Show resolved Hide resolved
@danmyway danmyway marked this pull request as draft October 31, 2024 14:37
@kokesak kokesak force-pushed the ci-poc branch 5 times, most recently from 998d7bf to f94b4d1 Compare October 31, 2024 15:56
Copy link

Failed to load packit config file:

Cannot parse package config. ValidationError({'jobs': {2: {'packages': defaultdict(<class 'dict'>, {'command-line-assistant': {'value': {'targets': ["Expected 'list[str]' or 'dict[str,dict]', got {'epel-9-x86_64': 'registry.access.redhat.com/ubi9'} (type <class 'dict'>)."]}}})}}})

For more info, please check out the documentation or contact the Packit team. You can also use our CLI command validate-config or our pre-commit hooks for validation of the configuration.

2 similar comments
Copy link

Failed to load packit config file:

Cannot parse package config. ValidationError({'jobs': {2: {'packages': defaultdict(<class 'dict'>, {'command-line-assistant': {'value': {'targets': ["Expected 'list[str]' or 'dict[str,dict]', got {'epel-9-x86_64': 'registry.access.redhat.com/ubi9'} (type <class 'dict'>)."]}}})}}})

For more info, please check out the documentation or contact the Packit team. You can also use our CLI command validate-config or our pre-commit hooks for validation of the configuration.

Copy link

Failed to load packit config file:

Cannot parse package config. ValidationError({'jobs': {2: {'packages': defaultdict(<class 'dict'>, {'command-line-assistant': {'value': {'targets': ["Expected 'list[str]' or 'dict[str,dict]', got {'epel-9-x86_64': 'registry.access.redhat.com/ubi9'} (type <class 'dict'>)."]}}})}}})

For more info, please check out the documentation or contact the Packit team. You can also use our CLI command validate-config or our pre-commit hooks for validation of the configuration.

@kokesak kokesak force-pushed the ci-poc branch 3 times, most recently from bba9cbe to 93499e6 Compare October 31, 2024 16:12
Copy link

Failed to load packit config file:

Cannot parse package config. ValidationError({'jobs': {2: {'packages': defaultdict(<class 'dict'>, {'command-line-assistant': {'value': {'environments': ['Unknown field.']}}})}}})

For more info, please check out the documentation or contact the Packit team. You can also use our CLI command validate-config or our pre-commit hooks for validation of the configuration.

2 similar comments
Copy link

Failed to load packit config file:

Cannot parse package config. ValidationError({'jobs': {2: {'packages': defaultdict(<class 'dict'>, {'command-line-assistant': {'value': {'environments': ['Unknown field.']}}})}}})

For more info, please check out the documentation or contact the Packit team. You can also use our CLI command validate-config or our pre-commit hooks for validation of the configuration.

Copy link

Failed to load packit config file:

Cannot parse package config. ValidationError({'jobs': {2: {'packages': defaultdict(<class 'dict'>, {'command-line-assistant': {'value': {'environments': ['Unknown field.']}}})}}})

For more info, please check out the documentation or contact the Packit team. You can also use our CLI command validate-config or our pre-commit hooks for validation of the configuration.

@kokesak
Copy link
Contributor Author

kokesak commented Nov 4, 2024

/packit test --labels vm

@kokesak
Copy link
Contributor Author

kokesak commented Nov 4, 2024

/packit test --labels vm

@kokesak
Copy link
Contributor Author

kokesak commented Nov 4, 2024

/packit test --labels vm

@kokesak kokesak force-pushed the ci-poc branch 4 times, most recently from d7e9ea9 to 99f0881 Compare November 4, 2024 15:29
Copy link

Failed to load packit config file:

Please correct data and retry.

For more info, please check out the documentation or contact the Packit team. You can also use our CLI command validate-config or our pre-commit hooks for validation of the configuration.

2 similar comments
Copy link

Failed to load packit config file:

Please correct data and retry.

For more info, please check out the documentation or contact the Packit team. You can also use our CLI command validate-config or our pre-commit hooks for validation of the configuration.

Copy link

Failed to load packit config file:

Please correct data and retry.

For more info, please check out the documentation or contact the Packit team. You can also use our CLI command validate-config or our pre-commit hooks for validation of the configuration.

@kokesak
Copy link
Contributor Author

kokesak commented Nov 4, 2024

/packit test --labels e2e

.packit.yaml Outdated Show resolved Hide resolved
@kokesak kokesak force-pushed the ci-poc branch 2 times, most recently from a0272e8 to e4e08f2 Compare November 5, 2024 12:52
@kokesak
Copy link
Contributor Author

kokesak commented Nov 5, 2024

/packit test --labels e2e

@kokesak kokesak force-pushed the ci-poc branch 2 times, most recently from 7792d4c to 09df9e6 Compare November 11, 2024 13:23
@kokesak
Copy link
Contributor Author

kokesak commented Nov 11, 2024

/packit test --labels e2e

@kokesak kokesak marked this pull request as ready for review November 11, 2024 19:47
.packit.yaml Show resolved Hide resolved
.packit.yaml Outdated Show resolved Hide resolved
.packit.yaml Outdated Show resolved Hide resolved
.packit.yaml Outdated Show resolved Hide resolved
Co-authored-by: Daniel Diblik <8378124+danmyway@users.noreply.github.com>
@kokesak
Copy link
Contributor Author

kokesak commented Nov 18, 2024

/packit test

Copy link
Contributor

@danmyway danmyway left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great, thanks!

Copy link
Contributor

@mm0ran mm0ran left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, we'll have the opportunity to iterate on this later.

@kokesak kokesak merged commit c0a5851 into rhel-lightspeed:main Nov 19, 2024
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants