Title: Enable manylinux1 builds on Pipelines for CI testing
Type: Stage: patch review
Components: Cross-Build Versions: Python 3.8, Python 3.7
Status: open Resolution:
Dependencies: Superseder:
Assigned To: Nosy List: Alex.Willmer, barry, steve.dower, terry.reedy, zach.ware
Priority: normal Keywords: patch, patch, patch

Created on 2019-01-08 04:19 by steve.dower, last changed 2019-01-22 18:50 by steve.dower.

Pull Requests
URL Status Linked Edit
PR 11493 merged steve.dower, 2019-01-10 00:39
PR 11493 merged steve.dower, 2019-01-10 00:39
PR 11493 merged steve.dower, 2019-01-10 00:39
Messages (4)
msg333209 - (view) Author: Steve Dower (steve.dower) * (Python committer) Date: 2019-01-08 04:19
Azure Pipelines can now support container jobs:

I experimented with enabling a manylinux1 build a while back, which should now be able to use identical steps to the POSIX build. With the new syntax, we can enable CI (and perhaps PR?) builds using the snippet below:

- job: ManyLinux1_CI_Tests
  displayName: ManyLinux1 CI Tests
  dependsOn: Prebuild
  condition: |
            eq(variables['manylinux'], 'true')
        eq(dependencies.Prebuild.outputs[''], 'true')

    - container: manylinux1
      image: dockcross:manylinux-x64

    vmImage: ubuntu-16.04

  container: manylinux1

    testRunTitle: '$(build.sourceBranchName)-manylinux1'
    testRunPlatform: manylinux1

  - template: ./posix-steps.yml

I don't have time right now to test this change, but someone else might. It's certainly going to be easier for someone to test it by adding this to the PR build first (or set up a build on your own Pipelines instance).

Maybe there are other more relevant containers we should be testing in?
msg333891 - (view) Author: Steve Dower (steve.dower) * (Python committer) Date: 2019-01-17 18:27
I have made the changes I suggested (though correctly...), but ultimately we need to create our own Docker image suitable for running these tests.

So for now, I'm proposing in my PR to make most of the change, as well as a few other Pipelines/test-related improvements, but to leave this open in case someone wants to come in later with a suitable image. At that point, the only change necessary to enable the tests will be to add a "" script, update the image name/tag and change the manylinux variable to 'true'.

PR 11493 also fixes a missing LICENSE.txt file in the app store package, which was causing an idlelib test to fail as the fallback text only has one line. *Way* too obscure a failure for my liking, but at least we had a test there, so thanks, Terry :)
msg333904 - (view) Author: Terry J. Reedy (terry.reedy) * (Python committer) Date: 2019-01-17 22:30
I believe it was Louie Lu's idea, #30290, to check more than one line.
I am glad it helped.
msg334227 - (view) Author: Steve Dower (steve.dower) * (Python committer) Date: 2019-01-22 18:50
New changeset 28f6cb34f602b9796987904a607dceaf2e4a9e78 by Steve Dower in branch 'master':
bpo-35683: Improve Azure Pipelines steps (GH-11493)
Date User Action Args
2019-01-22 18:50:04steve.dowersetmessages: + msg334227
2019-01-17 22:30:31terry.reedysetkeywords: patch, patch, patch

messages: + msg333904
2019-01-17 18:27:19steve.dowersetkeywords: patch, patch, patch
nosy: + terry.reedy
messages: + msg333891

2019-01-10 00:40:05steve.dowersetkeywords: + patch
stage: patch review
pull_requests: + pull_request11012
2019-01-10 00:39:57steve.dowersetkeywords: + patch
stage: (no value)
pull_requests: + pull_request11011
2019-01-10 00:39:50steve.dowersetkeywords: + patch
stage: (no value)
pull_requests: + pull_request11010
2019-01-08 19:05:03brett.cannonsettitle: Enable manylinux1 builds on Pipelines -> Enable manylinux1 builds on Pipelines for CI testing
2019-01-08 04:19:11steve.dowercreate