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

Specify parameter source with templating? #2561

Open
Tracked by #2667
carolynvs opened this issue Feb 9, 2023 · 0 comments
Open
Tracked by #2667

Specify parameter source with templating? #2561

carolynvs opened this issue Feb 9, 2023 · 0 comments
Assignees
Labels
pep003-advanced-dependencies Implementation of the Advanced Dependencies proposal placeholder Tracks work that has not yet be fully designed suggestion Idea for maintainers to consider. Do not take this issue until triaged.
Milestone

Comments

@carolynvs
Copy link
Member

carolynvs commented Feb 9, 2023

One change in PEP003 is the use of template syntax in areas outside of the install/upgrade/uninstall actions. For example, when specifying a dependency's parameters you can use template syntax to set it

dependencies:
  requires:
    - name: mydb
      parameters:
        username: ${bundle.parameters.username}

Should we do the same for parameter sources? We could support the older syntax but document/encourage the consistent templating syntax going forward.

parameters:
- name: port
  source:
    dependency: mysvc
    output: port

parameters:
- name: port
  source: ${bundle.dependencies.mysvc.outputs.port}

ℹ️ Read PEP003 - Advanced Dependencies for context about how dependencies should work, design details, and notes about desired behavior.

@carolynvs carolynvs added suggestion Idea for maintainers to consider. Do not take this issue until triaged. pep003-advanced-dependencies Implementation of the Advanced Dependencies proposal labels Feb 9, 2023
@carolynvs carolynvs added this to the Dependencies milestone Feb 9, 2023
@carolynvs carolynvs added the placeholder Tracks work that has not yet be fully designed label Mar 31, 2023
@carolynvs carolynvs self-assigned this Mar 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pep003-advanced-dependencies Implementation of the Advanced Dependencies proposal placeholder Tracks work that has not yet be fully designed suggestion Idea for maintainers to consider. Do not take this issue until triaged.
Projects
No open projects
Status: No status
Development

No branches or pull requests

1 participant