Add bot permissions and turn on feature #37943
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
michellescripts
force-pushed
the
michelle/bots-on
branch
3 times, most recently
from
2ce9945 to
cde32f9
Compare
This was referenced Feb 12, 2024
gzdunek
approved these changes
Feb 13, 2024
ravicious
approved these changes
Feb 13, 2024
public-teleport-github-review-bot
bot
removed the request for review
from strideynet
michellescripts
force-pushed
the
michelle/bots-on
branch
from
cde32f9 to
cc87e47
Compare
github-merge-queue
bot
removed this pull request from the merge queue due to failed status checks
michellescripts
force-pushed
the
michelle/bots-on
branch
from
cc87e47 to
e81b140
Compare
github-merge-queue
bot
removed this pull request from the merge queue due to failed status checks
michellescripts
force-pushed
the
michelle/bots-on
branch
from
e81b140 to
17193c4
Compare
github-merge-queue
bot
removed this pull request from the merge queue due to failed status checks
github-merge-queue
bot
removed this pull request from the merge queue due to failed status checks
michellescripts
force-pushed
the
michelle/bots-on
branch
from
17193c4 to
e0cbc8e
Compare
michellescripts
force-pushed
the
michelle/bots-on
branch
from
e0cbc8e to
c622bb4
Compare
michellescripts
force-pushed
the
michelle/bots-on
branch
from
c622bb4 to
e1db9db
Compare
michellescripts
added a commit
that referenced
this pull request
Feb 13, 2024
michellescripts
added a commit
that referenced
this pull request
Feb 16, 2024
github-merge-queue bot
pushed a commit
that referenced
this pull request
Feb 16, 2024
* Add endpoint to get bots by name (#37412) * Add endpoint to get bots by name * Lint * Fix test method's name Co-authored-by: Noah Stride <[email protected]> --------- Co-authored-by: Noah Stride <[email protected]> * Backend supporting changes for Bot creation flow (#37348) * Add bot ui labels and github join token * Use strings.HasPrefix instead of includes * Add `ssh` to label * Add bot join token endpoint * Lint * Fix comment typo Co-authored-by: Bartosz Leper <[email protected]> * Fix comment typo Co-authored-by: Bartosz Leper <[email protected]> * Improve webUIFlowLabelKey comment --------- Co-authored-by: Bartosz Leper <[email protected]> * add bots UI, disabled (#36845) - in original data table view * Add Bot + GitHub Actions SSH UI Flow (#37443) * add bots UI, disabled - in original data table view * Add bot creation UI * Remove duplicated types * add bots UI, disabled - in original data table view * Fix types and tests * Use bot join token api endpoint * Fix tests * Linting and small fixes * Fix tests * Add missing licenses * Improve styles, error messages, etc * Remove clusterId from bot routes * Undo enabling feature * Remove unused join role bot * Rename var * Reuse makeListBot * Add missing type * Revert MachineIDIntegrationSection for now * Lint * Apply suggestions from code review - fix typos Co-authored-by: Noah Stride <[email protected]> * Remove kubernetes section fromm example yaml * Remove border color from reftype selector * Small changes to address code review * Use setAttempt * add try/catch block when parsing repo addresses * Improve tests;remove unecessary fragment * Use gap in flex. Fix typo * Lint fix * Drop "ex" from input placeholders * Add stories for no perm and bot picker * Add copy to explain wrkflow name limits * fix setCurentStep * Fix invalid host error rendering * Use PascalCase for error components * Improve field name validation * Remove unecessary comments * Update copy and minor style change --------- Co-authored-by: Michelle Bergquist <[email protected]> Co-authored-by: Noah Stride <[email protected]> * add bot edit-role flow (#37828) * add bot edit-role flow * only send masked/updated fields * Add GitHub Actions bot view (#37852) * add bots UI, disabled - in original data table view * Add bot creation UI * Remove duplicated types * add bots UI, disabled - in original data table view * Fix types and tests * Use bot join token api endpoint * Fix tests * Linting and small fixes * Fix tests * Add missing licenses * Improve styles, error messages, etc * Remove clusterId from bot routes * Undo enabling feature * Remove unused join role bot * Rename var * Reuse makeListBot * Add missing type * Revert MachineIDIntegrationSection for now * Lint * Apply suggestions from code review - fix typos Co-authored-by: Noah Stride <[email protected]> * Remove kubernetes section fromm example yaml * Remove border color from reftype selector * Small changes to address code review * Use setAttempt * add try/catch block when parsing repo addresses * Improve tests;remove unecessary fragment * Use gap in flex. Fix typo * Lint fix * Drop "ex" from input placeholders * Add stories for no perm and bot picker * Add copy to explain wrkflow name limits * fix setCurentStep * Fix invalid host error rendering * Use PascalCase for error components * Improve field name validation * Remove unecessary comments * Add bot type by label * Add bot view... * Show view gh actions yaml only for gh bots * Use existing pattern for operations * Add story and missing license * Fix typos Co-authored-by: Michelle Bergquist <[email protected]> --------- Co-authored-by: Michelle Bergquist <[email protected]> Co-authored-by: Noah Stride <[email protected]> Co-authored-by: Michelle Bergquist <[email protected]> * Add buttons to download and copy to clipboard content to `TextEditor` (#37333) * Add editor buttons * Set icon size * Move downloadObject to OSS * remove unused test-id * Use values from theme * Fix buttons positioning * Convert jsx files to tsx * Use const instead of var * Add z-index to buttons * Add license * Improvements to Bot resource (additional validation and label propagation) (#38013) * PRevent creating bots with an empty string role * Propagate labels to Bot user and vice versa * Extract slice declaration for nonPropagatedLabels * Fixed web tests relying on empty string roles * Use "set" instead of slice for nonPropagatedLabels * Make testing of empty string handling more thorough * add role assertions to create * Appease linter as to want/got order --------- Co-authored-by: Michelle Bergquist <[email protected]> * Add bot permissions and turn on feature (#37943) * Fix makeBot labels (#38176) --------- Co-authored-by: matheus <[email protected]> Co-authored-by: Noah Stride <[email protected]> Co-authored-by: Bartosz Leper <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds the remaining bot permission checks (list, write, delete) and turns on the bot list feature.
changelog: Add the ability to view and manage Machine ID bots from the UI
closes #34260