Skip to content

context flag doesn't work properly for component create using --starter flag #4475

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

Closed
prietyc123 opened this issue Mar 3, 2021 · 3 comments
Labels
kind/bug Categorizes issue or PR as related to a bug.

Comments

@prietyc123
Copy link
Contributor

/kind bug

What versions of software are you using?

Operating System:
All supported

Output of odo version:
master

How did you run odo exactly?

$ odo project create abcderf
 ✓  Project 'abcderf' is ready for use
 ✓  New project created and now using project: abcderf

$ odo create nodejs --starter --context ./testdis/
Validation
 ✓  Creating a devfile component from devfile path: /Users/pkumari/go/src/github.com/openshift/odo/testdis/devfile.yaml [67778ns]
 ✓  Validating devfile component [240338ns]
 ✗  Folder /Users/pkumari/go/src/github.com/openshift/odo is not empty. It can only contain the devfile used.

However when I tried the same step with changing the directory to ./testdis/ and run

$ odo create nodejs --starter --context .
Validation
 ✓  Creating a devfile component from devfile path: /Users/pkumari/go/src/github.com/openshift/odo/testdis/devfile.yaml [524522ns]
 ✓  Validating devfile component [1ms]

Starter Project
 ✓  Downloading starter project nodejs-starter from https://github.com/odo-devfiles/nodejs-ex.git [1s]

Please use `odo push` command to create the component with source deployed

It works fine.

Actual behavior

Expected behavior

When we use --context flag, I thing cd should not require for creating the component.

Any logs, error output, etc?

Trying steps manually

@openshift-ci-robot openshift-ci-robot added the kind/bug Categorizes issue or PR as related to a bug. label Mar 3, 2021
@valaparthvi
Copy link
Contributor

It seems to be working fine for me -

➜  empty_dir mkdir smh1  
➜  empty_dir odo create nodejs --starter --context ./smh1/   
Devfile Object Validation
 ✓  Checking devfile existence [76637ns]
 ✓  Creating a devfile component from registry: DefaultDevfileRegistry [78892ns]
Validation
 ✓  Validating if devfile name is correct [138053ns]

Starter Project
 ✓  Downloading starter project nodejs-starter from https://github.com/odo-devfiles/nodejs-ex.git [863ms]

Please use `odo push` command to create the component with source deployed

odo version - odo v2.0.7 (f0d881d)
OS: Fedora 32

@kadel
Copy link
Member

kadel commented Mar 16, 2021

working for me as well (using 6a73597)
/close

@prietyc123 If this is still an issue for you feel free to reopen.
To help with the reproduction and investigation please try to paste the actual odo version output or specify what git revision you were using. Saying just master makes it impossible to find what you were running.

@openshift-ci-robot
Copy link
Collaborator

@kadel: Closing this issue.

In response to this:

working for me as well (using 6a73597)
/close

@prietyc123 If this is still an issue for you feel free to reopen.
To help with the reproduction and investigation please try to paste the actual odo version output or specify what git revision you were using. Saying just master makes it impossible to find what you were running.

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/bug Categorizes issue or PR as related to a bug.
Projects
None yet
Development

No branches or pull requests

4 participants