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

porter explain not showing bundle outputs with applyTo array #1667

Closed
vdice opened this issue Jul 12, 2021 · 2 comments · Fixed by #1668
Closed

porter explain not showing bundle outputs with applyTo array #1667

vdice opened this issue Jul 12, 2021 · 2 comments · Fixed by #1668
Assignees
Labels
1 - 🍫 Eat chocolate _after_ emergency donuts bug Oops, sorry!

Comments

@vdice
Copy link
Member

vdice commented Jul 12, 2021

Describe the bug

porter explain doesn't appear to show bundle-level outputs that have applyTo sections

To Reproduce

Steps to reproduce the behavior:

  1. Choose a bundle with outputs that use applyTo (I used the azure-terraform bundle in the examples directory of this repo)
  2. Run porter explain
  3. See that no outputs are listed even though the bundle lists 3 at the bundle-level:
 $ porter explain
Name: azure-terraform
Description: An example Porter Bundle using Terraform and Azure
Version: 1.1.0
Porter Version: v0.38.1

Credentials:
Name              Description   Required   Applies To
client_id                       true       All Actions
client_secret                   true       All Actions
subscription_id                 true       All Actions
tenant_id                       true       All Actions

Parameters:
Name                     Description   Type     Default            Required   Applies To
database-name                          string   porter-terraform   false      All Actions
location                               string   EastUS             false      All Actions
resource_group_name                    string   porter-terraform   false      All Actions
storage_account_name                   string   porterstorage      false      All Actions
storage_container_name                 string   portertf           false      All Actions
storage_rg                             string   porter-storage     false      All Actions

No outputs defined

No custom actions defined

No dependencies defined

Expected behavior

I would expect the 3 bundle-level outputs to be listed (with the specific actions that they apply to as well), e.g.:

Outputs:
Name                          Description   Type     Applies To
cosmos-db-uri                               string   install,upgrade
eventhubs_connection_string                 string   install,upgrade
storage_account_key                         string   install

Version

Copy the output of porter version below

 $ porter version
porter v0.38.1 (3018b91c)

cc @squillace who spotted this one

@vdice vdice added the bug Oops, sorry! label Jul 12, 2021
@carolynvs carolynvs added the 1 - 🍫 Eat chocolate _after_ emergency donuts label Jul 12, 2021
@carolynvs
Copy link
Member

I think that I may have been the one to break this. I'll take a look.

@squillace
Copy link

whoo hoo!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
1 - 🍫 Eat chocolate _after_ emergency donuts bug Oops, sorry!
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants