-
Notifications
You must be signed in to change notification settings - Fork 19
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
[GPII-4195] Add 1909 and LTSC Windows 10 version to Vagrantfile #192
base: master
Are you sure you want to change the base?
Conversation
CI job failed: https://ci.gpii.net/job/gpii-app-tests/1066/ |
ok to test @javihernandez could you take a look to the errors in the provisioning step?. You can replicate them running the following:
|
CI job failed: https://ci.gpii.net/job/gpii-app-tests/1067/ |
ok to test |
I was having exactly the same errors on Friday, I checked choco packages, env variables, build tools, etc ... couldn't find anything suspicious. Let's see how it goes on CI today. |
CI job failed: https://ci.gpii.net/job/gpii-app-tests/1070/ |
Right, today we're not facing the dynamic link library problem, building right both on 1909 and LTSC, see:
and
Errors on code analysis and tests are expected:
And should disappear after GPII/ci-service#75 is merged |
CI job failed: https://ci.gpii.net/job/gpii-app-tests/1071/ |
I've been running the tests and so far, they pass as expected.
So far, LGTM |
CI job failed: https://ci.gpii.net/job/gpii-app-tests/1088/ |
CI job failed: https://ci.gpii.net/job/gpii-app-tests/1089/ |
ok to test |
CI job failed: https://ci.gpii.net/job/gpii-app-tests/1090/ |
https://issues.gpii.net/browse/GPII-4195
This PR depends on GPII/ci-service#75. Both should be merged at the same time, otherwise the CI will fail.