[issue39837] Remove Azure Pipelines from GitHub PRs

[STINNER Victor]

STINNER Victor  added the comment:

Another issue: I still see "Azure Pipelines PR Expected — Waiting for status to 
be reported" 15 min after I created my PR :-/ Technically, I created the PR and 
then pushed a second commit to the PR. The only option is to close/reopen the 
PR to re-trigger *all* CIs :-/

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[hai shi]

hai shi  added the comment:

> Sadly (again), closing/reopening a PR re-runs all CIs. At the first run, GH 
> Action macOS job passed. At the second run, the "Tests" step of GH Action 
> macOS job, but I'm clueless with its logs: (...)

Oh, I encountered the same trouble twice :(
https://github.com/python/cpython/runs/628458496

--
nosy: +shihai1991

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[STINNER Victor]

STINNER Victor  added the comment:

> Sadly (again), closing/reopening a PR re-runs all CIs. At the first run, GH 
> Action macOS job passed. At the second run, the "Tests" step of GH Action 
> macOS job, but I'm clueless with its logs: (...)

Oops, I looked at two different PRs. In fact, the two CI failures are unrelated.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[STINNER Victor]

STINNER Victor  added the comment:

> Sadly, I had to close/reopen my PR since there is no button to only restart 
> the failure job, or even restart all Azure Pipeline jobs. This retrigger all 
> CI jobs :-(

Sadly (again), closing/reopening a PR re-runs all CIs. At the first run, GH 
Action macOS job passed. At the second run, the "Tests" step of GH Action macOS 
job, but I'm clueless with its logs:

https://github.com/python/cpython/pull/19776/checks?check_run_id=627916923

2020-04-28T23:33:03.5559341Z ##[section]Starting: Request a runner to run this 
job
2020-04-28T23:33:04.1879823Z Requesting a hosted runner in current repository's 
account/organization with labels: 'macos-latest', require runner match: True
2020-04-28T23:33:04.2799035Z Labels matched hosted runners has been found, 
waiting for one of them get assigned for this job.
2020-04-28T23:33:04.3183702Z ##[section]Finishing: Request a runner to run this 
job

On the web UI, I see that 6 steps completed, only the last "Tests" step failed. 
But can't I see logs of other steps?

I would prefer to be able to merge a PR even when Azure Pipelines fails: make 
the job optional.

Hopefully, GH Action macOS job is optional and so I can merge my PR ;-)

Note: I'm not sure if it's the right place to report GH Action macOS failure, 
but it seems to be related to Azure Pipelines.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[STINNER Victor]

STINNER Victor  added the comment:

Another Azure Pipeline failure on my 
https://github.com/python/cpython/pull/19769 PR, it looks like a random 
networking failure.

Sadly, I had to close/reopen my PR since there is no button to only restart the 
failure job, or even restart all Azure Pipeline jobs. This retrigger all CI 
jobs :-(

https://dev.azure.com/Python/cpython/_build/results?buildId=61891=logs=c83831cd-3752-5cc7-2f01-8276919eb334=6bd3fafc-d115-560e-4a08-fa9326c4b5c7

The win64 job of Azure Pipelines PR fails to build Python because it failed to 
fetch bzip2:

Fetching bzip2-1.0.6...
(...)
  File "C:\hostedtoolcache\windows\Python\3.8.2\x64\lib\http\client.py", line 
915, in connect
self.sock = self._create_connection(
  File "C:\hostedtoolcache\windows\Python\3.8.2\x64\lib\socket.py", line 787, 
in create_connection
for res in getaddrinfo(host, port, 0, SOCK_STREAM):
  File "C:\hostedtoolcache\windows\Python\3.8.2\x64\lib\socket.py", line 918, 
in getaddrinfo
for res in _socket.getaddrinfo(host, port, family, type, proto, flags):
socket.gaierror: [Errno 11001] getaddrinfo failed
(...)
  File "C:\hostedtoolcache\windows\Python\3.8.2\x64\lib\urllib\request.py", 
line 1362, in https_open
return self.do_open(http.client.HTTPSConnection, req,
  File "C:\hostedtoolcache\windows\Python\3.8.2\x64\lib\urllib\request.py", 
line 1322, in do_open
raise URLError(err)
urllib.error.URLError: 
(...)
c1 : fatal error C1083: Cannot open source file: 
'd:\a\1\b\externals\bzip2-1.0.6\blocksort.c': No such file or directory 
[D:\a\1\s\PCbuild\_bz2.vcxproj]
c1 : fatal error C1083: Cannot open source file: 
'd:\a\1\b\externals\bzip2-1.0.6\bzlib.c': No such file or directory 
[D:\a\1\s\PCbuild\_bz2.vcxproj]
c1 : fatal error C1083: Cannot open source file: 
'd:\a\1\b\externals\bzip2-1.0.6\compress.c': No such file or directory 
[D:\a\1\s\PCbuild\_bz2.vcxproj]
c1 : fatal error C1083: Cannot open source file: 
'd:\a\1\b\externals\bzip2-1.0.6\crctable.c': No such file or directory 
[D:\a\1\s\PCbuild\_bz2.vcxproj]
c1 : fatal error C1083: Cannot open source file: 
'd:\a\1\b\externals\bzip2-1.0.6\decompress.c': No such file or directory 
[D:\a\1\s\PCbuild\_bz2.vcxproj]
c1 : fatal error C1083: Cannot open source file: 
'd:\a\1\b\externals\bzip2-1.0.6\huffman.c': No such file or directory 
[D:\a\1\s\PCbuild\_bz2.vcxproj]
c1 : fatal error C1083: Cannot open source file: 
'd:\a\1\b\externals\bzip2-1.0.6\randtable.c': No such file or directory 
[D:\a\1\s\PCbuild\_bz2.vcxproj]

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[STINNER Victor]

STINNER Victor  added the comment:

> There is another issue: sometimes, the Ubuntu VM fails to download .deb from 
> azure.archive.ubuntu.com.

I created bpo-40188: "Azure Pipelines jobs failing randomly with: Unable to 
connect to azure.archive.ubuntu.com".

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[Brett Cannon]

Brett Cannon  added the comment:

> I'm not aware of Travis CI current issue. There were issues in the past, as 
> with any CI, right ;-) Travis CI looks quite reliable these days.

That's what everyone said when Travis was required and before it went flaky the 
last time. ;)

The point is I don't want to keep flipping on and off required checks based on 
whatever CI people deem flaky or not at any one time.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[Steve Dower]

Steve Dower  added the comment:

> I don't understand how it's possible that the Ubuntu job passed in the GitHub 
> action, but failed in Azure Pipelines.

Random network issues, most likely. Those are about the only flakiness I've 
seen in recent times.

The OpenSSL cache should only be for the from-source build we do on Ubuntu for 
CI. It uses the multissl script in our repo and downloads directly from 
openssl.org. The dependencies obtained using apt are not cached by us.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[STINNER Victor]

STINNER Victor  added the comment:

> About the "Install Dependencies" issue, I created bpo-40146: "Upgrade Azure 
> Pipelines to OpenSSL 1.1.1f".

I fixed this issue.

There is another issue: sometimes, the Ubuntu VM fails to download .deb from 
azure.archive.ubuntu.com.

Example:
https://github.com/python/cpython/pull/19282/checks?check_run_id=553744077

---
Err:1 http://azure.archive.ubuntu.com/ubuntu bionic/universe amd64 lcov all 
1.13-3
  Could not connect to azure.archive.ubuntu.com:80 (52.177.174.250), connection 
timed out
---

This error is followed by many similar errors.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[STINNER Victor]

STINNER Victor  added the comment:

About the "Install Dependencies" issue, I created bpo-40146: "Upgrade Azure 
Pipelines to OpenSSL 1.1.1f".

I don't understand how it's possible that the Ubuntu job passed in the GitHub 
action, but failed in Azure Pipelines. Maybe there was a download cache 
somehow, and the cache is now outdated.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[STINNER Victor]

STINNER Victor  added the comment:

> Yes, but I don't want to to do that as we have had equivalent flakiness 
> issues with Travis which is why it isn't required ATM.

I'm not aware of Travis CI current issue. There were issues in the past, as 
with any CI, right ;-) Travis CI looks quite reliable these days.

Whereas the Docs and Ubuntu GH Action job of Azure failed to install 
dependencies :-(

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[Steve Dower]

Steve Dower  added the comment:

Or we could remove the path filter on GitHub Actions so that all checks run 
even for doc-only changes? Then they can be marked as required.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[Brett Cannon]

Brett Cannon  added the comment:

> Is it possible to make Travis CI required and make Azure Pipelines not 
> required?

Yes, but I don't want to to do that as we have had equivalent flakiness issues 
with Travis which is why it isn't required ATM.

The only way to prevent flaky CI from blocking anything is to simply make no CI 
required and trust core devs not to merge unless they are certain they know why 
a CI run failed (although I don't know what that does to miss-islington). 
Passed that is being extremely specific about what CI is considered stable 
enough to block on an would probably need to be down to the OS level on top of 
what is being tested.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[STINNER Victor]

STINNER Victor  added the comment:

I cannot merge a PR until it completes. It re-runs jobs which are already run 
as GH Actions.

There is another annoying issue with Azure Pipelines. When a job fails randomly 
for whatever reason, a job cannot be re-run, even if I log in Microsoft Azure.

Usually, the workaround is to close/reopen a PR to re-run all CIs.

Except that for a backport PR created automatically by miss-islington bot, when 
I close the PR, the bot removes its branch and so the PR cannot be re-open.

Well, the second workaround is to ask the bot to create a new PR backport. That 
what I did.

I did that for PR 19276 of bpo-40121. It's annoying to have to use *two* 
workarounds.

On the other side, Travis CI is not currently required, I don't understand why.

Is it possible to make Travis CI required and make Azure Pipelines not required?

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[Ned Deily]

Ned Deily  added the comment:


New changeset b092892f9afd37fd3355b1e91fc5835f196ea914 by Ned Deily (Miss 
Islington (bot)) in branch '3.7':
bpo-39837: Disable macOS tests on Azure Pipelines (GH-18818)
https://github.com/python/cpython/commit/b092892f9afd37fd3355b1e91fc5835f196ea914


--
nosy: +ned.deily

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[Roundup Robot]

Change by Roundup Robot :


--
nosy: +python-dev
nosy_count: 4.0 -> 5.0
pull_requests: +18192
pull_request: https://github.com/python/cpython/pull/18835

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[miss-islington]

miss-islington  added the comment:


New changeset d692d52f4aaeb6feaabb18f18d49907dd578fbeb by Miss Islington (bot) 
in branch '3.8':
bpo-39837: Disable macOS tests on Azure Pipelines (GH-18818)
https://github.com/python/cpython/commit/d692d52f4aaeb6feaabb18f18d49907dd578fbeb


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[miss-islington]

miss-islington  added the comment:


New changeset 47b7c227048f2cb019cc3ec2fef7e867f1b232f3 by Miss Islington (bot) 
in branch '3.7':
bpo-39837: Disable macOS tests on Azure Pipelines (GH-18818)
https://github.com/python/cpython/commit/47b7c227048f2cb019cc3ec2fef7e867f1b232f3


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[Steve Dower]

Steve Dower  added the comment:

I've disabled macOS builds on Pipelines, so now they're essentially advisory 
through the GitHub Actions build.

I also pinged some contacts about the not-very-useful behaviour of required 
checks vs. path filters. So will see what they say.

--
versions: +Python 3.7, Python 3.8

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[miss-islington]

Change by miss-islington :


--
nosy: +miss-islington
nosy_count: 3.0 -> 4.0
pull_requests: +18178
pull_request: https://github.com/python/cpython/pull/18820

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[miss-islington]

Change by miss-islington :


--
pull_requests: +18179
pull_request: https://github.com/python/cpython/pull/18821

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[Steve Dower]

Steve Dower  added the comment:


New changeset 31350f9af09dcff7cf6ff4b0a0a7ea595942372e by Steve Dower in branch 
'master':
bpo-39837: Disable macOS tests on Azure Pipelines (GH-18818)
https://github.com/python/cpython/commit/31350f9af09dcff7cf6ff4b0a0a7ea595942372e


--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[Steve Dower]

Change by Steve Dower :


--
pull_requests: +18176
pull_request: https://github.com/python/cpython/pull/18818

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[Steve Dower]

Steve Dower  added the comment:

It would make the job definition significantly more complicated, and I don't 
want to do that just to work around an issue with github until we've got 
positive confirmation that the behaviour is intentional and won't change.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[Brett Cannon]

Brett Cannon  added the comment:

I've turned off the required checks for GH Actions and flipped Azure Pipelines 
back on.

And to answer Victor's question, yes, you can make things conditional at the 
workflow, job, and job step level. I don't know what would happen if the check 
was moved from workflow to job level.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[Steve Dower]

Steve Dower  added the comment:

Adding a screenshot here so I can point people at it.

Let's not rush into complicating the build steps yet - AP is basically fine. We 
should switch back the required checks (@Brett?)

If anything, let's add a "condition: false" to the macOS build to disable it 
and rely on the non-required GH check for now.

--
Added file: https://bugs.python.org/file48956/Annotation 2020-03-05 223851.png

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[STINNER Victor]

STINNER Victor  added the comment:

Instead of not running the job, is it technically possible to modify the jobs 
to do nothing for docs only changes?

.travis.yml works like that:

before_install:
  - set -e
  - |
  # Check short-circuit conditions
  if [[ "${TESTING}" != "docs" && "${TESTING}" != "doctest" ]]
  then
if [[ "$TRAVIS_PULL_REQUEST" == "false" ]]
then
  echo "Not a PR, doing full build."
else
  # Pull requests are slightly complicated because $TRAVIS_COMMIT_RANGE
  # may include more changes than desired if the history is convoluted.
  # Instead, explicitly fetch the base branch and compare against the
  # merge-base commit.
  git fetch -q origin +refs/heads/$TRAVIS_BRANCH
  changes=$(git diff --name-only HEAD $(git merge-base HEAD FETCH_HEAD))
  echo "Files changed:"
  echo "$changes"
  if ! echo "$changes" | grep -qvE '(\.rst$)|(^Doc)|(^Misc)'
  then
echo "Only docs were updated, stopping build process."
exit
  fi
fi
  fi

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[Steve Dower]

Steve Dower  added the comment:

Looks like it isn't fixed... https://github.com/python/cpython/pull/18774

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[Brett Cannon]

Brett Cannon  added the comment:

It was an old issue that if required checks didn't run it would block, but 
hopefully it's fixed. :)

I have gone ahead and removed the Azure Pipelines requirement from 3.7, 3.8, 
and master and flipped on the check requirements for the ones I listed.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[Steve Dower]

Steve Dower  added the comment:

Are you sure the required check isn't just for failures? Surely GitHub is 
smarter than requiring checks that it can tell aren't required (as it's their 
logic to include/exclude, not ours).

If they're not, I now have many internal contacts there, so we can probably get 
it fixed :)

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[Brett Cannon]

Brett Cannon  added the comment:

Actually, I just realized we can't make these status checks required because 
they don't always run. :) Our Actions are smart enough to not run when they 
aren't necessary, i.e. doc changes don't run the rest of the checks. And so 
making the OS-specific tests required would block doc tests.

Basically we would either have to waste runs on things that aren't really 
necessary but then be able to require runs, or we have to just rely on people 
paying attention to failures.

I'm personally for the latter.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[STINNER Victor]

STINNER Victor  added the comment:

> And to answer Victor's question on the PR that he closed, we can make any 
> individual status check required.

Great! I saw a macOS failure this week which was caused by an internal CI 
error. I hope that it will be fixed soon. In the meanwhile, I would suggest to 
not make the macOS job mandatory, but pay attention manually to its status ;-) 
Apart of macOS, other jobs look reliable.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[Steve Dower]

Steve Dower  added the comment:

There's no PR required. We need to change the required check so when I disable 
new builds from running it won't break existing PRs.

As for actually changing the files in the repo, I don't see any hurry there. We 
can clean up a few of them, and maybe I can move the release build scripts into 
the PC folder (though would have to backport that through to 3.7).

But I'm ready to disable the builds from running as soon as they're not a 
required check.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[Brett Cannon]

Brett Cannon  added the comment:

Yes, I can do it.

And to answer Victor's question on the PR that he closed, we can make any 
individual status check required. So probably:

- Docs
- Ubuntu
- Windows x86
- Windows x64

Just let me know when we are ready to merge a PR and I will switch off Azure 
Pipelines and make these checks required.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[Steve Dower]

Steve Dower  added the comment:

> Who is allowed to do that?

Brett did it the first time. I'm having too much trouble with GitHub right now 
to find the current admins.

--
nosy: +brett.cannon

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[STINNER Victor]

STINNER Victor  added the comment:

> PR 18769 should *not* be merged.

Ok, I closed it.

> First, it needs to be changed to a non-required check.

Who is allowed to do that?

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[Steve Dower]

Steve Dower  added the comment:

Deleting the files is not the right first step.

First, it needs to be changed to a non-required check.

Then, I can use the web UI to disable it starting.

*Then*, we can remove *some* of the files in the directory. Others are used for 
the official release, and have to stay.

PR 18769 should *not* be merged.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[Brett Cannon]

Change by Brett Cannon :


--
nosy: +steve.dower

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[STINNER Victor]

Change by STINNER Victor :


--
keywords: +patch
pull_requests: +18128
stage:  -> patch review
pull_request: https://github.com/python/cpython/pull/18769

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue39837] Remove Azure Pipelines from GitHub PRs

[STINNER Victor]

New submission from STINNER Victor :

The Azure Pipelines jobs have been reimplemented as GitHub actions which are 
better integrated with GitHub:

* Docs / Docs (pull_request)
* Tests / Windows (x64) (pull_request)
* Tests / macOS (pull_request)
* Tests / Ubuntu (pull_request)
* etc.

Azure Pipelines runs the same jobs, but it looks slower. It is voting and so 
prevents to merge a PR until it completes.

I propose to simply remove the job.

I already proposed it on python-dev:
https://mail.python.org/archives/list/python-...@python.org/message/NC2ZS4WSF5AYGUUMBB7I4YIQ4YJXWBA5/

In this thread:
https://mail.python.org/archives/list/python-...@python.org/thread/2NSPJUEWULTLLALR3HY3H2PRYAUT474C/#NC2ZS4WSF5AYGUUMBB7I4YIQ4YJXWBA5

--
components: Tests
messages: 363279
nosy: vstinner
priority: normal
severity: normal
status: open
title: Remove Azure Pipelines from GitHub PRs
versions: Python 3.9

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com