You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
It is possible to create a ParameterSet as a YAML file and apply it as a CredentialSet and the other way around. I would expect porter to complain as the schemaType does not match.
To Reproduce
Steps to reproduce the behavior:
Run porter parameters create myparams.yaml
See creating porter parameter set in the current directory
Run porter credentials apply myparams.yaml
See error: Applied NAMESPACE/NAME credential set
Or run porter credentials create mycreds.yaml
See creating porter credential set in the current directory
Run porter parameters apply mycreds.yaml
See error: Applied NAMESPACE/NAME parameter set
Expected behavior
Porter should abort, stating that a CredentialSet can not be applied as a ParameterSet and the other way around.
That field was originally intended just to help us out later when implementing support for VS Code autocomplete, but yeah we can totally check for if the schemaType is set (it's optional) and then validate it.
Describe the bug
It is possible to create a ParameterSet as a YAML file and apply it as a CredentialSet and the other way around. I would expect porter to complain as the schemaType does not match.
To Reproduce
Steps to reproduce the behavior:
porter parameters create myparams.yaml
creating porter parameter set in the current directory
porter credentials apply myparams.yaml
Applied NAMESPACE/NAME credential set
porter credentials create mycreds.yaml
creating porter credential set in the current directory
porter parameters apply mycreds.yaml
Applied NAMESPACE/NAME parameter set
Expected behavior
Porter should abort, stating that a CredentialSet can not be applied as a ParameterSet and the other way around.
Porter Command and Output
See above
Version
porter v1.0.0-beta.1 (eaf8d0d)
The text was updated successfully, but these errors were encountered: