Skip to content

Removes use of the undocumented dev-varnish vm#67

Merged
jason0x43 merged 1 commit intotheintern:masterfrom
nosilleg:patch-1
Oct 10, 2018
Merged

Removes use of the undocumented dev-varnish vm#67
jason0x43 merged 1 commit intotheintern:masterfrom
nosilleg:patch-1

Conversation

@nosilleg
Copy link
Copy Markdown
Contributor

Tests starting failing with Sauce Labs unable to bring up a machine. Later got an email from SL saying that they were "planning to remove" undocumented features including dev-varnish setting. I've removed this code from my local install of DigDug and Intern is now able to run tests against Sauce Labs again.

Tests starting failing with Sauce Labs unable to bring up a machine.  Later got an email from SL saying that they were planning on removing undocumented features including dev-varnish setting.  I've removed this code from my local install of DigDug and Intern is now able to run tests against Sauce Labs.
@jsf-clabot
Copy link
Copy Markdown

jsf-clabot commented Sep 27, 2018

CLA assistant check
All committers have signed the CLA.

@chandra-sauce
Copy link
Copy Markdown

Hi, as @nosilleg mentioned we are dropping support for undocumented Sauce Connect options. Please accept Jason's PR so that you are not impacted by this upcoming change on Sauce Labs. Please refer to Sauce Connect Wiki for supported options.

Thanks,
Chandra
Product Manager - Sauce Connect

@jason0x43 jason0x43 merged commit 6510928 into theintern:master Oct 10, 2018
@nosilleg nosilleg deleted the patch-1 branch October 10, 2018 11:55
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.

4 participants