Skip to content

Bump maccore to get fix for test-docs. - #5163

Merged
rolfbjarne merged 2 commits into
masterfrom
maccore-issue-1118-v2
Nov 21, 2018
Merged

Bump maccore to get fix for test-docs.#5163
rolfbjarne merged 2 commits into
masterfrom
maccore-issue-1118-v2

Conversation

@rolfbjarne

Copy link
Copy Markdown
Member

Also make it possible to force docs testing by applying a label to pull requests.

Fixes https://github.com/xamarin/maccore/issues/1118.

Commit list for xamarin/maccore:

Diff: https://github.com/xamarin/maccore/compare/7ba9c5a962c1fcdeb651488e7459e89f3c9932fd...6e9b63e53755a74138b78dafd098836a79660edd

Commit list for xamarin/maccore:

* xamarin/maccore@6e9b63e537 Merge pull request #1162 from rolfbjarne/docfixer-fixes
* xamarin/maccore@4cf1d63b7b [docfixer] Create project files and fix a few issues. Fixes #1118.
* xamarin/maccore@714bd73a55 [docfixer] Avoid declaring the same target twice.
* xamarin/maccore@34f4bfa339 [populate] Directories need to call Directory.Exists for existence checks.
* xamarin/maccore@341333db88 [docs] Remove truly ancient and outdated targets to publish updated docs.

Diff: xamarin/maccore@7ba9c5a...6e9b63e
@rolfbjarne rolfbjarne added run-internal-tests If tests should be executed on the internal Jenkins instance. run-docs-tests Runs the test-docs step (only applicable for internal Jenkins) skip-public-jenkins Completely skip execution in the public Jenkins instance labels Nov 21, 2018
@monojenkins

Copy link
Copy Markdown
Contributor

Build success
ℹ️ Skipped execution

@spouliot spouliot left a comment

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.

The current output is not very specific about what happened and clicking on the Tests failed does not show anything red.

@xamarin-release-manager

Copy link
Copy Markdown
Collaborator

Jenkins job (on internal Jenkins) succeeded

Build succeeded
API Diff (from stable)
ℹ️ API Diff (from PR only) (please review changes)
Generator Diff (no change)
Test run succeeded

@rolfbjarne

Copy link
Copy Markdown
Member Author

The current output is not very specific about what happened and clicking on the Tests failed does not show anything red.

There are two issues here:

  1. Nothing here shows up as red at first sight. This is a Jenkins limitation (if we want to continue executing more tests after something fails, we have to capture any errors, and that makes everything show up with green checkmarks). There are bugs filed with Jenkins about this, and they're working on fixing it.
  2. xharness ran out of memory, which somehow made xharness not update test results, even if some tests ran fine (thus the result is just NotStarted: 284, Ignored: 1657). This is filed already: https://github.com/xamarin/maccore/issues/1163

None of these are related to this PR.

@rolfbjarne
rolfbjarne merged commit 6f54cb7 into master Nov 21, 2018
@rolfbjarne
rolfbjarne deleted the maccore-issue-1118-v2 branch November 27, 2018 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run-docs-tests Runs the test-docs step (only applicable for internal Jenkins) run-internal-tests If tests should be executed on the internal Jenkins instance. skip-public-jenkins Completely skip execution in the public Jenkins instance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants