Skip to content
This repository has been archived by the owner on Mar 28, 2020. It is now read-only.

fix sort, with a helper so the sort is done 'naturally' #2099

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

jurgenweber
Copy link

@jurgenweber jurgenweber commented Jul 2, 2019

Please read https://github.com/coreos/etcd-operator/blob/master/CONTRIBUTING.md#contribution-flow

I hit the edge case that when it sorts, it thinks

etcd_v1066060_2019-07-02-02:51:51
is smaller/older than
etcd_v999523_2019-06-30-14:56:49

because it is sorting on the first digit only.

Added all the stuff needed to sort this properly.

right now the periodic backup is continually deleting the same file.

@etcd-bot
Copy link
Collaborator

etcd-bot commented Jul 2, 2019

Can one of the admins verify this patch?

2 similar comments
@etcd-bot
Copy link
Collaborator

etcd-bot commented Jul 2, 2019

Can one of the admins verify this patch?

@etcd-bot
Copy link
Collaborator

etcd-bot commented Jul 2, 2019

Can one of the admins verify this patch?

@jurgenweber
Copy link
Author

so I built the image and used it overnight, it works as expected now.

Thanks

Copy link

@daxmc99 daxmc99 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me but I'm not associated with coreos

# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants