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

'odo create nodejs-angular' creates component with two the same named 'dist' storage items #5397

Closed
dgolovin opened this issue Jan 27, 2022 · 8 comments
Labels
area/UX Issues or PRs related to User Experience kind/bug Categorizes issue or PR as related to a bug. priority/Medium Nice to have issue. Getting it done before priority changes would be great.

Comments

@dgolovin
Copy link
Contributor

/kind bug

What versions of software are you using?

Operating System: Linux

Output of odo version: v2.5.0

How did you run odo exactly?

odo create nodejs-angular --starter=nodejs-angular-starter

Actual behavior

Creates component with two storage items with the same name dist. There is no way to delete one of the items using odo storage delete dist, because it deletes both of the storage items.

Expected behavior

Storage items have unique names and odo delete storage lets delete specific storage item.

Any logs, error output, etc?

@openshift-ci openshift-ci bot added the kind/bug Categorizes issue or PR as related to a bug. label Jan 27, 2022
@kadel
Copy link
Member

kadel commented Jan 27, 2022

Creates component with two storage items with the same name dist. There is no way to delete one of the items using odo storage delete dist, because it deletes both of the storage items.

This behavior is actually correct. There is only one storage. The devfile defines only one storage (volume) that is shared across into two containers.

But we will probably need to revisit the odo storage list output, as it might not clearly communicate what is going on.

$ odo storage list
The component 'nodejs-angular-angular-qafh' has the following storage attached:
NAME     SIZE     PATH                             CONTAINER     STATE
dist     4Gi      /project/dist/nodejs-angular     builder       Pushed
dist     4Gi      /usr/share/nginx/html            server        Pushed

/kind ux

@openshift-ci
Copy link

openshift-ci bot commented Jan 27, 2022

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

In response to this:

Creates component with two storage items with the same name dist. There is no way to delete one of the items using odo storage delete dist, because it deletes both of the storage items.

This behavior is actually correct. There is only one storage. The devfile defines only one storage (volume) that is shared across into two containers.

But we will probably need to revisit the odo storage list output, as it might not clearly communicate what is going on.

$ odo storage list
The component 'nodejs-angular-angular-qafh' has the following storage attached:
NAME     SIZE     PATH                             CONTAINER     STATE
dist     4Gi      /project/dist/nodejs-angular     builder       Pushed
dist     4Gi      /usr/share/nginx/html            server        Pushed

/kind ux

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.

@kadel kadel added priority/Medium Nice to have issue. Getting it done before priority changes would be great. area/UX Issues or PRs related to User Experience labels Jan 27, 2022
@dgolovin
Copy link
Contributor Author

@kadel basically you are saying that IDE should show one 'dist' even though it is two reported in json. Will fix that.

@kadel
Copy link
Member

kadel commented Jan 28, 2022

@kadel basically you are saying that IDE should show one 'dist' even though it is two reported in json. Will fix that.

yes that is correct. Just the way how odo shows this is not intuitive :-/

@dgolovin
Copy link
Contributor Author

dgolovin commented Jan 28, 2022

I have couple other questions:

  • How do I list containers for component?
  • Can I delete build container or it should not be allowed, because it breaks 'odo push'?
  • How do I create component with multiple containers?

@kadel
Copy link
Member

kadel commented Jan 31, 2022

  • How do I list containers for component?

No way to do that using odo cli, you would have to check devfile.yaml

  • Can I delete build container or it should not be allowed, because it breaks 'odo push'?

You can't do that from odo cli, but you can edit devfile.yaml directly and remove it. If you are thinking about doing this in nodejs-angular devfile than it will definitely break that devfile.

  • How do I create component with multiple containers?

You write devfile.yaml with multiple containers. Odo is not creating new devfiles, it is just downloading devfiles that are present in Devile registry.

@valaparthvi
Copy link
Contributor

/close
Closing this in favor of v3. odo does not handle storage as of now, so this issue isn't really relevant.

Feel free to open it if you think otherwise.

@openshift-ci openshift-ci bot closed this as completed Oct 13, 2022
@openshift-ci
Copy link

openshift-ci bot commented Oct 13, 2022

@valaparthvi: Closing this issue.

In response to this:

/close
Closing this in favor of v3. odo does not handle storage as of now, so this issue isn't really relevant.

Feel free to open it if you think otherwise.

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
area/UX Issues or PRs related to User Experience kind/bug Categorizes issue or PR as related to a bug. priority/Medium Nice to have issue. Getting it done before priority changes would be great.
Projects
Archived in project
Development

No branches or pull requests

3 participants