[Choreo] Removal of unused internal service resource path named retrieve-api-artifacts and its usage #3639
Conversation
…rtifacts and its usage
adapter/internal/adapter/adapter.go
Outdated
| } else { | ||
| adapter.GetAPIs(c, nil, envs, sync.APIArtifactEndpoint, true, apiUUIDList, queryParamMap) | ||
| } | ||
| } else if conf.ControlPlane.DynamicEnvironments.Enabled { |
There was a problem hiding this comment.
We no longer need Dynamic Environments Enabled flag too.
There was a problem hiding this comment.
Config is used in multiple places. Since the issue -> https://github.com/wso2-enterprise/choreo/issues/31875
Is bit urgent shall we track removal of it with another task ?
There was a problem hiding this comment.
If we remove the code, wont the config get ignored or throw an error?
There was a problem hiding this comment.
You mean the cp-overlay config right ?
I'll not throw an error. But we'll have to clean up the repos if we remove the config.
There was a problem hiding this comment.
If we remove the code, wont the config get ignored or throw an error?
Ignored.
Since it is always TRUE (feature flag) we can think it's boolean true and remove unnecessary if blocks.
Purpose
$subject
Relates with: https://github.com/wso2-enterprise/choreo/issues/31875
Issues
Fixes #
Automation tests
Tested environments
Not Tested
Maintainers: Check before merge