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

Make sure to run odo v3 tests for stacks that have no starter projects #218

Conversation

rm3l
Copy link
Member

@rm3l rm3l commented Oct 6, 2023

What does this PR do?:

/kind bug

This fixes the issue reported in devfile/api#1103, where the odo v3 tests would not run any tests on certain stacks.
The issue has been narrowed down to stacks that have no starter projects in their Devfiles, like udi.

This PR makes sure to test such stacks, by leveraging the odo dev --no-commands command, to check that the Dev Session just starts properly.

Which issue(s) this PR fixes:

Fixes devfile/api#1103

PR acceptance criteria:

  • Contributing guide
    Have you read the devfile registry contributing guide and followed its instructions?
  • Test automation
    Does this repository's tests pass with your changes?
  • Documentation
    Does any documentation need to be updated with your changes?
  • Check Tools Provider
    Have you tested the changes with existing tools, i.e. Odo, Che, Console? (See devfile registry contributing guide on how to test changes)

How to test changes / Special notes to the reviewer:

@openshift-ci
Copy link

openshift-ci bot commented Oct 6, 2023

@rm3l: The label(s) kind/bug cannot be applied, because the repository doesn't have them.

In response to this:

What does this PR do?:

/kind bug

This fixes the issue reported in devfile/api#1103, where the odo v3 tests would not run any tests on certain stacks.
The issue has been narrowed down to stacks that have no starter projects in their Devfiles, like udi

This PR makes sure to test such stacks, by leveraging the odo dev --no-commands command, to check that the Dev Session just starts properly.

Which issue(s) this PR fixes:

Fixes devfile/api#1103

PR acceptance criteria:

  • Contributing guide
    Have you read the devfile registry contributing guide and followed its instructions?
  • Test automation
    Does this repository's tests pass with your changes?
  • Documentation
    Does any documentation need to be updated with your changes?
  • Check Tools Provider
    Have you tested the changes with existing tools, i.e. Odo, Che, Console? (See devfile registry contributing guide on how to test changes)

How to test changes / Special notes to the reviewer:

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.

@openshift-ci openshift-ci bot requested review from elsony and feloy October 6, 2023 12:02
@openshift-ci openshift-ci bot added the approved label Oct 6, 2023
@rm3l rm3l added the bug Something isn't working label Oct 6, 2023
@rm3l
Copy link
Member Author

rm3l commented Oct 6, 2023

PR acceptance criteria:

The tests pass on this branch, but without executing any actual v3 or v2 tests, because this PR did not change any files under the stacks directory.
I think this might be problematic. To me, we should still test all stacks on PRs when there are changes done to certain files like tests/check_odo-v3.sh or tests/odov3, ... But this can be handled in a separate issue.

Meanwhile, I checked this PR in my own fork, by making changes to some stacks (with and without starter projects) and making sure that the odo v3 tests would run against those stacks. See rm3l#2:

rm3l added 2 commits October 6, 2023 15:53
…s defined

This is done via the `odo dev --no-commands` command (introduced in odo v3.12.0 [1]),
to make sure that a Dev Session could actually be started properly.
This is to cover cases for example where the container has a terminating command
or ends up being restarted for some reason, or `odo` is unable to sync files.

[1] https://odo.dev/docs/command-reference/dev#running-with-no-commands

Signed-off-by: Armel Soro <[email protected]>
@rm3l rm3l force-pushed the 1103-investigate-new-registry-stacks-not-executed-in-odo-v3-tests branch from b2e8853 to 15c0a33 Compare October 6, 2023 13:53
Copy link
Member

@michael-valdron michael-valdron left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm Looks good to me label Oct 6, 2023
@openshift-ci
Copy link

openshift-ci bot commented Oct 6, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: michael-valdron, rm3l

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:
  • OWNERS [michael-valdron,rm3l]

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

@rm3l rm3l merged commit 5cbceb7 into devfile:main Oct 9, 2023
6 checks passed
@rm3l rm3l deleted the 1103-investigate-new-registry-stacks-not-executed-in-odo-v3-tests branch October 9, 2023 15:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved bug Something isn't working lgtm Looks good to me
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Investigate new registry stacks not executed in odo v3 tests
2 participants