Skip to content

add get secrets perms to cluster-manager-account service account#572

Merged
okdas merged 2 commits intomainfrom
add-privkey-get-to-clustermgr
Mar 9, 2023
Merged

add get secrets perms to cluster-manager-account service account#572
okdas merged 2 commits intomainfrom
add-privkey-get-to-clustermgr

Conversation

@okdas
Copy link
Copy Markdown
Contributor

@okdas okdas commented Mar 9, 2023

Description

Allows cluster-manager to get v1-localnet-validators-private-keys secret.

Issue

Cluster manager complained about inability to access private keys secret when ran localnet from scratch.

Type of change

Please mark the relevant option(s):

  • New feature, functionality or library
  • Bug fix
  • Code health or cleanup
  • Major breaking change
  • Documentation
  • Other

List of changes

  • add cluster-manager-account service account to an existing binding

Testing

  • make develop_test
  • LocalNet w/ all of the steps outlined in the README

Required Checklist

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have tested my changes using the available tooling
  • I have updated the corresponding CHANGELOG

If Applicable Checklist

  • I have updated the corresponding README(s); local and/or global
  • I have added tests that prove my fix is effective or that my feature works
  • I have added, or updated, mermaid.js diagrams in the corresponding README(s)
  • I have added, or updated, documentation and mermaid.js diagrams in shared/docs/* if I updated shared/*README(s)

@okdas okdas self-assigned this Mar 9, 2023
@okdas okdas added infra Core infrastructure - not protocol related tooling tooling to support development, testing et al labels Mar 9, 2023
@okdas okdas added this to the M*: Pocket QoL (Quality of Life) milestone Mar 9, 2023
@okdas okdas requested a review from deblasis March 9, 2023 02:33
Copy link
Copy Markdown
Contributor

@deblasis deblasis left a comment

Choose a reason for hiding this comment

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

Oh, interesting, it looks like I must have missed a commit or something. 🤦‍♂️

Nice catch!

LGTM 🚀

@okdas okdas merged commit aedabed into main Mar 9, 2023
@okdas okdas deleted the add-privkey-get-to-clustermgr branch March 9, 2023 20:18
bryanchriswhite added a commit that referenced this pull request Mar 16, 2023
* pokt/main:
  [P2P] test: URL conversion with mockdns (#586)
  Create devlog4.md
  Update devlog3.md to remove demo
  Create devlog3.md
  [CLI] add private keys embed to debug cli build (#578)
  [Documentation] Fix SLIP Master Key and Child Key diagrams to be in the correct order (#573)
  add get secrets perms to cluster-manager-account service account (#572)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

infra Core infrastructure - not protocol related tooling tooling to support development, testing et al

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants