Skip to content

Fix cosmos CI build failures caused by dev_setup.py changes#5310

Merged
scbedd merged 3 commits intoAzure:masterfrom
scbedd:fix-cosmos-ci
May 20, 2019
Merged

Fix cosmos CI build failures caused by dev_setup.py changes#5310
scbedd merged 3 commits intoAzure:masterfrom
scbedd:fix-cosmos-ci

Conversation

@scbedd
Copy link
Member

@scbedd scbedd commented May 17, 2019

Simple change! From here on out will grep dev_setup.py through the repo on any change to the input strategy. This kind of thing shouldn't happen again.

@scbedd scbedd requested a review from bryevdv May 17, 2019 22:07
@scbedd scbedd requested a review from lmazuel as a code owner May 17, 2019 22:07
@adxsdk6
Copy link

adxsdk6 commented May 17, 2019

Can one of the admins verify this patch?

@bryevdv
Copy link
Contributor

bryevdv commented May 20, 2019

@scbedd actually, are the Cosmos emulator tests running here? I don't seem to see them in the output above but maybe I am missing something?

@scbedd
Copy link
Member Author

scbedd commented May 20, 2019

@scbedd actually, are the Cosmos emulator tests running here? I don't seem to see them in the output above but maybe I am missing something?

Yep, needed to update the dev_setup.py call in two locations. Bad assumptions and half-attention.

@scbedd scbedd merged commit 180640d into Azure:master May 20, 2019
xiangyan99 added a commit that referenced this pull request May 20, 2019
* add missing import (#5340)

* Fix cosmos CI build failures caused by dev_setup.py changes (#5310)

* fixing pipeline issues caused by changes to dev_setup. both in standard and nightly tests.

* Adds a event hubs PR pipeline YAML. (#5297)

* [AutoPR] hanaonazure/resource-manager (#5275)

* Generated from 5c4ec331dfcb15ae3c469ab9fba625800df0dded (#5165)

Added new SKUs

* Move HanaOnAzure to SDK folder

* Generated from 5e0b04675163b24ef5fd29c3455d0bb121e4465b (#5343)

Hanaonazure on SDK folder

* ChangeLog
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