Skip to content

Add support for IBM Cloud Custom Credentials secret type#4838

Closed
yonatanyell wants to merge 5 commits intoexternal-secrets:mainfrom
yonatanyell:main
Closed

Add support for IBM Cloud Custom Credentials secret type#4838
yonatanyell wants to merge 5 commits intoexternal-secrets:mainfrom
yonatanyell:main

Conversation

@yonatanyell
Copy link
Copy Markdown

@yonatanyell yonatanyell commented May 26, 2025

Problem Statement

IBM Cloud's Secret Manager service added a new secret type called Custom Credentials.

Related Issue

Feature request

Proposed Changes

We would like to add the support for this secret kind in ESO, for the use of IBM Cloud users who need to manage the life cycle of custom credentials within Secrets Manager, and want to use ESO to pull them into Kubernetes secrets

Checklist

  • I have read the contribution guidelines
  • All commits are signed with git commit --signoff
  • My changes have reasonable test coverage
  • All tests pass with make test
  • I ensured my PR is ready for review with make reviewable

Yonathan-Yellin added 2 commits May 26, 2025 13:47
Signed-off-by: Yonathan-Yellin <yonathan.yellin@ibm.com>
@yonatanyell yonatanyell requested a review from a team as a code owner May 26, 2025 11:01
@yonatanyell yonatanyell requested a review from knelasevero May 26, 2025 11:01
@IdanAdar
Copy link
Copy Markdown
Collaborator

@knelasevero Remind me, this PR should have an image auto-built for it so we can verify it further right?

@Skarlso
Copy link
Copy Markdown
Contributor

Skarlso commented May 26, 2025

It should be somewhere in the log, yes.

Yonathan-Yellin added 3 commits May 26, 2025 15:17
Signed-off-by: Yonathan-Yellin <yonathan.yellin@ibm.com>
# Conflicts:
#	pkg/provider/ibm/provider_test.go
Signed-off-by: Yonathan-Yellin <yonathan.yellin@ibm.com>
@sonarqubecloud
Copy link
Copy Markdown

@IdanAdar
Copy link
Copy Markdown
Collaborator

Replaced by this PR #4899

@IdanAdar IdanAdar closed this Jun 12, 2025
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