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

Vet execution plan before execution #2628

Open
Tracked by #2681
carolynvs opened this issue Mar 14, 2023 · 0 comments
Open
Tracked by #2681

Vet execution plan before execution #2628

carolynvs opened this issue Mar 14, 2023 · 0 comments
Labels
pep003-advanced-dependencies Implementation of the Advanced Dependencies proposal

Comments

@carolynvs
Copy link
Member

carolynvs commented Mar 14, 2023

Before running an execution plan, Porter must check that in theory it's possible to run:

  • All bundles exist and are pulled
  • The bundle uses supported extensions (right now this happens very late, right before we run it)
  • The action is valid
  • All dependency wiring in the graph is valid, i.e. you aren't passing a credential that doesn't exist to a dependency
  • Required credentials and parameters of a dependency in the graph are satisfied either through wiring or because the workflow is supplying values for unmapped params/creds
  • All credential and parameter sets used by the graph can be resolved

Before running a bundle in the execution plan, recheck before running that:

  • All credential and parameter sets used by the graph from this bundle down can be resolved
  • All bundles in the graph from this bundle down can be resolved

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

@carolynvs carolynvs added the pep003-advanced-dependencies Implementation of the Advanced Dependencies proposal label Mar 14, 2023
@carolynvs carolynvs mentioned this issue Mar 23, 2023
6 tasks
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
Projects
No open projects
Status: No status
Development

No branches or pull requests

1 participant