Skip to content

Conversation

@directhex
Copy link
Contributor

No description provided.

@directhex directhex requested a review from rolfbjarne as a code owner August 6, 2019 14:56
@directhex directhex added run-all-tests Run all our tests. run-internal-tests If tests should be executed on the internal Jenkins instance. skip-api-comparison Skips API / generator diffs when testing pull requests labels Aug 6, 2019
@monojenkins

This comment has been minimized.

@directhex
Copy link
Contributor Author

This is my first iOS rodeo, and I have no idea how to get started from here.

@xamarin-release-manager

This comment has been minimized.

@rolfbjarne
Copy link
Member

The build failed:

[2019-08-13T21:20:05.813Z] /Users/builder/jenkins/workspace/xamarin-macios/xamarin-macios/tools/common/StaticRegistrar.cs(554,38): error CS1061: 'TypeReference' does not contain a definition for 'ConstraintType' and no accessible extension method 'ConstraintType' accepting a first argument of type 'TypeReference' could be found (are you missing a using directive or an assembly reference?) [/Users/builder/jenkins/workspace/xamarin-macios/xamarin-macios/tools/mmp/mmp.csproj]

We're using a Mono.Cecil.dll from the mono archive, so that probably has to be updated.

@directhex
Copy link
Contributor Author

Yeah, I need to update the Mono in the PR to one with the new Cecil, but I don't want to waste time doing that if I'm about to merge an important PR I need to point to

@dotnet dotnet deleted a comment from monojenkins Aug 14, 2019
@dotnet dotnet deleted a comment from monojenkins Aug 14, 2019
@dotnet dotnet deleted a comment from monojenkins Aug 14, 2019
@dotnet dotnet deleted a comment from monojenkins Aug 14, 2019
@directhex
Copy link
Contributor Author

directhex commented Aug 15, 2019

PROGRESS.

We have:

  • At least one HE0038 known issue on WatchOS
  • Undefined _mono_fconv_u8 on WatchOS
  • Authenticode_AsByteArray_Properties failure on WatchOS
  • MonoPosixHelper not present in bundle : /Users/builder/jenkins/workspace/xamarin-macios-pr-builder/tests/mmptest/regression/link-posix-2/bin/Debug/link-posix-2.app/Contents/MonoBundle/libMonoPosixHelper.dylib

@akoeplinger
Copy link
Member

Regarding the mono_fconv_u8 issue that sounds like it was introduced by @filipnavara's recent PR: mono/mono#15930

Probably missing #define MONO_ARCH_EMULATE_FCONV_TO_U8 1 for mini-arm64.h

@directhex
Copy link
Contributor Author

To what extent is 32-bit iOS (i.e. devices considered out of support & unserviceable by Apple as of 2 years ago) still a target?

@chamons
Copy link
Contributor

chamons commented Sep 5, 2019

I know the iPhone 5C is 32-bit, but isn't some of the Apple watches also 32-bit @rolfbjarne / @spouliot ?

@directhex
Copy link
Contributor Author

WatchOS is a different thing to iOS isn't it?

@spouliot
Copy link
Contributor

spouliot commented Sep 5, 2019

@directhex developers still target iOS 32 bits devices - it's fully supported on the app store (and it's even hard to remove architectures from apps until Apple removes them from the store).

@directhex
Copy link
Contributor Author

Huh. TIL.

@mandel-macaque
Copy link
Contributor

mandel-macaque commented Sep 5, 2019

@directhex we are not to worry, I noticed that some tests are allocating TOO much memory on an iOS 32 device. I reported that to mono already and did a cherry-pick of a fix we have in master. We should be ok. This is not the first time that corefx has tests that are memory hungry and cannot be ran on a 32b devices (some File.IO tests were moving gbs from one side to another).

Mono should either fix the tests or add them to the categories that are ignored in mobile. I'm surprised that android did not run on this (unless they do not have 32b devices).

@xamarin-release-manager
Copy link
Collaborator

Build was (probably) aborted

🔥 Jenkins job (on internal Jenkins) failed in stage(s) 'Test run, Test run' 🔥

Build succeeded
✅ Packages: xamarin.ios-12.15.0.344.pkg xamarin.mac-5.15.0.344.pkg
API Diff (from stable)
API Diff (from PR only) (no change)
Generator Diff (no change)
🔥 Test run failed 🔥

Test results

3 tests failed, 168 tests passed.

Failed tests

  • introspection/tvOS - simulator/Debug (tvOS 9.0): Crashed
  • introspection/watchOS 32-bits - simulator/Debug (watchOS 2.0): Crashed
  • MMP Regression Tests/Mac: Failed

@spouliot
Copy link
Contributor

spouliot commented Sep 5, 2019

Since they are now largely fixed I'll be running the submission tests with the 344 packages and mono 6.6.0.48 (as per Make.config). i'll post results here

@monojenkins
Copy link
Collaborator

Build failure
Build succeeded
API Diff (from stable)
❎ Skipped API comparison because the PR has the label 'skip-api-comparison'
🔥 Test run failed 🔥

Test results

3 tests failed, 877 tests' device not found, 167 tests passed.

Failed tests

  • mscorlib/iOS Unified 32-bits - simulator/Debug: Crashed
  • introspection/watchOS 32-bits - simulator/Debug (watchOS 2.0): Crashed
  • MMP Regression Tests/Mac: Failed

@spouliot
Copy link
Contributor

spouliot commented Sep 6, 2019

Submission tests results (and emails to xamappledev) are identical to d16-3 👍

@directhex
Copy link
Contributor Author

@mandel-macaque looks like that latest change caused a conflict

@mandel-macaque
Copy link
Contributor

@directhex fixed ;)

@monojenkins
Copy link
Collaborator

Build failure
Build failed or was aborted

Build succeeded
API Diff (from stable)
❎ Skipped API comparison because the PR has the label 'skip-api-comparison'

@xamarin-release-manager
Copy link
Collaborator

Build was (probably) aborted

🔥 Jenkins job (on internal Jenkins) failed in stage(s) 'Test run' 🔥 : org.jenkinsci.plugins.workflow.steps.FlowInterruptedException

Build succeeded
✅ Packages: xamarin.ios-12.15.0.345.pkg xamarin.mac-5.15.0.345.pkg
API Diff (from stable)
API Diff (from PR only) (no change)
Generator Diff (no change)

@monojenkins
Copy link
Collaborator

Build failure
Build succeeded
API Diff (from stable)
❎ Skipped API comparison because the PR has the label 'skip-api-comparison'
🔥 Test run failed 🔥

Test results

5 tests failed, 877 tests' device not found, 165 tests passed.

Failed tests

  • [NUnit] Mono SystemSecurityTests/watchOS 32-bits - simulator/Debug: Crashed
  • [xUnit] Mono SystemCoreXunit/watchOS 32-bits - simulator/Debug: Crashed
  • [xUnit] Mono SystemNetHttpUnitTestsXunit/watchOS 32-bits - simulator/Debug: Crashed
  • introspection/watchOS 32-bits - simulator/Debug (watchOS 2.0): Crashed
  • MMP Regression Tests/Mac: Failed

@xamarin-release-manager
Copy link
Collaborator

Build was (probably) aborted

🔥 Jenkins job (on internal Jenkins) failed in stage(s) 'Test run, Test run' 🔥

Build succeeded
✅ Packages: xamarin.ios-12.15.0.349.pkg xamarin.mac-5.15.0.349.pkg
API Diff (from stable)
ℹ️ API Diff (from PR only) (please review changes)
Generator Diff (no change)
🔥 Test run failed 🔥

Test results

3 tests failed, 168 tests passed.

Failed tests

  • mscorlib/iOS Unified 32-bits - simulator/Debug: TimedOut
  • introspection/watchOS 32-bits - simulator/Debug (watchOS 2.0): Crashed
  • MMP Regression Tests/Mac: Failed

@mandel-macaque
Copy link
Contributor

@spouliot @directhex looking at https://jenkins.mono-project.com/job/xamarin-macios-pr-builder/11194/Test_20Report/ the 32b issues are solved. I'll investigate why the internal build gave an error.

@xamarin-release-manager
Copy link
Collaborator

Build was (probably) aborted

🔥 Jenkins job (on internal Jenkins) failed in stage(s) 'Test run, Test run' 🔥

Build succeeded
✅ Packages: xamarin.ios-12.15.0.350.pkg xamarin.mac-5.15.0.350.pkg
API Diff (from stable)
API Diff (from PR only) (no change)
Generator Diff (no change)
🔥 Test run failed 🔥

Test results

4 tests failed, 167 tests passed.

Failed tests

  • [NUnit] Mono Mac OS X BCL tests group 2/Mac/Full: Failed (Test run failed.)
  • [NUnit] Mono SystemWebServicesTests/watchOS 32-bits - simulator/Debug: Crashed
  • introspection/watchOS 32-bits - simulator/Debug (watchOS 2.0): Crashed
  • MMP Regression Tests/Mac: Failed

@monojenkins
Copy link
Collaborator

Build failure
Build succeeded
API Diff (from stable)
❎ Skipped API comparison because the PR has the label 'skip-api-comparison'
🔥 Test run failed 🔥

Test results

2 tests failed, 877 tests' device not found, 168 tests passed.

Failed tests

  • introspection/watchOS 32-bits - simulator/Debug (watchOS 2.0): Crashed
  • MMP Regression Tests/Mac: Failed

@mandel-macaque
Copy link
Contributor

MMP tests are expected to fail. Rest are HE003 or a flaky test related to sockets. We are good to go.

@mandel-macaque mandel-macaque merged commit 926372c into master Sep 9, 2019
@steveisok steveisok mentioned this pull request Sep 24, 2019
12 tasks
@rolfbjarne rolfbjarne added the not-notes-worthy Ignore for release notes label Sep 14, 2020
@mandel-macaque mandel-macaque deleted the mono-2019-08 branch December 7, 2021 20:29
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 run-all-tests Run all our tests. run-internal-tests If tests should be executed on the internal Jenkins instance. skip-api-comparison Skips API / generator diffs when testing pull requests

Projects

None yet

Development

Successfully merging this pull request may close these issues.