ungleich-public/cdist
ungleich-public/cdist
7
4
Fork 10
Code Issues 58 Pull requests 15 Projects Releases Wiki Activity

Locally sort remote group information, to prevent differences in sort output. #367

Merged
fnux merged 1 commit from mark/cdist:__user_groups-sort into master 2024-05-01 11:58:14 +00:00
Conversation 1 Commits 1 Files changed 2 +5 -3
mark commented 2023-11-02 10:15:15 +00:00
Contributor
Copy link

Our current cdist system is running Debian 11. When configuring a Debian 12 system, we ran into a problem with the __user_groups type. This type sorts groups in the explorer and other group info in the gencode-remote on the cdist server. After some searching we observed the following.

Debian 11:

echo -e "Debian-exim\ncertificate" | sort
certificate
Debian-exim

Debian 12:

echo -e "Debian-exim\ncertificate" | sort
Debian-exim
certificate

It seems sort has different policy when upgrading from coreutils 8.32 to 9.1.

To prevent such a thing happening, it might be best to sort everything cdist side and not split over client and server.

Regards,

Mark

Our current cdist system is running Debian 11. When configuring a Debian 12 system, we ran into a problem with the `__user_groups` type. This type sorts groups in the explorer and other group info in the gencode-remote on the cdist server. After some searching we observed the following. Debian 11: ``` echo -e "Debian-exim\ncertificate" | sort certificate Debian-exim ``` Debian 12: ``` echo -e "Debian-exim\ncertificate" | sort Debian-exim certificate ``` It seems sort has different policy when upgrading from coreutils 8.32 to 9.1. To prevent such a thing happening, it might be best to sort everything cdist side and not split over client and server. Regards, Mark
mark added 1 commit 2023-11-02 10:15:34 +00:00
fnux commented 2024-05-01 11:58:01 +00:00
Collaborator
Copy link

LGTM, Thanks!

LGTM, Thanks!
fnux merged commit b7394ff4c2 into master 2024-05-01 11:58:14 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels   
bugfix

   
cleanup

   
discussion

   
documentation

   
doing

   
done

   
feature

   
improvement

   
packaging

   
Stale

   
testing

   
TODO
No labels
bugfix
cleanup
discussion
documentation
doing
done
feature
improvement
packaging
Stale
testing
TODO
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: ungleich-public/cdist#367
No description provided.
Delete branch "mark/cdist:__user_groups-sort"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?