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

attaching to serial console is not working #882

Closed
RaitoBezarius opened this issue Apr 20, 2020 · 3 comments · Fixed by moul/gotty-client#77 or #960
Closed

attaching to serial console is not working #882

RaitoBezarius opened this issue Apr 20, 2020 · 3 comments · Fixed by moul/gotty-client#77 or #960

Comments

@RaitoBezarius
Copy link

RaitoBezarius commented Apr 20, 2020

Whenever I try to do scw attach or scw run --attach, it attaches to nothing, i.e. "You are connected", wait some seconds and return the shell.
I'm trying to make work https://github.com/moul/scaleway-ipxe to have NixOS install through iPXE, but I have this error:

spawn slow-stream --raw -b 1 -i 10 --stdout-passthrough -- scw run --attach --commercial-type=VC1S 50G
You are connected, type 'Ctrl+q' to quit.
panic: provided file is not a console

goroutine 40 [running]:
github.com/scaleway/scaleway-cli/vendor/github.com/containerd/console.Current(...)
	vendor/github.com/containerd/console/console.go:67
github.com/scaleway/scaleway-cli/vendor/github.com/moul/gotty-client.(*Client).Loop(0xc00044a380, 0x0, 0x0)
	vendor/github.com/moul/gotty-client/gotty-client.go:313 +0x302
github.com/scaleway/scaleway-cli/pkg/utils.AttachToSerial.func1(0xc00044a380, 0xc00022c4e0)
	pkg/utils/utils.go:221 +0x2d
created by github.com/scaleway/scaleway-cli/pkg/utils.AttachToSerial
	pkg/utils/utils.go:220 +0x369
ERRO[0000] exit status 2                                
send: spawn id exp4 not open
    while executing
"send   "\x02""
    (file "./scw-ipxe.expect" line 27)

I'm not sure how to diagnose this.

@Sh4d1
Copy link
Contributor

Sh4d1 commented Apr 21, 2020

Hello @RaitoBezarius ! After some digging it looks like it's because of changes in some vendors. Version 1.19 still works so I encourage you to use this version intead of the 1.20 if you want to make it work. Tough it's an issue on moul/gotty-client not on the cli itself. I'll try to fix it if I find some time though!

@Sh4d1
Copy link
Contributor

Sh4d1 commented Apr 21, 2020

@RaitoBezarius need moul/gotty-client#77 merged then we'll bump the dep on the cli :)

@RaitoBezarius
Copy link
Author

RaitoBezarius commented Apr 27, 2020

Thank you a lot for your fast response (& fix), @Sh4d1 I really appreciate it :)
I'll wait for your PR to be merged upstream

clement-gilbert pushed a commit to clement-gilbert/scaleway-cli that referenced this issue Mar 3, 2022
clement-gilbert pushed a commit to clement-gilbert/scaleway-cli that referenced this issue Mar 3, 2022
* feat(lb): Add LB Private Network support

* feat(lb): Add LB Route support

* fixup! feat(lb): Add LB Private Network support

* fix: increase rdb instance creation timeout to 15m (scaleway#868)

Co-authored-by: Antoine Barbare <[email protected]>

* doc: fix broken links following the release of the LB Zoned API (scaleway#871)

* fix(docs) wrong name for resource iot_device (scaleway#869)

Replace "scaleway_iot_hub_device" with "scaleway_iot_device"

Co-authored-by: Rémy Léone <[email protected]>

* chore: add a SECURITY.md (scaleway#872)

* feat(domain): add record resource (scaleway#854)

Co-authored-by: Jeremy JACQUEMIN <[email protected]>

* doc: fix default runtime on k8s_pool (scaleway#873)

* feat(iot): Add ability to set user device certificate (scaleway#859)

* feat(iot): add ability to set user device certificate

* test(iot): add test for user device certificate

* test(iot): use unique resource names in tests to prevent conflict between tests set

* test(iot): update cassettes for iot-device-minimal

* chore: add a SECURITY.md (scaleway#872)

* feat(domain): add record resource (scaleway#854)

Co-authored-by: Jeremy JACQUEMIN <[email protected]>

* doc: fix default runtime on k8s_pool (scaleway#873)

Co-authored-by: Jerome Malinge <[email protected]>
Co-authored-by: Rémy Léone <[email protected]>
Co-authored-by: jerjako <[email protected]>
Co-authored-by: Jeremy JACQUEMIN <[email protected]>
Co-authored-by: Jérémy THERIN <[email protected]>
Co-authored-by: jaime Bernabe <[email protected]>

* feat(instance): Adding the datasource for instance ip (scaleway#870)

* feat(instance): Adding the datasource for instance ip

* refactoring: using ParseIP to validate the IP

* fix: use proper validation

* minor fixes

* style(tfproviderlint): S019: schema should omit Computed, Optional, or Required set to false

* fix: avoiding one api call if the ID is passed

Co-authored-by: Rémy Léone <[email protected]>
Co-authored-by: jaime Bernabe <[email protected]>

* feat(lb): add data source for lb (scaleway#760)

* feat(lb): add data source for lb

* Fix

* Update scaleway/data_source_lb.go

Co-authored-by: Jerome Quere <[email protected]>

* Update scaleway/data_source_lb.go

Co-authored-by: Jerome Quere <[email protected]>

* Update docs/data-sources/lb.md

Co-authored-by: Quentin Brosse <[email protected]>

* Fix

* Fix

* Fix

* feat(lb): remove repeated explanation

* feat(lb): update lb datasource reader with zone

* feat(lb): update lb datasource reader with zone and test

Co-authored-by: Jerome Quere <[email protected]>
Co-authored-by: Quentin Brosse <[email protected]>
Co-authored-by: jaime Bernabe <[email protected]>
Co-authored-by: Jaime BERNABE <[email protected]>

* feat(vpcgw): add support for DHCP (scaleway#865)

* feat(vpcgw): add support for DHCP

* Fix

* Fix

* Fix

* Fix

* feat(lb): update vpc gw dhcp and small corrections

* feat(vpc_public_gateway_dhcp): fix doc and test

Co-authored-by: jaime Bernabe <[email protected]>
Co-authored-by: Jaime BERNABE <[email protected]>

* feat(domain): Add domain record data source (scaleway#874)

* feat(domain): Add domain record data source

* fix tests

Co-authored-by: Jeremy JACQUEMIN <[email protected]>
Co-authored-by: jaime Bernabe <[email protected]>

* chore: add support for dependabot (scaleway#877)

* chore(deps): bump github.com/aws/aws-sdk-go from 1.39.3 to 1.40.53 (scaleway#878)

Bumps [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) from 1.39.3 to 1.40.53.
- [Release notes](https://github.com/aws/aws-sdk-go/releases)
- [Changelog](https://github.com/aws/aws-sdk-go/blob/main/CHANGELOG.md)
- [Commits](aws/aws-sdk-go@v1.39.3...v1.40.53)

---
updated-dependencies:
- dependency-name: github.com/aws/aws-sdk-go
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* chore(deps): bump github.com/hashicorp/terraform-plugin-sdk/v2 (scaleway#880)

Bumps [github.com/hashicorp/terraform-plugin-sdk/v2](https://github.com/hashicorp/terraform-plugin-sdk) from 2.7.0 to 2.8.0.
- [Release notes](https://github.com/hashicorp/terraform-plugin-sdk/releases)
- [Changelog](https://github.com/hashicorp/terraform-plugin-sdk/blob/main/CHANGELOG.md)
- [Commits](hashicorp/terraform-plugin-sdk@v2.7.0...v2.8.0)

---
updated-dependencies:
- dependency-name: github.com/hashicorp/terraform-plugin-sdk/v2
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Rémy Léone <[email protected]>

* chore(deps): bump github.com/dnaeon/go-vcr from 1.1.0 to 1.2.0 (scaleway#879)

Bumps [github.com/dnaeon/go-vcr](https://github.com/dnaeon/go-vcr) from 1.1.0 to 1.2.0.
- [Release notes](https://github.com/dnaeon/go-vcr/releases)
- [Commits](dnaeon/go-vcr@v1.1.0...v1.2.0)

---
updated-dependencies:
- dependency-name: github.com/dnaeon/go-vcr
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Rémy Léone <[email protected]>

* chore(deps): bump github.com/aws/aws-sdk-go from 1.40.53 to 1.40.54 (scaleway#881)

Bumps [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) from 1.40.53 to 1.40.54.
- [Release notes](https://github.com/aws/aws-sdk-go/releases)
- [Changelog](https://github.com/aws/aws-sdk-go/blob/main/CHANGELOG.md)
- [Commits](aws/aws-sdk-go@v1.40.53...v1.40.54)

---
updated-dependencies:
- dependency-name: github.com/aws/aws-sdk-go
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* feat(provider): add profile name from configuration file support (scaleway#875)

* feat(provider): add profile name from configuration file support

* feat(provider): enhance error handling

* chore(provider): add section about new profile feature usage

* fix(provider): fix condition

* Apply suggestions from code review

Co-authored-by: jaime Bernabe <[email protected]>

* feat(lb): test for private network

* fix(lb): linter correction

* fix(lb): lint correction

* fix(lb): lint correction

* chore(deps): bump github.com/aws/aws-sdk-go from 1.40.54 to 1.40.55 (scaleway#883)

Bumps [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) from 1.40.54 to 1.40.55.
- [Release notes](https://github.com/aws/aws-sdk-go/releases)
- [Changelog](https://github.com/aws/aws-sdk-go/blob/main/CHANGELOG.md)
- [Commits](aws/aws-sdk-go@v1.40.54...v1.40.55)

---
updated-dependencies:
- dependency-name: github.com/aws/aws-sdk-go
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* chore: bump go version (scaleway#882)

* k8s: add type on cluster (scaleway#886)

* fix(vpc): reverse issue (scaleway#887)

* fix(vpc): reverse issue

* fix(vpc): add test to check reverse and address

* fix(vpc): double retry interval

Co-authored-by: scaleway-bot <[email protected]>

* chore(deps): bump github.com/aws/aws-sdk-go from 1.40.55 to 1.40.56 (scaleway#888)

Bumps [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) from 1.40.55 to 1.40.56.
- [Release notes](https://github.com/aws/aws-sdk-go/releases)
- [Changelog](https://github.com/aws/aws-sdk-go/blob/main/CHANGELOG.md)
- [Commits](aws/aws-sdk-go@v1.40.55...v1.40.56)

---
updated-dependencies:
- dependency-name: github.com/aws/aws-sdk-go
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: jaime Bernabe <[email protected]>

* fix(instance): security group rule read from api and acl database not statefull (scaleway#892)

* Convert acl_rules from list to set

* Fix read security group rules, removed from api

* fix update typeSet for acl_rules

* change order

Co-authored-by: Rémy Léone <[email protected]>

* chore(deps): bump github.com/aws/aws-sdk-go from 1.40.56 to 1.40.59 (scaleway#893)

Bumps [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) from 1.40.56 to 1.40.59.
- [Release notes](https://github.com/aws/aws-sdk-go/releases)
- [Changelog](https://github.com/aws/aws-sdk-go/blob/main/CHANGELOG.md)
- [Commits](aws/aws-sdk-go@v1.40.56...v1.40.59)

---
updated-dependencies:
- dependency-name: github.com/aws/aws-sdk-go
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: jaime Bernabe <[email protected]>

* feat(vpcgw): add gateway network resource (scaleway#891)

Co-authored-by: scaleway-bot <[email protected]>

* chore(deps): bump github.com/aws/aws-sdk-go from 1.40.59 to 1.41.0 (scaleway#894)

Bumps [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go) from 1.40.59 to 1.41.0.
- [Release notes](https://github.com/aws/aws-sdk-go/releases)
- [Changelog](https://github.com/aws/aws-sdk-go/blob/main/CHANGELOG.md)
- [Commits](aws/aws-sdk-go@v1.40.59...v1.41.0)

---
updated-dependencies:
- dependency-name: github.com/aws/aws-sdk-go
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* feat(lb): add wait lb in every resource

* fix(lb): add test steps

* feat(lb): add test and change schema

* feat(lb): add test for dhcp

* feat(lb): add cassette

* feat(lb): add cassette basic and dhcp. add doc. add test dhcp

* fix(lb_pn): lint

* Remove trailing spaces

* fix(lb_certificate): remove wait on lb_certificate

* feat(lb): add release_ip to data_source

* fix(lbacl): fix test lb acl

* fix(lb-acl): update cassettes lb acl

* fix(lb-pn): fix cassette

* fix(lb-route): cassette

Co-authored-by: abarbare <[email protected]>
Co-authored-by: Antoine Barbare <[email protected]>
Co-authored-by: Jean-Sébastien Hedde <[email protected]>
Co-authored-by: jerousseau <[email protected]>
Co-authored-by: Rémy Léone <[email protected]>
Co-authored-by: jerjako <[email protected]>
Co-authored-by: Jeremy JACQUEMIN <[email protected]>
Co-authored-by: Jérémy THERIN <[email protected]>
Co-authored-by: grom-42 <[email protected]>
Co-authored-by: Jerome Malinge <[email protected]>
Co-authored-by: jaime Bernabe <[email protected]>
Co-authored-by: Jerome Quere <[email protected]>
Co-authored-by: Quentin Brosse <[email protected]>
Co-authored-by: Jaime BERNABE <[email protected]>
Co-authored-by: jaime Bernabe <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Alexandre Bouthinon <[email protected]>
Co-authored-by: scaleway-bot <[email protected]>
Co-authored-by: LucasBoisserie <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants