Repository navigation
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: xrl The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Hi @xrl. Thanks for your PR. I'm waiting for a etcd-io member to verify that this patch is reasonable to test. If it is, they should reply with Regular contributors should join the org to skip this step. Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. 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. |
The API documents validityDuration as a human-readable duration like "100d12h", and the samples used "365d", but parseValidityDuration delegated directly to time.ParseDuration which rejects the "d" unit, so reconciliation failed with 'failed to parse ValidityDuration: time: unknown unit "d" in duration "365d"'. Expand a leading day segment to hours before parsing. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01HTTkYNHaMM2bBj2uRY4Pz4 Signed-off-by: Xavier Lange <xrlange@gmail.com>
14e2589 to
3d4bf7f
Compare
validityDurationis documented as a human-readable duration (100d12h, with defaults described as90d/365d; seeapi/v1alpha1/etcdcluster_types.go:126), butparseValidityDurationhands the string straight totime.ParseDuration, which has nodunit. Setting365dmakes every reconcile fail withfailed to parse ValidityDuration: time: unknown unit "d" in duration "365d". The samples were switched to8760hto sidestep it; the API doc still promises days.This expands a leading day segment to hours before parsing. Plain Go durations and the empty-string default are unchanged. Table-driven tests cover day-suffix, mixed, plain, default and invalid inputs.
Rebased onto main after the #362 rewrite.
make testandmake lintpass.