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

runtime: unexpected return pc for runtime.sigpanic called on go1.17-devel darwin #46645

Closed
meling opened this issue Jun 8, 2021 · 3 comments
Closed
Labels
FrozenDueToAge WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided.

Comments

@meling
Copy link

meling commented Jun 8, 2021

What version of Go are you using (go version)?

$ go version
go version devel go1.17-c20bcb6488 Tue Jun 8 05:02:19 2021 +0000 darwin/amd64

Does this issue reproduce with the latest release?

No, only devel (tip).

What operating system and processor architecture are you using (go env)?

go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/meling/Library/Caches/go-build"
GOENV="/Users/meling/Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/meling/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/meling/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/Cellar/go/HEAD-c20bcb6/libexec"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/Cellar/go/HEAD-c20bcb6/libexec/pkg/tool/darwin_amd64"
GOVCS=""
GOVERSION="devel go1.17-c20bcb6488 Tue Jun 8 05:02:19 2021 +0000"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/meling/work/quickfeed/go.mod"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/qf/stfl1sbn3p9939ylh7v1fm680000gn/T/go-build3142141964=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?

Ran make test on my quickfeed project. This works fine with go1.16.4.

What did you expect to see?

That the tests pass without the failure below.

What did you see instead?

See bug.txt.
This problem has persisted for 3-6 weeks (don't recall exactly when I first saw this problem).
I don't know where to start debugging this if this is a compatibility problem with my dependencies.
It appears to be limited to the code and tests in the web and web/auth packages, but several of the tests I tried to run individually fail too. Hence, I suspect it is one of the dependencies that is loaded in these packages that are not loaded in other parts of the codebase.

@cherrymui
Copy link
Member

What version of golang.org/x/sys are you using? Could you try updating that? We recently made a fix in the x/sys repo. Thanks.

@cherrymui cherrymui added the WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided. label Jun 8, 2021
@meling
Copy link
Author

meling commented Jun 17, 2021

Upgrading golang.org/x/sys fixed the problem:

go get: upgraded golang.org/x/sys v0.0.0-20210119212857-b64e53b001e4 => v0.0.0-20210616094352-59db8d763f22

Thanks!

@meling meling closed this as completed Jun 17, 2021
@cherrymui
Copy link
Member

Thanks for confirming.

alinefr pushed a commit to escaletech/terraform-provider-kubernetes that referenced this issue Aug 31, 2021
alinefr pushed a commit to escaletech/terraform-provider-kubernetes that referenced this issue Jan 20, 2022
alinefr pushed a commit to escaletech/terraform-provider-kubernetes that referenced this issue Jan 21, 2022
* Check that resource exists before trying to create (hashicorp#1390)

* Docs for ingress_class_name in kubernetes_ingress

* Add timeouts block to kubernetes_manifest (hashicorp#1353)

* Add timeouts block to kubernetes_manifest

This commit adds a timeouts block to the kubernetes_manifest
resource to give parity with resources implemented with the SDK.

See here: https://www.terraform.io/docs/extend/resources/retries-and-customizable-timeouts.html

Co-authored-by: Alex Somesan <alex.somesan@gmail.com>

* Add field_manager block to kubernetes_manifest resource (hashicorp#1333)

This exposes the field manager options available via kubectl by
adding a field_manager block that allows the user to change
the field manager name and force conflicts equivalent to
--field-manager and --force-conflicts.

* Support for "computed" attributes

* Prerequisites for "computed" attributes acceptance test:
* a mutating webhook which adds an annotation to any object
* Terraform configuration to build and deploy the webhook

* Acceptance test for computed attributes

* GH workflow changes for annotations acceptance test

* Allow configuration for 'computed_attributes' in provider block

* computed_attributes reference documentation

* Usage guide for "computed_attributes"

* fix doc linter issues

* Move "computed_attributes" to the "kubernetes_manifest" resource instead of provider block.

* Update documentation to reflect computed_attributes on the resource config.

* Rename "computed_attributes" to "computed_fields"

* fix doc linter errors

* Don't fail when new attribute path is not completely present in previous state

* Add acceptance test

* Discover supported apiVersion for CRD via RESTMapper

* Add import support for kubernetes_manifest

* Acceptance test for kubernetes_manifest import operation

* * test helper for creating ConfigMap resources
* generic test helper for deleting resources

* Docs for importing kubernetes_manifest

* Fix website linter issues

* Remove usage of go-spew

* Add checks for min TF version and feature flag

* key-value, comma-separated import ID + more verbose guidance during
import

* Fix import ID structure in import acceptance test

* Adapt alreadyExists test to use generic delete helper.

* Code review suggestions

* Fix import and some tests

* Release notes for v2.5.0

* Correct changelog header

* v2.5.0

* Fix documentation example for field_manager block (hashicorp#1410)

* Import fixes (hashicorp#1422)

* Include missing computed_attributes value in imported state
* Fix namespace deletion in acceptance test
* Disable client-go deprecation warnings in tests

* Fix failing unit test for parseImportID (hashicorp#1426)

* Copy kubernetes_manifest examples from alpha repository (hashicorp#1401)

* Allow setting kubernetes_job parallelism to zero (hashicorp#1334)

* Fix kubernetes_job "No waiting" documentation example (hashicorp#1383)

* Replace heredoc with jsonencode in docs (hashicorp#1327)

* Add kubernetes_ingress_class resource (hashicorp#1236)

Co-authored-by: John Houston <jhouston@hashicorp.com>

* Fix docs formatting for kubernetes_secret (hashicorp#1434)

* Add build matrix for multiple k8s versions

* Add acceptance test for cert-manager (hashicorp#1417)

* Add immutable field to kubernetes_secret (hashicorp#1280)

Co-authored-by: John Houston <jhouston@hashicorp.com>

* Add behavior field to horizontal_pod_autoscaler (hashicorp#1030)

Co-authored-by: tuannvm <tuannvm@hotmail.com>
Co-authored-by: John Houston <jhouston@hashicorp.com>

* Add proxy_url attribute to provider configuration block (hashicorp#1441)

Co-authored-by: John Houston <jhouston@hashicorp.com>
Co-authored-by: Peter Hrvola <peter.hrvola@wundermanthompson.com>

* Always generate standard ObjectMeta for CRD types

* Acceptance test for CRD with explicit metadata in openAPIV3Schema

* Extract API version retrieval logic into reusable method

* Vendor github.com/Masterminds/semver

* Add cluster version gate to nonstructural CRD test

* Bump terraform-plugin-go to v0.3.1 (hashicorp#1449)

* Update CHANGELOG.md for v2.5.1

* v2.5.1

* Experiment with handling empty blocks

* Filter empty objects unless set by user before API payload generation

* Remove type exceptions for CRD spec.subresources.status. Now covered generically as empty block.

* Fix tests

* Acceptance test for emtpy blocks

* Add missing build tag to test file

* Default kubernetes_manifest experiment to true (hashicorp#1436)

* Hack: work around CRD.spec.versions type corruption

* Acceptance tests for CRD multiversion fix

* Update CHANGELOG for v2.6.0

* v2.6.0

* Update manifest.html.markdown

* Check env and exec blocks are fully known

* Fix import ID syntax in manifest import docs

* Remove reference to beta status of kubernetes_manifest

* Update CHANGELOG.md for v2.6.1

* v2.6.1

* Add support for storage/v1  (hashicorp#1457)

Co-authored-by: Alex Somesan <alex.somesan@gmail.com>

* Add support for networking/v1 (hashicorp#1458)

Co-authored-by: Daniel Sonck <daniel@sonck.nl>
Co-authored-by: Ilia Lazebnik <Ilia.lazebnik@gmail.com>

* Update GH action github/issue-labeler to v2.4

* Add support for certificates/v1 API (hashicorp#1473)

* Add kuberentes_manifest acceptance tests for Service (hashicorp#1486)

Co-authored-by: Phil Sautter <psautter@hashicorp.com>

* Fix logo

* Fix crash when env is empty

* Add aliases for v1 resources (hashicorp#1484)

This commit adds a version suffix to each resource. Most resources are v1, with the exception of PodSecurityPolicy and HorizontalPodAutoscaler which also have v1beta1 and v2beta2 respectively.

This is so Terrafom has an explicitly versioned schema for each resource version, rather than having to include attributes from different API versions in a single schema and figure out which API version needs to be used. This will make the provider easier to maintain, and subsequently to automate the code generation of resource schemas for each API version.

* Update actions/stale version (hashicorp#1502)

* Remove references to 'kubernetes_alpha' provider from Service tests.

* Fix invalid allowedHostPaths PodSecurityPolicy patch

* Update README.md TF logo location

* Add documentation page explaining version suffixes (hashicorp#1504)

* Enable cached RESTMapper

* Bump client-go to v0.21.7

* Don't repeat reading OpenAPI spec after each apply

* Use Go 1.16

* Clean up go.mod

* Fix mishandling of 'null' values in configuration during type morphing

* Handle morphing into same type instead of reusing config values

* Add null attribute to CRD acceptance test to validate correct handling of null

* Update CRD structure assertions

* Add null element case to morph unit test

* Use subcategories to group resources into API groups (hashicorp#1513)

* Update CHANGELOG for v2.7.0 (hashicorp#1514)

* v2.7.0

* Update Github Token in stale issue task (hashicorp#1361)

* Update issue-opened.yml

* Update GHA stale bot issue message (hashicorp#1526)

* Update GHA stale bot issue message

Update what the GHA bot leaves as a comment when marking a stale issue as stale. TH

* refine stale bot message for clarity

* Fix type-morphing of Map into Map

* Test type-morphing of Map to Map

* Test type-morphing of Object to Object

* Update CHANGELOG.md for v2.7.1 release

* v2.7.1

* Update test infrastructure for GKE to match recent Terraform patterns

* GKE test infra: use gcloud as auth provider (hashicorp#1540)

GKE test infra: use gcloud as auth provider

* Fix 1214: update timeout and fix broken link (hashicorp#1527)

* update timeout and fix broken link

* fix linting

* Add mutating_webhook_configuration_v1 data source (hashicorp#1423)

* Update FAQ.md to reflect current team (hashicorp#1542)

* `backend.service.port.name` in `kubernetes_ingress_v1` should be type String  (hashicorp#1541)

Co-authored-by: John Houston <jhouston@hashicorp.com>

* remove enabling experiment section (hashicorp#1564)

* added tests for CSIVolumeSource flattener and expander

* implement flatteners and expanders, and fix tests

* connect CSI Volume flatteners and expanders to pod and update schemas

* update docs

* implement acceptance test for CSI ephemeral volume

* fix: upgrade golang.org/x/sys

Ref: golang/go#46645

* chore: upgrade vendor

* chore: sync deps

Co-authored-by: John Houston <jrhouston@users.noreply.github.com>
Co-authored-by: Aris van Ommeren <avommeren@vxcompany.com>
Co-authored-by: Alex Somesan <alex.somesan@gmail.com>
Co-authored-by: tf-release-bot <terraform@hashicorp.com>
Co-authored-by: Sahal <89029406+sansarspectrum@users.noreply.github.com>
Co-authored-by: Hiieu <6671375+Hiieu@users.noreply.github.com>
Co-authored-by: Andy Li <andy@onthewings.net>
Co-authored-by: Andreas Lindhé <lindhe@users.noreply.github.com>
Co-authored-by: Ilia Lazebnik <Ilia.lazebnik@gmail.com>
Co-authored-by: John Houston <jhouston@hashicorp.com>
Co-authored-by: Sergey Kolodyazhnyy <sergey.kolodyazhnyy@gmail.com>
Co-authored-by: tuannvm <tuannvm@hotmail.com>
Co-authored-by: Peter Hrvola <peter.hrvola@wundermanthompson.com>
Co-authored-by: Daniel Sonck <daniel@sonck.nl>
Co-authored-by: Phil Sautter <psautter@hashicorp.com>
Co-authored-by: Brandy Jackson <90709515+iBrandyJackson@users.noreply.github.com>
Co-authored-by: Will Medlar <will.medlar@nordstrom.com>
Co-authored-by: Aareet Mahadevan <aareet@users.noreply.github.com>
Co-authored-by: Li Wang <li.wang3@fmr.com>
Co-authored-by: Keith Mattix II <keithmattix2@gmail.com>
Co-authored-by: Robin Ketelbuters <robin.ketelbuters@gmail.com>
Co-authored-by: Phil Sautter <20444474+redeux@users.noreply.github.com>
Co-authored-by: Brian Somes <bsomes@excetinc.com>
@golang golang locked and limited conversation to collaborators Jun 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
FrozenDueToAge WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided.
Projects
None yet
Development

No branches or pull requests

3 participants