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

Adds debug-command flag to odo watch #4381

Merged
merged 2 commits into from
Feb 4, 2021

Conversation

mik-dass
Copy link
Contributor

What type of PR is this?

/kind feature

What does does this PR do / why we need it:

It takes the custom debug command from the CLI while trigerring odo watch.

Which issue(s) this PR fixes:

Fixes #4357

PR acceptance criteria:

How to test changes / Special notes to the reviewer:

  • odo watch --debug-command <custom debug command> should work.

It takes the custom debug command from the CLI while trigerring odo watch.

Signed-off-by: mik-dass <mrinald7@gmail.com>
@openshift-ci-robot openshift-ci-robot added the kind/feature Categorizes issue as a feature request. For PRs, that means that the PR is the implementation label Jan 22, 2021
@openshift-ci-robot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: mik-dass

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

@openshift-ci-robot openshift-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. Required by Prow. label Jan 22, 2021
@dharmit
Copy link
Member

dharmit commented Jan 28, 2021

@mik-dass can you please add docs about how to use this so that we can update odo.dev website as well?

@mik-dass
Copy link
Contributor Author

mik-dass commented Feb 3, 2021

@mik-dass can you please add docs about how to use this so that we can update odo.dev website as well?

I have added a mention of the watch flags in a doc.

@dharmit
Copy link
Member

dharmit commented Feb 3, 2021

$ odo create nodejs --starter
$ odo push --debug
...
...

$ odo watch --debug-command npm run debug
Error: unknown command "run" for "odo watch"
Usage:
  odo watch [flags]
...
...

Am I doing something wrong here? The debug command that I have used here npm run debug was available from https://odo.dev/docs/debugging-using-devfile/.

Tangential note: it would help to have more documentation about odo watch on odo.dev. Or maybe it's just me struggling to find examples to test functionality when a PR related to odo watch pops up. 😞

@mik-dass
Copy link
Contributor Author

mik-dass commented Feb 3, 2021

Am I doing something wrong here? The debug command that I have used here npm run debug was available from https://odo.dev/docs/debugging-using-devfile/.

You need to mention the id of the devfile command not the command itself.

odo watch --debug-command debug

@dharmit
Copy link
Member

dharmit commented Feb 4, 2021

Am I doing something wrong here? The debug command that I have used here npm run debug was available from https://odo.dev/docs/debugging-using-devfile/.

You need to mention the id of the devfile command not the command itself.

odo watch --debug-command debug

🤦‍♂️

@mik-dass I am LGTM'ing this but, is it possible for you to review the docs for debug support on odo.dev and add something if needed? I personally feel that what you've added to docs in this PR can also be added to Debugging using devfile. Maybe we can ask @kadel to look at these from new user's perspective like he did for odo service and odo link.

/lgtm

@openshift-ci-robot openshift-ci-robot added the lgtm Indicates that a PR is ready to be merged. Required by Prow. label Feb 4, 2021
@openshift-bot
Copy link

/retest

Please review the full test history for this PR and help us cut down flakes.

@mik-dass
Copy link
Contributor Author

mik-dass commented Feb 4, 2021

@mik-dass I am LGTM'ing this but, is it possible for you to review the docs for debug support on odo.dev and add something if needed?

OK I will try to review the docs for proper debug support.

@openshift-merge-robot openshift-merge-robot merged commit 1f44e44 into redhat-developer:master Feb 4, 2021
kadel pushed a commit to kadel/odo that referenced this pull request Feb 4, 2021
* Adds debug-command flag to odo watch

It takes the custom debug command from the CLI while trigerring odo watch.

Signed-off-by: mik-dass <mrinald7@gmail.com>

* Adds a mention of the custom build, run and debug flag for watch in the doc
@mik-dass mik-dass deleted the watch-debug branch March 16, 2021 06:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. Required by Prow. kind/feature Categorizes issue as a feature request. For PRs, that means that the PR is the implementation lgtm Indicates that a PR is ready to be merged. Required by Prow.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Watch mode doesn't support debug command
5 participants