Skip to content
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

add extra flag to override default plan or size of the exit-node #106

Merged
merged 1 commit into from
Jul 21, 2021

Conversation

jsiebens
Copy link
Contributor

Description

This PR adds an additional flag to override the default plan or size of the exit node

How Has This Been Tested?

Successfully created an exit node with a different plan, e.g. Scaleway Stardust:

$ go run main.go create -p scaleway --region=nl-ams-1 --plan=STARDUST1-S ...
Using provider: scaleway
Requesting host: strange-wiles1 in nl-ams-1, from scaleway
Host: d1622777-0c3e-4a03-975c-6825e42d9163, status: stopped
[1/500] Host: d1622777-0c3e-4a03-975c-6825e42d9163, status: starting
[2/500] Host: d1622777-0c3e-4a03-975c-6825e42d9163, status: starting
...
[18/500] Host: d1622777-0c3e-4a03-975c-6825e42d9163, status: active
inlets PRO TCP (0.8.3) server summary:
  IP: 51.158.189.204
  Auth-token: JoWxVv7yBuOH7BEEEZIzItd9EPENIifZtAIHJWe11n6oMSFN797wy3bXroRpzSxG

Connect client:

$ inlets-pro tcp client --url "wss://51.158.189.204:8123" \
>   --token "JoWxVv7yBuOH7BEEEZIzItd9EPENIifZtAIHJWe11n6oMSFN797wy3bXroRpzSxG" \
>   --upstream $UPSTREAM \
>   --ports $PORTS
2021/07/19 21:37:10 Starting TCP client. Version 0.8.5 - 8db64ded51b1455cf4ad027c52bc9cfbeb55c4b3
2021/07/19 21:37:10 Licensed to: Johan Siebens <redacted>, expires: 72 day(s)
2021/07/19 21:37:10 Upstream server: localhost, for ports: 8000
inlets-pro client. Copyright Alex Ellis, OpenFaaS Ltd 2020
INFO[2021/07/19 21:37:10] Connecting to proxy                           url="wss://51.158.189.204:8123/connect"
INFO[2021/07/19 21:37:10] Connection established.. OK.               

How are existing users impacted? What migration steps/scripts do we need?

no impact

Checklist:

I have:

  • updated the documentation and/or roadmap (if required)
  • read the CONTRIBUTION guide
  • signed-off my commits with git commit -s
  • added unit tests

Signed-off-by: Johan Siebens <johan.siebens@gmail.com>
Copy link
Member

@alexellis alexellis left a comment

Choose a reason for hiding this comment

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

Approved

@alexellis alexellis merged commit cc3a495 into inlets:master Jul 21, 2021
@alexellis
Copy link
Member

cc @dirien

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