Skip to content

Component tracking#776

Merged
crstrn13 merged 1 commit intoKuadrant:mainfrom
crstrn13:component_tracking
Oct 23, 2025
Merged

Component tracking#776
crstrn13 merged 1 commit intoKuadrant:mainfrom
crstrn13:component_tracking

Conversation

@crstrn13
Copy link
Copy Markdown
Contributor

Description

Embedded make report-portal with component-metadata step, where it will collect and store in a json file the sha of all the components under kuadrant-system.

Comment thread scripts/collect-component-metadata Outdated
@azgabur
Copy link
Copy Markdown
Member

azgabur commented Oct 14, 2025

Testsuite already gets some limited info about catalog source url of deployed kuadrant. See:

def pytest_report_header(config):

and
def kuadrant_version():

But even when it was added it was just informational as it can sometimes miss version (when deployed from official OperatorHub or helm). Your proposal tries to get all versions which would be good for tracking in the junit results. Maybe good place to put is is to extend the existing tracking? What do you think?

@crstrn13 crstrn13 self-assigned this Oct 16, 2025
@crstrn13 crstrn13 force-pushed the component_tracking branch 10 times, most recently from 130fbf3 to dd4326f Compare October 16, 2025 14:32
@azgabur
Copy link
Copy Markdown
Member

azgabur commented Oct 17, 2025

I remembered there was an issue for this as well #549
Maybe it can be closed with the merge of this PR if everything from it was addressed.

Comment thread scripts/junit2reportportal Outdated
@crstrn13 crstrn13 force-pushed the component_tracking branch 2 times, most recently from c562e2d to 447e8ff Compare October 23, 2025 12:19
Signed-off-by: Alexander Cristurean <acristur@redhat.com>
Copy link
Copy Markdown
Contributor

@averevki averevki left a comment

Choose a reason for hiding this comment

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

LGTM!

@crstrn13 crstrn13 merged commit 817e841 into Kuadrant:main Oct 23, 2025
3 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