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

RSA keys from existing secret #1014

Merged
merged 4 commits into from
Oct 30, 2023

Conversation

drustan
Copy link
Contributor

@drustan drustan commented Jul 31, 2023

The goal is to implement the ability to retrieve the RSA keys from an existing secret

@CLAassistant
Copy link

CLAassistant commented Jul 31, 2023

CLA assistant check
All committers have signed the CLA.

@drustan
Copy link
Contributor Author

drustan commented Oct 25, 2023

Hi! Is there any reason this is not reviewed / accepted?

Copy link
Contributor

@arikalon1 arikalon1 left a comment

Choose a reason for hiding this comment

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

Sorry for the late reply @drustan
Thanks for your contribution

@arikalon1 arikalon1 merged commit 6ee8d1f into robusta-dev:master Oct 30, 2023
8 checks passed
anakaiti pushed a commit to BrioHR/robusta that referenced this pull request Oct 31, 2023
* feat(helm): add the ability to retrieve the Robusta RSA keys from an existing secret.

* doc(setup-robusta): add the part mentionning the ability to retrieve the Robusta RSA keys from an existing secret
pavangudiwada pushed a commit to pavangudiwada/robusta that referenced this pull request Nov 6, 2023
* feat(helm): add the ability to retrieve the Robusta RSA keys from an existing secret.

* doc(setup-robusta): add the part mentionning the ability to retrieve the Robusta RSA keys from an existing secret
@jmalloc
Copy link

jmalloc commented Nov 23, 2023

Any chance of a new tag that includes these changes? It took me a while to work out why this wasn't working, because the documentation for it is already live 😉

@arikalon1
Copy link
Contributor

You're right @jmalloc
The docs contains the change, but it wasn't released yet.
It will be included in 0.10.26
It should be released within a week or so

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.

None yet

4 participants