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

cli: "porter list: list INSTALLED bundles" #2773

Closed
schristoff opened this issue Jun 7, 2023 · 2 comments
Closed

cli: "porter list: list INSTALLED bundles" #2773

schristoff opened this issue Jun 7, 2023 · 2 comments
Labels
bug Oops, sorry! good first issue Good for new contributors

Comments

@schristoff
Copy link
Member

Describe the bug

porter list shows installed and uninstalled bundles.

To Reproduce

Steps to reproduce the behavior:
the 'helper' for the cli command could be updated, or we can add a flag to porter list

Porter Command and Output

Aliased Commands:
  archive       Archive a bundle from a reference
  build         Build a bundle
  copy          Copy a bundle
  create        Create a bundle
  explain       Explain a bundle
  inspect       Inspect a bundle
  install       Create a new installation of a bundle
  invoke        Invoke a custom action on an installation
  lint          Lint a bundle
 * list          List installed bundles*
porter list
----------------------------------------------------------------------------
  NAMESPACE  NAME          VERSION  STATE        STATUS     MODIFIED
----------------------------------------------------------------------------
             porter-hello  0.2.0    installed    succeeded  14 minutes ago
             hello         0.2.0    uninstalled  succeeded  2023-05-17
                           0.2.0    installed    succeeded  2023-05-17

Potential Fixes

  • just change the tag to show "lists installed and uninstalled bundles"
  • remove uninstalled bundles unless we pass an --all flag, but this could break any code that relies on it showing all uninstalled bundles
@schristoff schristoff added bug Oops, sorry! good first issue Good for new contributors labels Jun 7, 2023
@harjyotbagga
Copy link

I would like to pick this issue up.

@ludfjig
Copy link
Contributor

ludfjig commented Sep 19, 2023

Closing this since #2888 fixes this

@ludfjig ludfjig closed this as completed Sep 19, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Oops, sorry! good first issue Good for new contributors
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants