Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

removed "rustup" & added new runner tag #9731

Merged
merged 3 commits into from Oct 26, 2018
Merged

Conversation

fevo1971
Copy link
Contributor

@parity-cla-bot
Copy link

It looks like this contributor signed our Contributor License Agreement. 👍

Many thanks,

Parity Technologies CLA Bot

@gabreal
Copy link
Contributor

gabreal commented Oct 10, 2018

this will switch the windows build to the new azure windows 10 runners - in case something goes wrong one can simply apply the windows tag to the old runners 👍

.gitlab-ci.yml Outdated Show resolved Hide resolved
@5chdn 5chdn added this to the 2.2 milestone Oct 10, 2018
@5chdn 5chdn added A0-pleasereview 🤓 Pull request needs code review. A2-insubstantial 👶 Pull request requires no code review (e.g., a sub-repository hash update). B1-patch-beta 🕷 M1-ci 🙉 Continuous integration. B0-patch-stable 🕷 Pull request should also be back-ported to the stable branch. A5-grumble 🔥 Pull request has minor issues that must be addressed before merging. and removed A0-pleasereview 🤓 Pull request needs code review. labels Oct 10, 2018
@General-Beck
Copy link
Contributor

new tags for runners are planned, more standard and understandable. like docker, darwin, windows & shell

Copy link
Contributor

@5chdn 5chdn left a comment

Choose a reason for hiding this comment

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

(just making sure my question is answered before merge)

@fevo1971
Copy link
Contributor Author

ok, i changed the tag to (just) windows and added the new tag to the runner settings of betelgeuse and bellatrix so we can managed the migration from the old to the new runners nodes in gitlab instead of changing the gitlab-ci.yml in the future.

@5chdn
Copy link
Contributor

5chdn commented Oct 12, 2018

Hehe, but why aren't we just applying the rust-windows tag to the new runners? This way we won't need any changes to the current gitlab-ci YML? :)

@ddorgan
Copy link
Collaborator

ddorgan commented Oct 18, 2018

@5chdn good to go now? The tag name was reverted. This is required to test the new windows CI hosts..

Copy link
Contributor

@5chdn 5chdn left a comment

Choose a reason for hiding this comment

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

Thanks

@5chdn 5chdn merged commit 879e730 into master Oct 26, 2018
@5chdn 5chdn deleted the felix-new-windows-runner branch October 26, 2018 11:39
5chdn pushed a commit that referenced this pull request Oct 26, 2018
* removed "rustup" & added new runner tag

* exchanged tag "rust-windows" with "windows"

* revert windows tag change
ordian pushed a commit that referenced this pull request Oct 26, 2018
* removed "rustup" & added new runner tag

* exchanged tag "rust-windows" with "windows"

* revert windows tag change
5chdn added a commit that referenced this pull request Oct 28, 2018
* version: bump parity beta to 2.1.4

* ethcore: bump ropsten forkblock checkpoint (#9775)

* ethcore: handle vm exception when estimating gas (#9615)

* removed "rustup" & added new runner tag (#9731)

* removed "rustup" & added new runner tag

* exchanged tag "rust-windows" with "windows"

* revert windows tag change

* sync: retry different peer after empty subchain heads response (#9753)

* If no subchain heads then try a different peer

* Add log when useless chain head

* Restrict ChainHead useless peer to ancient blocks

* sync: replace `limit_reorg` with `block_set` condition

* update jsonrpc-core to a1b2bb742ce16d1168669ffb13ffe856e8131228

* Allow zero chain id in EIP155 signing process (#9792)

* Allow zero chain id in EIP155 signing process

* Rename test

* Fix test failure

* Insert dev account before unlocking (#9813)
5chdn added a commit that referenced this pull request Oct 28, 2018
* version: bump parity stable to 2.0.9

* ethcore: bump ropsten forkblock checkpoint (#9775)

* ethcore: handle vm exception when estimating gas (#9615)

* update jsonrpc-core to a1b2bb742ce16d1168669ffb13ffe856e8131228

* removed "rustup" & added new runner tag (#9731)

* removed "rustup" & added new runner tag

* exchanged tag "rust-windows" with "windows"

* revert windows tag change

* Allow zero chain id in EIP155 signing process (#9792)

* Allow zero chain id in EIP155 signing process

* Rename test

* Fix test failure

* Insert dev account before unlocking (#9813)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A2-insubstantial 👶 Pull request requires no code review (e.g., a sub-repository hash update). A5-grumble 🔥 Pull request has minor issues that must be addressed before merging. B0-patch-stable 🕷 Pull request should also be back-ported to the stable branch. M1-ci 🙉 Continuous integration.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants