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

OCP must gather is not collected #10526

Open
petr-balogh opened this issue Sep 18, 2024 · 2 comments
Open

OCP must gather is not collected #10526

petr-balogh opened this issue Sep 18, 2024 · 2 comments
Assignees
Labels
team/ecosystem Ecosystem team related issues/PRs

Comments

@petr-balogh
Copy link
Member

With change of directory structure, the logic we had implemented here:

for x in [
"ecosystem",
"e2e/performance",
"tests/functional/z_cluster",

To collect logs for OCP based on test path doesn't work anymore.

I propose to use the specific marker for tests which requires OCP must gather. Something like

@collect_ocp_must_gather
def test_case1

So we can decorate those tests which requires this and instead of path, to check if test item has that mark.

Or we can include some other marks like @purple_squad
As I would like to have OCP Must gather for all our purple squad tests collected always.

@petr-balogh petr-balogh added the team/ecosystem Ecosystem team related issues/PRs label Sep 18, 2024
@clacroix12
Copy link
Contributor

@petr-balogh

We are doing things in a similar fashion for ocs and mcg logs collection. I assume we will markers for each of these applied to the appropriate tests?

ocs_logs_collection = (
    False
    if any(x in item.location[0] for x in ["_ui", "must_gather"])
    else True
)
mcg_logs_collection = (
    True if any(x in item.location[0] for x in ["mcg", "ecosystem"]) else False
)

@petr-balogh
Copy link
Member Author

yes for ecosystem we can make like in example above for ocp_logs_collection as well I think if that's the question. For others, we can have new explicit marker. And once it exists - we can ask owners who require OCP MG to be collected when their test will fail, that they will apply that marker over their test case which requires this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team/ecosystem Ecosystem team related issues/PRs
Projects
None yet
Development

No branches or pull requests

2 participants