forked from redhat-developer/odo
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
<!-- Thank you for opening a PR! Here are some things you need to know before submitting: 1. Please read our developer guideline: https://github.com/redhat-developer/odo/wiki/Developer-Guidelines 2. Label this PR accordingly with the '/kind' line 3. Ensure you have written and ran the appropriate tests: https://github.com/redhat-developer/odo/wiki/Writing-and-running-tests 4. Read how we approve and LGTM each PR: https://github.com/redhat-developer/odo/wiki/PR-Review Documentation: If you are pushing a change to documentation, please read: https://github.com/redhat-developer/odo/wiki/Contributing-to-Docs --> **What type of PR is this:** <!-- Add one of the following kinds: /kind bug /kind cleanup /kind tests /kind documentation Feel free to use other [labels](https://github.com/redhat-developer/odo/labels) as needed. However one of the above labels must be present or the PR will not be reviewed. This instruction is for reviewers as well. --> /kind feature **What does this PR do / why we need it:** This redesigns the `odo list` functionality like so: ```sh ~/nodejs ▶ odo list NAME TYPE MODE MANAGED URLS PORTS STATUS AGE * nodejs-prj1-api-abhz nodejs devfile.yaml odo Not Pushed foo nodejs Dev odo http://foobar.k8s.land 3000 Running 2m15s digitalocean-dyndns <unknown> <unknown> Helm Running 17d duplicati <unknown> <unknown> Helm https://duplicati.k8s.land 8200 Running 37d hass <unknown> <unknown> Helm https://hass.k8s.land 8123 Running 17d hostpath-provisioner <unknown> <unknown> Helm Running 37d syncthing <unknown> <unknown> Helm https://syncthing.k8s.land 21027, 22000, 8384 Running 39d ``` Changes: - Refactors components in component/component.go files - Implement changes so we rely less on the local environment files and more from retrieving from the cluster **Which issue(s) this PR fixes:** <!-- Specifying the issue will automatically close it when this PR is merged --> Fixes redhat-developer#5405 **PR acceptance criteria:** - [ ] Unit test - [ ] Integration test - [ ] Documentation **How to test changes / Special notes to the reviewer:** ```sh git clone https://github.com/odo-devfiles/nodejs-ex cd nodejs-ex odo create nodejs test odo push odo list ```
- Loading branch information
Showing
206 changed files
with
14,768 additions
and
5,195 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.