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: CI use orka for macos agents #56

Merged
merged 5 commits into from
Aug 4, 2022

Conversation

pazone
Copy link
Contributor

@pazone pazone commented Jul 5, 2022

We are in the transition to use ephemeral workers.

TL;DR
Moved macos build from github actions to the martix build with orka in Jenkinsfile
MacOS will build with go 1.17 only

@pazone pazone requested a review from v1v July 5, 2022 15:03
@pazone pazone force-pushed the chore/ci-use-orka-macos branch from 3101d2b to b71280b Compare July 5, 2022 15:26
@elasticmachine
Copy link
Collaborator

elasticmachine commented Jul 5, 2022

💔 Tests Failed

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2022-07-08T12:37:48.866+0000

  • Duration: 16 min 27 sec

Test stats 🧪

Test Results
Failed 3
Passed 1039
Skipped 0
Total 1042

Test errors 3

Expand to view the tests failures

Test / Matrix - GO_VERSION = '1.13', PLATFORM = 'ubuntu-18 && immutable' / Test / TestMetaPage – github.com/elastic/go-txfile
    Expand to view the error details

     error 
    

    Expand to view the stacktrace

     
            
           
    

Test / Matrix - GO_VERSION = '1.13', PLATFORM = 'ubuntu-18 && immutable' / Test / TestMetaPage/with_checksum – github.com/elastic/go-txfile
    Expand to view the error details

     error 
    

    Expand to view the stacktrace

     
                    layout_test.go:53: 
                	Error Trace:	layout_test.go:53
                	            				mint.go:69
                	Error:      	Received unexpected error:
                	            	invalid operation: checksum mismatch
                	Test:       	TestMetaPage/with_checksum        qc.go:62: quick check rng seed:  1657284091626486981
            qc.go:62: quick check rng seed:  1657284091626627523
            qc.go:62: quick check rng seed:  1657284091626737665
            qc.go:62: quick check rng seed:  1657284091626854111
            qc.go:62: quick check rng seed:  1657284091626965573
            qc.go:62: quick check rng seed:  1657284091627078570
            qc.go:62: quick check rng seed:  1657284091627176085
            qc.go:62: quick check rng seed:  1657284091627273999
            qc.go:62: quick check rng seed:  1657284091627359245
            qc.go:62: quick check rng seed:  1657284091627446288
            qc.go:62: quick check rng seed:  1657284091627512910
            qc.go:62: quick check rng seed:  1657284091627600153
            qc.go:62: quick check rng seed:  1657284091627685677
            qc.go:62: quick check rng seed:  1657284091627752449
            qc.go:62: quick check rng seed:  1657284091627854748
            qc.go:62: quick check rng seed:  1657284091627921460
            qc.go:62: quick check rng seed:  1657284091628014109
            qc.go:62: quick check rng seed:  1657284091628084697
            qc.go:62: quick check rng seed:  1657284091628154541
            qc.go:62: quick check rng seed:  1657284091628221924
            qc.go:62: quick check rng seed:  1657284091628290454
            qc.go:62: quick check rng seed:  1657284091628357713
            qc.go:62: quick check rng seed:  1657284091628422406
            qc.go:62: quick check rng seed:  1657284091628493211
            qc.go:62: quick check rng seed:  1657284091628559313
            qc.go:62: quick check rng seed:  1657284091628625325
            qc.go:62: quick check rng seed:  1657284091628705719
            qc.go:62: quick check rng seed:  1657284091628771976
            qc.go:62: quick check rng seed:  1657284091628847828
            qc.go:62: quick check rng seed:  1657284091628917766
            qc.go:62: quick check rng seed:  1657284091628984385
            qc.go:62: quick check rng seed:  1657284091629052131
           
    

Test / Matrix - GO_VERSION = '1.15', PLATFORM = 'macos11 && x86_64' / Test / [empty] – junit-1.15.xml
  • no error details
  • Expand to view the stacktrace

     Test report file /Users/admin/workspace/Library_go-txfile-mbp_PR-56/src/github.com/elastic/go-txfile/build/junit-1.15.xml was length 0 
    

Steps errors 8

Expand to view the steps failures

Installing golang.org/x/tools/cmd/goimports
  • Took 0 min 4 sec . View more details here
  • Description: go get -u golang.org/x/tools/cmd/goimports
Installing golang.org/x/tools/cmd/goimports
  • Took 0 min 1 sec . View more details here
  • Description: go get -u golang.org/x/tools/cmd/goimports
Installing golang.org/x/tools/cmd/goimports
  • Took 0 min 1 sec . View more details here
  • Description: go get -u golang.org/x/tools/cmd/goimports
go test for 1.13 in ubuntu-18 && immutable
  • Took 1 min 44 sec . View more details here
  • Description: .ci/test.sh
Installing golang.org/x/tools/cmd/goimports
  • Took 0 min 8 sec . View more details here
  • Description: go get -u golang.org/x/tools/cmd/goimports
Installing golang.org/x/tools/cmd/goimports
  • Took 0 min 2 sec . View more details here
  • Description: go get -u golang.org/x/tools/cmd/goimports
Installing golang.org/x/tools/cmd/goimports
  • Took 0 min 2 sec . View more details here
  • Description: go get -u golang.org/x/tools/cmd/goimports
go test for 1.15 in macos11 && x86_64
  • Took 11 min 38 sec . View more details here
  • Description: .ci/test.sh

Copy link
Member

@v1v v1v left a comment

Choose a reason for hiding this comment

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

Remove support for 1.17 since it's not supported based on the CI results

.ci/Jenkinsfile Outdated Show resolved Hide resolved
Copy link
Member

@v1v v1v left a comment

Choose a reason for hiding this comment

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

I also reported some of the errors I managed to find when I enabled the MacOS GitHub action, see #55

I don't know if this project is still active, if you discover any errors that are not related to 1.17, I'd say to raise a GitHub issue with those details, so the project owner can prioritise it accordingly

.ci/Jenkinsfile Outdated Show resolved Hide resolved
.ci/Jenkinsfile Outdated Show resolved Hide resolved
.ci/Jenkinsfile Outdated Show resolved Hide resolved
Co-authored-by: Victor Martinez <[email protected]>
Copy link
Member

@v1v v1v left a comment

Choose a reason for hiding this comment

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

LGTM! Minor comment regarding the description since 1.17 is not in the supported list in the CI at the moment

@pazone pazone merged commit 0e62d33 into elastic:main Aug 4, 2022
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.

3 participants