FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

gh-125741: Update check_generated_files CI to use our published container image by corona10 · Pull Request #125744 · python/cpython · GitHub

  1. python
  2. cpython

Repository navigation

gh-125741: Update check_generated_files CI to use our published container image - #125744

Merged
corona10 merged 7 commits into
python:mainfrom
corona10:gh-125741
Oct 20, 2024
ConversationCommits (7)ChecksFiles changed
Merged

gh-125741: Update check_generated_files CI to use our published container image#125744
corona10 merged 7 commits into
python:mainfrom
corona10:gh-125741

Conversation

corona10 commented Oct 20, 2024 •
edited by hugovk
Loading

Copy link
Copy Markdown
Member

bedevere-app Bot mentioned this pull request Oct 20, 2024
Closed
corona10 marked this pull request as ready for review October 20, 2024 02:33

corona10 commented Oct 20, 2024

Copy link
Copy Markdown
Member Author

@erlend-aasland @hugovk

I verified the the workflows well through https://github.com/python/cpython/actions/runs/11422707050/job/31781018424
This PR separates the task from check_generated_files but for unifying to the single task, we may need full unified devcontainer images to be used at https://github.com/python/cpython-devcontainers

Let's handle it separately.

Damien-Chen commented Oct 20, 2024

Copy link
Copy Markdown
Contributor

Since the container python used has something different from Github Action official base image and that is the reason why we should switch to the internal container ?

corona10 commented Oct 20, 2024 •
edited
Loading

Copy link
Copy Markdown
Member Author

See: https://discuss.python.org/t/pinning-gnu-autotools-using-the-github-container-registry/47437/29

FYI, The purpose of CI is verification of whether the person generate autoconf properly

So that’s why we have to unify the image to be same.

erlend-aasland commented Oct 20, 2024

Copy link
Copy Markdown
Contributor

Thanks. I need this for #125571.

hugovk changed the title gh-125741: Update check_generated_files CI to use our published conta… gh-125741: Update check_generated_files CI to use our published container image Oct 20, 2024
hugovk approved these changes Oct 20, 2024
View reviewed changes
Comment thread .github/workflows/build.yml Outdated
Update .github/workflows/build.yml
91a4f55 
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
corona10 enabled auto-merge (squash) October 20, 2024 07:31
corona10 merged commit ed24702 into python:main Oct 20, 2024

Damien-Chen commented Oct 20, 2024

Copy link
Copy Markdown
Contributor

Does that should be backport to 3.13 and 3.12 ?

erlend-aasland commented Oct 20, 2024

Copy link
Copy Markdown
Contributor

Does that should be backport to 3.13 and 3.12 ?

I think we should backport it, yes. Hugo?

hugovk commented Oct 20, 2024

Copy link
Copy Markdown
Member

Yes, it'll be easier to maintain if the three branches are more similar.

erlend-aasland added needs backport to 3.12 only security fixes needs backport to 3.13 only security fixes labels Oct 20, 2024

This comment was marked as outdated.

Sign in to view

This comment was marked as outdated.

Sign in to view

This comment was marked as outdated.

Sign in to view

This comment was marked as outdated.

Sign in to view

bedevere-app Bot commented Oct 20, 2024

Copy link
Copy Markdown

GH-125759 is a backport of this pull request to the 3.13 branch.

bedevere-app Bot removed the needs backport to 3.13 only security fixes label Oct 20, 2024
erlend-aasland pushed a commit to erlend-aasland/cpython that referenced this pull request Oct 20, 2024
[3.13] pythongh-125741: Update check_generated_files CI to use our pu…
8bfcaa1 
…blished container image (pythongh-125744)

(cherry picked from commit ed24702)

Co-authored-by: Donghee Na <donghee.na@python.org>

This comment was marked as resolved.

Sign in to view

bedevere-app Bot commented Oct 20, 2024

Copy link
Copy Markdown

GH-125760 is a backport of this pull request to the 3.12 branch.

bedevere-app Bot removed the needs backport to 3.12 only security fixes label Oct 20, 2024
erlend-aasland pushed a commit to erlend-aasland/cpython that referenced this pull request Oct 20, 2024
[3.12] pythongh-125741: Update check_generated_files CI to use our pu…
941e5c3 
…blished container image (pythongh-125744)

(cherry picked from commit ed24702)

Co-authored-by: Donghee Na <donghee.na@python.org>
corona10 mentioned this pull request Oct 20, 2024
Merged
AA-Turner mentioned this pull request Oct 21, 2024
Merged
erlend-aasland added a commit that referenced this pull request Oct 21, 2024
[3.13] gh-125741: Update check_generated_files CI to use our publishe…
a6d81b4 
…d container image (gh-125744) (#125759)

(cherry picked from commit ed24702)

Co-authored-by: Donghee Na <donghee.na@python.org>
erlend-aasland added a commit that referenced this pull request Oct 21, 2024
[3.12] gh-125741: Update check_generated_files CI to use our publishe…
2bc2aae 
…d container image (gh-125744) (#125760)

(cherry picked from commit ed24702)

Co-authored-by: Donghee Na <donghee.na@python.org>
sobolevn mentioned this pull request Nov 12, 2024
Merged
ebonnal pushed a commit to ebonnal/cpython that referenced this pull request Jan 12, 2025
pythongh-125741: Update check_generated_files CI to use our published…
b4e9908 
… container image (pythongh-125744)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

hugovk hugovk approved these changes
erlend-aasland erlend-aasland approved these changes
ezio-melotti Awaiting requested review from ezio-melotti ezio-melotti is a code owner

Assignees

corona10

Labels

skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

Back | FazBrowse Home | New Git URL