Skip to content

feat(yclockbox): add authorized_key support for yclockbox provider#1105

Merged
yxxhero merged 2 commits intohelmfile:mainfrom
Eug1n1:feat/yclockbox
Apr 9, 2026
Merged

feat(yclockbox): add authorized_key support for yclockbox provider#1105
yxxhero merged 2 commits intohelmfile:mainfrom
Eug1n1:feat/yclockbox

Conversation

@Eug1n1
Copy link
Copy Markdown
Contributor

@Eug1n1 Eug1n1 commented Apr 8, 2026

Added support for using Yandex Cloud service account authorized_key in YC_TOKEN for the yclockbox provider.

Signed-off-by: Shumskiy Evgeniy <vkzheny@gmail.com>
@yxxhero yxxhero requested a review from Copilot April 8, 2026 23:43
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds support for authenticating the yclockbox provider using a Yandex Cloud service account authorized_key JSON passed via YC_TOKEN, alongside existing IAM token support.

Changes:

  • Parse YC_TOKEN as either an IAM token string or a JSON authorized_key and build appropriate SDK credentials.
  • Update Lockbox provider authentication documentation to describe the dual-mode YC_TOKEN behavior.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 3 comments.

File Description
pkg/providers/yclockbox/yclockbox.go Adds env credential parsing to support authorized_key JSON in YC_TOKEN
README.md Documents the new YC_TOKEN behavior for IAM token vs authorized_key JSON

Comment thread pkg/providers/yclockbox/yclockbox.go Outdated
Comment thread pkg/providers/yclockbox/yclockbox.go Outdated
Comment thread pkg/providers/yclockbox/yclockbox.go
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Shumskiy Evgeniy <vkzheny@gmail.com>
@yxxhero yxxhero merged commit a575f9d into helmfile:main Apr 9, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants