Skip to content

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

Merged
merged 1 commit into from
Jun 22, 2021
Merged

Clean up import list #645

merged 1 commit into from
Jun 22, 2021

Conversation

mumoshu
Copy link
Collaborator

@mumoshu mumoshu commented Jun 22, 2021

Resolves #644

@mumoshu mumoshu mentioned this pull request Jun 22, 2021
@mumoshu mumoshu merged commit 8b90b0f into master Jun 22, 2021
@mumoshu mumoshu deleted the clean-up-import-list branch June 22, 2021 08:55
@@ -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 |
Copy link
Contributor

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
Copy link
Contributor

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?

Copy link
Collaborator Author

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

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.

Clean up import list
2 participants