Sort automount configmaps and secrets to ensure deterministic ordering#1578
Sort automount configmaps and secrets to ensure deterministic ordering#1578dkwon17 wants to merge 2 commits intodevfile:mainfrom
Conversation
Signed-off-by: David Kwon <dakwon@redhat.com>
|
I plan to create e2e tests for this PR, in a new commit |
|
/retest |
|
I tested with the abovementioned steps and PR seems to be working as expected ✔️ |
Signed-off-by: David Kwon <dakwon@redhat.com> Assisted-by: Claude
|
/retest |
|
@dkwon17: The following test failed, say
Full PR test history. Your PR dashboard. DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here. |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: akurinnoy, dkwon17, rohanKanojia The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
What does this PR do?
Sorts automount configmaps and secrets so that they are ordered the same way for each devworkspace reconciliation. Sorting is done, since otherwise, there is no guarantee on the configmap/secret ordering returned by
api.Client.List. This can cause the ordering to differ between DevWorkspace reconciliations, causing the problem described in #1572 (comment).A similar change was made in the past with git credentials: e17e799.
In this PR, we apply the same idea to the automount secrets and configmaps.
What issues does this PR fix or reference?
#1572 (comment)
Is it tested? How?
It can be tested by running:
The output should include:
PR Checklist
/test v8-devworkspace-operator-e2e, v8-che-happy-pathto trigger)v8-devworkspace-operator-e2e: DevWorkspace e2e testv8-che-happy-path: Happy path for verification integration with Che