Skip to content

Conversation

@e-sp
Copy link
Contributor

@e-sp e-sp commented Feb 27, 2023

user@work ~/s/piston (update-golang)> ./piston run go -l 1.20.1 packages/go/1.20.1/test.*
== Run ==
STDOUT
OK

HexF
HexF previously approved these changes Feb 28, 2023
@HexF HexF enabled auto-merge February 28, 2023 01:24
@e-sp
Copy link
Contributor Author

e-sp commented Feb 28, 2023

Apparently the test failed due to restraints of its environment:

Running packages/go/1.20.1/test.go with runtime=go
== Run ==
STDERR
# math
    runtime: failed to create new OS thread (have 3 already; errno=11)
    runtime: may need to increase max user processes (ulimit -u)
    fatal error: newosproc

Since test.go basically only contains "hello, world", would it make sense to rerun the test? Or do I need to change something?

package main

import "fmt"

func main() {
	fmt.Println("OK")
}

@e-sp
Copy link
Contributor Author

e-sp commented Mar 3, 2023

@HexF anything I can or should do?

auto-merge was automatically disabled March 4, 2023 15:15

Head branch was pushed to by a user without write access

@e-sp e-sp force-pushed the update-golang branch 7 times, most recently from a419238 to 9d69449 Compare March 5, 2023 10:05
@HexF
Copy link
Collaborator

HexF commented Apr 10, 2023

Might need to up the run timeout for tests to pass

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