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

add new role and role binding for adviser #2607

Merged

Conversation

KPostOffice
Copy link
Member

add role which will be used for adviser workflows so that secrets created for callback urls can be deleted

Related Issues and Dependencies

closes: #2557

Does this require new deployment ?

  • Deployment for Test and Stage AICoE/aicoe-cd and Prod operate-first/argocd-apps.

@sesheta sesheta added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jul 6, 2022
@KPostOffice KPostOffice force-pushed the serv_account4workflow_helpers branch 2 times, most recently from b3eb836 to 4dbf50b Compare July 7, 2022 19:30
Copy link
Member

@harshad16 harshad16 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 the pr
i dont see the service account being referenced anywhere.

adviser/base/adviser-sa.yaml Outdated Show resolved Hide resolved
@harshad16
Copy link
Member

can you fix the kustomize issue to get this pr in
thank you

add role which will be used for send webhook step of adviser workflows so that secrets created for callback urls can be deleted
@KPostOffice KPostOffice force-pushed the serv_account4workflow_helpers branch from 694d212 to cfad29d Compare August 3, 2022 21:10
Copy link
Member

@harshad16 harshad16 left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

@sesheta sesheta added the lgtm Indicates that a PR is ready to be merged. label Sep 6, 2022
@sesheta
Copy link
Member

sesheta commented Sep 6, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: harshad16

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@sesheta sesheta added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 6, 2022
@sesheta sesheta merged commit f28c2f7 into thoth-station:master Sep 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[2pt] Create a new ServiceAccount for callback sending task with the ability to delete secrets
3 participants