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

chore: expose tink server via Vagrant port forward #221

Merged
merged 1 commit into from
Jul 17, 2020
Merged

chore: expose tink server via Vagrant port forward #221

merged 1 commit into from
Jul 17, 2020

Conversation

gianarb
Copy link
Contributor

@gianarb gianarb commented Jul 17, 2020

Description

I am working at the e2e test for the vagrant setup. It is very useful to
be able to reach tink-server from outside the VM because that where the
logic that asserts the right execution lives.

It is also good for experimentation because I was able to build the tink
cli:

go build -o tink ./cmd/tink-cli/main.go

I was able to use the tink cli from my local environment. Way more
familiar that docker exec as a workflow imho:

$ TINKERBELL_GRPC_AUTHORITY=127.0.0.1:42113 TINKERBELL_CERT_URL=http://127.0.0.1:42114/cert ./tink hardware list
+----+-------------+------------+----------+
| ID | MAC ADDRESS | IP ADDRESS | HOSTNAME |
+----+-------------+------------+----------+
+----+-------------+------------+----------+

Why is this needed

it is more friendly to use and it helps me writing e2e tests #169

@gianarb gianarb added area/setup Issue related to tinkerbell setup size/XS estimate of the amount of work to address the issue labels Jul 17, 2020
gauravgahlot
gauravgahlot previously approved these changes Jul 17, 2020
Copy link
Contributor

@gauravgahlot gauravgahlot left a comment

Choose a reason for hiding this comment

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

A much needed one.

@gauravgahlot gauravgahlot added the ready-to-merge Signal to Mergify to merge the PR. label Jul 17, 2020
I am working at the e2e test for the vagrant setup. It is very useful to
be able to reach tink-server from outside the VM because that where the
logic that asserts the right execution lives.

It is also good for experimentation because I was able to build the tink
cli:

```
go build -o tink ./cmd/tink-cli/main.go
```

I was able to use the tink cli from my local environment. Way more
familiar that `docker exec` as a workflow imho:

```
$ TINKERBELL_GRPC_AUTHORITY=127.0.0.1:42113 TINKERBELL_CERT_URL=http://127.0.0.1:42114/cert ./tink hardware list
+----+-------------+------------+----------+
| ID | MAC ADDRESS | IP ADDRESS | HOSTNAME |
+----+-------------+------------+----------+
+----+-------------+------------+----------+
```
@gauravgahlot gauravgahlot merged commit 11a289b into tinkerbell:master Jul 17, 2020
@mmlb mmlb removed the ready-to-merge Signal to Mergify to merge the PR. label Jan 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/setup Issue related to tinkerbell setup size/XS estimate of the amount of work to address the issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants