-
Notifications
You must be signed in to change notification settings - Fork 525
Eric/use mule in jenkins #1072
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
Merged
Merged
Eric/use mule in jenkins #1072
Changes from all commits
Commits
Show all changes
88 commits
Select commit
Hold shift + click to select a range
25c4982
Refactor make and mule in order to execute travis jobs using Mule
bricerisingalgorand 718d0d9
Only use GOPROXY direct
bricerisingalgorand d54ef1d
Move modified ci-* make targets to makefile.mule
bricerisingalgorand 5c2081b
Fix make target for osx jobs
bricerisingalgorand 59c376e
Merge branch 'master' of github.com:algorand/go-algorand into brice/u…
bricerisingalgorand c611355
Merge branch 'master' of github.com:algorand/go-algorand into brice/u…
bricerisingalgorand 7eaeb18
Added Jenkinsfile mule.yaml file for go-algorand.
egieseke 5a6f06b
Updated jenkins file.
egieseke d48d6da
Update mule.yaml with actual tasks and jobs.
egieseke c3562ad
Make timeout for upgrade tests longer
bricerisingalgorand f622224
Limit to mule.yaml to build step.
egieseke e274336
Include test-linux-arm64-shortest.
egieseke 8c4b392
Include test-linux-arm64-integration.
egieseke 8534d7f
Remove build dependencies from ci-integration and ci-test targets.
egieseke 9486c08
Replaced build dependencies for ci-integration and ci-test targets.
egieseke cf0c16e
Added jobs for architectures amd63 and arm32.
egieseke d26fc4c
extended timeouts for e2e_sub tests.
egieseke 842dd76
Remove build task which is done as part of integration testing.
egieseke aa8748d
Remove build task which is done as part of integration testing.
egieseke f43c989
Remove build task which is done as part of integration testing.
egieseke 7bcaf36
Increase timeout for goal network start.
egieseke d37fab4
Increase timeout for goal network create.
egieseke 0fb60e9
Remove timeout for algotmpl calls.
egieseke 7342d7a
add timeout for algotmpl calls.
egieseke 6bf5ff8
Increase wait time to 30 seconds for goal node wait commands in teal …
egieseke 3565f02
Fixed error in expect script.
egieseke 18a16ba
Merge branch 'master' of https://github.com/egieseke/go-algorand into…
egieseke bbab6c0
Fix merge conflict in cicd.centos.Dockerfile.
egieseke 3e676ed
Fix scripting error in createWalletTest.exp.
egieseke ee013ad
Added amd64 build job.
egieseke ab943a0
Increase number of rounds for all transactions of group to confirm in
egieseke d3c2381
Fixed round check to less than in place of equal to case where a roun…
egieseke 6b061b9
Don't exit when account config fails.
egieseke 92f4fc4
Added bin cache upload to build job.
egieseke 0d2c4a5
Added bin cache upload to build job.
egieseke 342daf9
Updated to mule version 0.0.7.
egieseke e483a31
Restrict the scope of file upload to node_pkgs dir.
egieseke 6865290
Add second test job to use bin cache.
egieseke a56c0f1
Added empty suffix.
egieseke 239f312
Added empty suffix.
egieseke e07be1c
Implement Jenkins muleCI compatible mule yaml
bricerisingalgorand 6d60963
Merge branch 'brice/useMuleInTravisJob' of github.com:bricerisingalgo…
bricerisingalgorand 8c7107f
Fix docker ensure file paths
bricerisingalgorand 6c969f0
Change name for stash_id field in muleCi mule.yaml
bricerisingalgorand 23044bb
Bump mule cli version
bricerisingalgorand 9a82fba
Remove arm32 tests from muleCI job
bricerisingalgorand 9cd7806
Update muleCI mule yaml to use multiple globspecs
bricerisingalgorand ff3914a
Bump mule version to 0.0.9
bricerisingalgorand 526250f
Merge branch 'eric/useMuleInJenkins' of github.com:egieseke/go-algora…
bricerisingalgorand 22a943d
Merge pull request #3 from bricerisingalgorand/eric/useMuleInJenkins
egieseke af30896
Include crypto libs in muleCI pipeline stash
bricerisingalgorand 5820354
Add online argument to goal account changeonlinestatus
bricerisingalgorand 77c4f85
Comment out arm32 build
bricerisingalgorand bcd0dfa
Try not running arm go tests one at a time
bricerisingalgorand 840dd4a
Merge branch 'master' of https://github.com/algorand/go-algorand into…
egieseke 7f9d446
Updated comments regarding extending timeouts for tests.
egieseke 9bb32f2
Remove excluded portions of muleCI mule.yaml
bricerisingalgorand 447575a
Reset .travis.yml file back to original travis.yml
bricerisingalgorand f0711ba
Uncomment out individual test exec setting in e2e_go_tests
bricerisingalgorand 72a0a2d
Merge pull request #4 from bricerisingalgorand/eric/useMuleInJenkins
egieseke 057de17
Revert "Don't exit when account config fails."
egieseke 0b32e7a
Change waitForRoundOne timeout back to 1 minute
bricerisingalgorand 436ff22
Use bash booleans for true/false variables.
egieseke 15995dd
Remove PLATFORM_ROOT prior to building package.
egieseke 4d60f9e
Merge branch 'eric/useMuleInJenkins' into eric/useMuleInJenkins
bricerisingalgorand 911275a
Merge pull request #5 from bricerisingalgorand/eric/useMuleInJenkins
egieseke 951c092
Update docker/build/cicd.centos.Dockerfile
egieseke 0852797
Revert extending number of rounds for transaction group test since this
egieseke 9bf729b
Merge branch 'eric/useMuleInJenkins' of https://github.com/egieseke/g…
egieseke ed650f7
Use first index of GOPATH instead of full gopath
bricerisingalgorand 1624e33
Merge branch 'master' of github.com:algorand/go-algorand into eric/us…
bricerisingalgorand d3104e6
Merge branch 'master' of github.com:algorand/go-algorand into eric/us…
bricerisingalgorand 0445102
Remove minimum time requirement in lastpingroundtime
bricerisingalgorand 201d3b6
Merge pull request #6 from bricerisingalgorand/eric/useMuleInJenkins
egieseke 4ff2aa1
Revert fix to create wallet test, moved to a separate PR.
egieseke fecd331
Merge branch 'master' of https://github.com/algorand/go-algorand
egieseke 19b8bb1
Moving this fix to a different PR since it is not related to Jenkins or
egieseke 1942d9d
Use environment variable to set S3_RELEASE_BUCKET.
egieseke 8477840
Add S3_RELEASE_BUCKET environment var to mule.yaml docker jobs.
egieseke b35463d
Added back removal of PKG_ROOT at the end of e2e.sh with conditional on
egieseke 5306f10
Updated verify_at_least_one_running function to loop up to 2 seconds.
egieseke 9ab1df3
Merge branch 'master' of https://github.com/algorand/go-algorand
egieseke 0d4fb90
Merge branch 'master' of https://github.com/egieseke/go-algorand into…
egieseke eff1999
Adjust first timeout round from 5 to 10.
egieseke 3ab549f
Reverted addition of --online flag for goal account changeonlinestatus
egieseke 451a947
Revert to old folder structure for build tarballs
bricerisingalgorand e201fd3
Stop exporting vars in our mule makefile
bricerisingalgorand 6276a5f
Merge pull request #8 from bricerisingalgorand/eric/useMuleInJenkins
egieseke File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
could you provide some background why the GOCACHE need to be modified ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll need to do some digging to find out what the bug was, but we found a bug where go installs fail a small percentage of the time when the go-cache is empty on travis. Since we're not changing the travis job this may actually not be necessary anymore
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can't find where that solution was proposed, but here's an issue about that problem. Apparently this error happens a small percentage of the time on certain types of drive mounts
golang/go#29667
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The issue is this - if you're fixing it here, it's a very partial solution.
Go is also building and using the GOCACHE directory in many other locations as well.
( i.e. in out testing scripts under ./test/scripts/.. )
I don't want to loose this fix if it's happening, but we need to make this change all across the go compiler usage. ( otherwise, you've deployed a fix that only sometimes works )
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That makes sense. Is it alright for us to make a ticket to develop a better solution?
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
sure. you might also want to add some documentation around this one, since it might be irrelevant in future go releases, and we need to know why we placed this one to start with.