Skip to content
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

[CI] Detect if we are running a vm and set the vendor. #11471

Merged
merged 1 commit into from
May 6, 2021

Conversation

mandel-macaque
Copy link
Member

We are using vms for older machines, this results in certain tests that required hardware to fail, how can we skip those? If we find that there is a manufacturer that matches a known VM vendor, we set it in an env var.

Once this is landed, we can change tests to be skipped in vms. I just have 2 vendors I have tested this with, if anyone has another, we can add it.

@mandel-macaque mandel-macaque added the not-notes-worthy Ignore for release notes label May 6, 2021
@mandel-macaque mandel-macaque changed the title [CI] Detect ig we are running a vm and set the vendor. [CI] Detect if we are running a vm and set the vendor. May 6, 2021
Copy link
Member

@dalexsoto dalexsoto left a comment

Choose a reason for hiding this comment

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

👍 🤯

@vs-mobiletools-engineering-service2
Copy link
Collaborator

✅ [PR Build] Tests passed on Build. ✅

Tests passed on Build.

API diff

✅ API Diff from stable

View API diff

API & Generator diff

API Diff (from PR only) (no change)
Generator Diff (no change)

🎉 All 80 tests passed 🎉

Pipeline on Agent XAMBOT-1101.BigSur'
Merge 590f671 into 3b85075

@mandel-macaque mandel-macaque merged commit 5fba609 into xamarin:main May 6, 2021
@mandel-macaque
Copy link
Member Author

/sudo backport d16-10

@vs-mobiletools-engineering-service2
Copy link
Collaborator

Backport Job to branch d16-10 Created! The magic is happening here

@mandel-macaque
Copy link
Member Author

/sudo backport release/6.0.1xx-preview4

@vs-mobiletools-engineering-service2
Copy link
Collaborator

Backport Job to branch release/6.0.1xx-preview4 Created! The magic is happening here

@vs-mobiletools-engineering-service2
Copy link
Collaborator

Oh no! Backport failed! Please see https://devdiv.visualstudio.com/DevDiv/_build/results?buildId=4732073 for more details.

@vs-mobiletools-engineering-service2
Copy link
Collaborator

Hooray! Backport succeeded! Please see https://devdiv.visualstudio.com/DevDiv/_build/results?buildId=4732109 for more details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
not-notes-worthy Ignore for release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants