Skip to content

Conversation

PatAKnight
Copy link
Member

Description

Adds tests for the recent introduction of the backstage.io/template-version annotation. Templates are now able to enhance entities with versioning information.

Also, renames the catalog-scaffolded-from-Link tests to scaffolder-backend-module-annotator to better reflect the full test suite. The tests are no longer focused solely on the link and are now focus on the ability to label, annotate, and version software templates.

I also updated the scaffolder template link from the Janus QE repo to the community plugins repo

Finally, fixes an issue with referencing the wrong file name for the original catalog scaffolded from link tests.

Which issue(s) does this PR fix

PR acceptance criteria

Please make sure that the following steps are complete:

  • GitHub Actions are completed and successful
  • Unit Tests are updated and passing
  • E2E Tests are updated and passing
  • Documentation is updated if necessary (requirement for new features)
  • Add a screenshot if the change is UX/UI related

How to test changes / Special notes to the reviewer

Copy link
Contributor

@PatAKnight PatAKnight force-pushed the e2e-template-version branch 2 times, most recently from 804b4ff to 40b17ff Compare July 24, 2025 14:17
Copy link
Contributor

@PatAKnight
Copy link
Member Author

/retest

@PatAKnight PatAKnight force-pushed the e2e-template-version branch from 40b17ff to 21bad9f Compare July 25, 2025 01:26
@PatAKnight
Copy link
Member Author

/retest

@PatAKnight PatAKnight force-pushed the e2e-template-version branch 2 times, most recently from 826e4b7 to f17fb6d Compare July 25, 2025 05:02
@PatAKnight
Copy link
Member Author

/retest

3 similar comments
@PatAKnight
Copy link
Member Author

/retest

@PatAKnight
Copy link
Member Author

/retest

@PatAKnight
Copy link
Member Author

/retest

Copy link
Member

@dzemanov dzemanov left a comment

Choose a reason for hiding this comment

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

Thanks @PatAKnight, looks great.
The annotator actions are passing on the e2e runs:
image

/lgtm
/approve

@openshift-ci openshift-ci bot added the lgtm label Jul 28, 2025
@PatAKnight PatAKnight force-pushed the e2e-template-version branch from f17fb6d to 42b76e0 Compare July 29, 2025 12:26
@openshift-ci openshift-ci bot removed the lgtm label Jul 29, 2025
Copy link
Contributor

@PatAKnight
Copy link
Member Author

/retest

@PatAKnight PatAKnight force-pushed the e2e-template-version branch from 42b76e0 to 1b121e5 Compare July 29, 2025 15:30
@PatAKnight
Copy link
Member Author

/cherry-pick release-1.7

@openshift-cherrypick-robot
Copy link
Contributor

@PatAKnight: once the present PR merges, I will cherry-pick it on top of release-1.7 in a new PR and assign it to you.

In response to this:

/cherry-pick release-1.7

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-sigs/prow repository.

@PatAKnight
Copy link
Member Author

/retest

3 similar comments
@PatAKnight
Copy link
Member Author

/retest

@PatAKnight
Copy link
Member Author

/retest

@PatAKnight
Copy link
Member Author

/retest

@openshift-ci openshift-ci bot added the lgtm label Jul 29, 2025
Copy link

openshift-ci bot commented Jul 29, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: AndrienkoAleksandr, dzemanov, gustavolira

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@PatAKnight
Copy link
Member Author

/retest

@PatAKnight PatAKnight force-pushed the e2e-template-version branch from 1b121e5 to 6358ca4 Compare July 30, 2025 01:40
@openshift-ci openshift-ci bot removed the lgtm label Jul 30, 2025
Copy link

openshift-ci bot commented Jul 30, 2025

New changes are detected. LGTM label has been removed.

Copy link
Contributor

@PatAKnight
Copy link
Member Author

/retest

@PatAKnight PatAKnight force-pushed the e2e-template-version branch from 6358ca4 to 40bf15b Compare July 30, 2025 14:49
Copy link
Contributor

@PatAKnight PatAKnight force-pushed the e2e-template-version branch from 40bf15b to 5765deb Compare August 4, 2025 02:50
Copy link
Contributor

github-actions bot commented Aug 4, 2025

@PatAKnight
Copy link
Member Author

/retest

2 similar comments
@PatAKnight
Copy link
Member Author

/retest

@PatAKnight
Copy link
Member Author

/retest

@dzemanov dzemanov force-pushed the e2e-template-version branch from 5765deb to 4d05b3c Compare August 18, 2025 15:54
Copy link
Contributor

@lholmquist
Copy link
Member

/retest

@openshift-merge-bot openshift-merge-bot bot merged commit c0a8a45 into redhat-developer:main Aug 19, 2025
13 checks passed
@openshift-cherrypick-robot
Copy link
Contributor

@PatAKnight: new pull request created: #3301

In response to this:

/cherry-pick release-1.7

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-sigs/prow repository.

guyoron1 pushed a commit to guyoron1/rhdh that referenced this pull request Sep 29, 2025
…veloper#3189)

* chore(e2e): add e2e tests for software template versioning

Signed-off-by: Patrick <[email protected]>

* chore(e2e): fix issue in referencing the template instead of the component

Signed-off-by: Patrick <[email protected]>

* chore(e2e): switch the order of the tests

Signed-off-by: Patrick <[email protected]>

* chore(e2e): fix failing adoption insights test

Signed-off-by: Patrick <[email protected]>

---------

Signed-off-by: Patrick <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants