Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

add help info for '--labels' #472

Merged
merged 1 commit into from
May 11, 2020
Merged

add help info for '--labels' #472

merged 1 commit into from
May 11, 2020

Conversation

TingluoHuang
Copy link
Member

@TingluoHuang TingluoHuang commented May 11, 2020 •
edited
Loading

Fix: #262

Config Options:
 --unattended     Disable interactive prompts for missing arguments. Defaults will be used for missing options
 --url string     Repository to add the runner to. Required if unattended
 --token string   Registration token. Required if unattended
 --name string    Name of the runner to configure (default htl-mac)
 --labels string  Extra labels in addition to the default: 'self-hosted,OSX,X64'
 --work string    Relative runner work directory (default _work)
 --replace        Replace any existing runner with the same name (default false)

Examples:
 Configure a runner non-interactively:
  ./config.sh --unattended --url <url> --token <token>
 Configure a runner non-interactively, replacing any existing runner with the same name:
  ./config.sh --unattended --url <url> --token <token> --replace [--name <name>]
 Configure a runner non-interactively with three extra labels:
  ./config.sh --unattended --url <url> --token <token> --labels L1,L2,L3

ericsciple
ericsciple reviewed May 11, 2020
View reviewed changes
src/Runner.Listener/Runner.cs Outdated
@@ -466,6 +466,7 @@ private void PrintUsage(CommandSettings command)
--url string Repository to add the runner to. Required if unattended
--token string Registration token. Required if unattended
--name string Name of the runner to configure (default {Environment.MachineName ?? "myrunner"})
--labels string Additional labels of the runner (default 'self-hosted', '{Constants.Runner.Platform}', '{Constants.Runner.PlatformArchitecture}')
Copy link
Collaborator

Choose a reason for hiding this comment

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

should this be default 'self-hosted,{Constants.Runner.Platform},{Constants.Runner.PlatformArchitecture}'

Copy link
Collaborator

Choose a reason for hiding this comment

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

nm.... this is additional

Copy link
Collaborator

Choose a reason for hiding this comment

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

Extra labels in addition to the default: 'self-hosted,{Constants.Runner.Platform},{Constants.Runner.PlatformArchitecture}'

@TingluoHuang TingluoHuang force-pushed the users/tihuang/labelshelp branch from 9c756a7 to 1fee46c Compare May 11, 2020 16:06
@TingluoHuang TingluoHuang merged commit 911135e into master May 11, 2020
@TingluoHuang TingluoHuang deleted the users/tihuang/labelshelp branch May 11, 2020 16:36
AdamOlech pushed a commit to antmicro/runner that referenced this pull request Jan 28, 2021
@TingluoHuang
add help info for '--labels' (actions#472)
b1d5bdc
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@ericsciple ericsciple ericsciple approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Setting and targeting based on custom labels
2 participants
@TingluoHuang @ericsciple