Skip to content

[js/web] Integrate ONNX Runtime Web CI with BrowserStack#8859

Merged
hanbitmyths merged 2 commits into
masterfrom
sunghcho/browserstack
Sep 2, 2021
Merged

[js/web] Integrate ONNX Runtime Web CI with BrowserStack#8859
hanbitmyths merged 2 commits into
masterfrom
sunghcho/browserstack

Conversation

@hanbitmyths
Copy link
Copy Markdown
Collaborator

It adds ONNX Runtime Web testing on real-device, MacOS, Android and iOS using BrowserStack solution. New CI pipeline yaml is added, which will be triggered by build completion of ONNX Runtime Web CI pipeline. Currently, CI uses two separate parallel BrowserStack jobs due to iOS failure on WebGL. Once the issue is resolved, we can issue only one parallel BrowserStack jobs.

@hanbitmyths hanbitmyths requested a review from a team as a code owner August 26, 2021 19:19
@hanbitmyths hanbitmyths requested a review from fs-eire August 26, 2021 19:20
@hanbitmyths hanbitmyths changed the title Integrate ONNX Runtime Web CI with BrowserStack [js/web] Integrate ONNX Runtime Web CI with BrowserStack Aug 27, 2021
displayName: 'Clean Agent Directories'
condition: always()

- job: build_onnxruntime_web_windows
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I think this job is not necessary for browserstack.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

this job is not running on BrowserStack, but on different agent pool, which supports Firefox and Edge.

@hanbitmyths hanbitmyths requested a review from fs-eire September 1, 2021 06:44
@hanbitmyths hanbitmyths merged commit 332c2ba into master Sep 2, 2021
@hanbitmyths hanbitmyths deleted the sunghcho/browserstack branch September 2, 2021 00:25
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.

2 participants