Skip to content

Fix loading from -hubConfig and -nodeConfig files #2675

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
Aug 26, 2016

Conversation

mach6
Copy link
Member

@mach6 mach6 commented Aug 26, 2016


Changes to make -hubConfg and -nodeConfig work again.

  • If/when a hub/node config file is passed, it is used to initialize
    the GridHubConfiguration or GridNodeConfiguration object in
    GridLauncherV3
  • Any passed args (to GridLauncherV3) present from the command line
    will override values supplied in the hub/node config file.
  • Any value supplied for -hubConfig or -nodeConfig is checked
    for existence via new FileExistsValueValidator class
    Additional changes;
  • Hide parameter "-avoidProxy" from usage()
  • Remove whitespace in DefaultHub.json

Changes to make -hubConfg and -nodeConfig work again.
* If/when a hub/node config file is passed, it is used to initialize
  the GridHubConfiguration or GridNodeConfiguration object in
  GridLauncherV3
* Any passed args (to GridLauncherV3)  present from the command line
  will override values supplied in the hub/node config file.
* Any value supplied for -hubConfig or -nodeConfig is checked
  for existence via new FileExistsValueValidator class
Additional changes;
* Hide parameter "-avoidProxy" from usage()
* Remove whitespace in DefaultHub.json
@lukeis lukeis merged commit fbc82cd into SeleniumHQ:master Aug 26, 2016
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.

2 participants