Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix absence of the Symlinks parameter for Destination when using tbot kube credentials #49809

Conversation

verdel
Copy link
Contributor

@verdel verdel commented Dec 5, 2024

The problem is described in the issue. After the tbot CLI refactoring, the bot.Destination object created for the directory specified in the --destination-dir parameter does not have default values set for the Symlinks attribute.

Fixes #49802

Copy link
Contributor

@timothyb89 timothyb89 left a comment

Choose a reason for hiding this comment

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

Thanks for catching this! I'll make a note to audit other codepaths to make sure there aren't any other missing CheckAndSetDefaults() calls.

@verdel
Copy link
Contributor Author

verdel commented Dec 17, 2024

@timothyb89, can I do anything to help and speed up the review process of this PR?

@zmb3
Copy link
Collaborator

zmb3 commented Dec 17, 2024

@timothyb89 feel free to open a buddy PR for this so we can push it through CI.

@timothyb89
Copy link
Contributor

Hi @verdel, I've created an internal PR for this due some CI limitations for external contributions on our end: #50370

CI looks to be passing and I don't anticipate much in the way of review feedback, but if anything comes up I'll sync any changes here back over to that PR. Otherwise, we'll get this merged and backported to v17 as soon as it gets through review. You'll retain authorship of your commits.

We'll close this PR and the associated issue once the internal PR is merged.

@verdel
Copy link
Contributor Author

verdel commented Dec 18, 2024

@timothyb89, Thank you for the PR status update and the additional recommendations with the alternative temporary solution for the issue. I will wait for the acceptance of PR #50370.

@timothyb89
Copy link
Contributor

The fix and its v17 backport have been merged, so I'll close this PR. Thanks again @verdel!

@timothyb89 timothyb89 closed this Dec 23, 2024
@verdel verdel deleted the fix-symlinks-param-absent-for-tbot-kube-credentials-destination-dir branch December 26, 2024 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Tbot: invalid symlinks mode error in tbot kube credentials
3 participants