Chore: Update configuration.environmentId docs #427
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Issue & Steps to Reproduce / Feature Request
resolves : #347
Since Zip doesn't use
environment.configuration
but onlyconfiguration
with environment scope, a solution for using both is irrelevant.There was a misunderstanding regarding their usage but eventually, we decided to update the docs by describing that using
configuration.environment_id
requiresignoring changes
ofenvironment.configuration
The issue with allowing both is that
configuration
requires an environment, so at the first run, the environment will be deployed without configuration variables. From the 2nd deployment, there will be a drift because the configuration resource metadata will be retrieved by the API of the environment.In other words, those are 2 ways to manage the same resource so only one way should be picked. ( preferably the
environment.configuration
to prevent the first usage issue )Solution
Add ignore changes requirement in
configuration.environment_id
description