-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Clean up import list #645
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
Clean up import list #645
Conversation
Resolves #644
@@ -21,7 +21,7 @@ _Default values are the defaults set in the charts values.yaml, some properties | |||
| `authSecret.github_app_installation_id` | The ID of your GitHub App installation. **This can't be set at the same time as `authSecret.github_token`** | | | |||
| `authSecret.github_app_private_key` | The multiline string of your GitHub App's private key. **This can't be set at the same time as `authSecret.github_token`** | | | |||
| `authSecret.github_token` | Your chosen GitHub PAT token. **This can't be set at the same time as the `authSecret.github_app_*`** | | | |||
| `image.repository` | The "repository/image" of the controller container | summerwind/actions-runner-controller | | |||
| `image.repository` | The "repository/image" of the controller container | actions-runner-controller/actions-runner-controller | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I do not think that one needs changing? This is the container image that will run the controller right?
@@ -26,7 +26,7 @@ authSecret: | |||
#github_token: "" | |||
|
|||
image: | |||
repository: summerwind/actions-runner-controller | |||
repository: actions-runner-controller/actions-runner-controller |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same as above, not sure that one needs changing?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sledigabel Ah, sorry I missed your comments! Thanks- I'm fixing the two issues in #663
Resolves #644