Skip to content

Added configurable docker build directory for stress test#2599

Merged
1 commit merged intomainfrom
albertcheng/configurable-docker-build
Jan 25, 2022
Merged

Added configurable docker build directory for stress test#2599
1 commit merged intomainfrom
albertcheng/configurable-docker-build

Conversation

@ckairen
Copy link
Copy Markdown
Contributor

@ckairen ckairen commented Jan 22, 2022

No description provided.

@ckairen ckairen added the Central-EngSys This issue is owned by the Engineering System team. label Jan 22, 2022
@ckairen ckairen self-assigned this Jan 22, 2022
@ckairen ckairen changed the title Added configurable docker build directory Added configurable docker build directory for stress test Jan 22, 2022
@ckairen ckairen marked this pull request as ready for review January 22, 2022 00:52
@ckairen ckairen force-pushed the albertcheng/configurable-docker-build branch from 8bd4734 to e19f78b Compare January 24, 2022 19:14
@richardpark-msft
Copy link
Copy Markdown
Member

Do you have a test inside of here somewhere that makes sure this all works? It looks right to me....

@ckairen ckairen force-pushed the albertcheng/configurable-docker-build branch from 637dddd to c2ff979 Compare January 24, 2022 21:25
@ckairen
Copy link
Copy Markdown
Contributor Author

ckairen commented Jan 24, 2022

Do you have a test inside of here somewhere that makes sure this all works? It looks right to me....

No, but I did use your PR's directory/file structure to test it locally.

@azure-sdk
Copy link
Copy Markdown
Collaborator

The following pipelines have been queued for testing:
java - template
java - template - tests
js - template
net - template
net - template - tests
python - template
python - template - tests
You can sign off on the approval gate to test the release stage of each pipeline.
See eng/common workflow

@ghost
Copy link
Copy Markdown

ghost commented Jan 25, 2022

Hello @azure-sdk!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@ghost ghost merged commit 3874513 into main Jan 25, 2022
@ghost ghost deleted the albertcheng/configurable-docker-build branch January 25, 2022 00:32
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Central-EngSys This issue is owned by the Engineering System team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants