GeezerVison is showing github.com — Netlify builds happen for PRs that do not change docs · Issue #100072 · python/cpython
Netlify builds happen for PRs that do not change docs #100072Description
Bug report
After #92852 all PRs get a notification from netlify about new doc builds. Example: #100070 (notice no Doc/ changes made)
I don't think it is right:
- We are wasting resources for no good reason
- It is a noise for both contributors and core devs
But, netlify has build.ignore option that can work similarly to these lines:
|
paths: |
|
- 'Doc/**' |
|
- 'Misc/**' |
|
- '.github/workflows/doc.yml' |
Docs: https://docs.netlify.com/configure-builds/ignore-builds/
I will send a PR to test it :)
Linked PRs
Activitychanged the title [-]Netlify build happen for PR that do not change docs[/-] [+]Netlify builds happen for PRs that do not change docs[/+] on Dec 7, 2022sobolevn commented on Dec 8, 2022 @hugovk looks like we haven't solved this. Example PR that should not have this message, but still received it: #100114 hugovk commented on Dec 8, 2022 There's two things going on here.
First, deploys are now being cancelled, as a result of PR #100074. 👍
We can see that from the comment ("Deploy Preview for python-cpython-preview canceled"), the status checks ("Deploy canceled") and the logs:
3:05:11 PM: Detected ignore command in Netlify configuration file. Proceeding with the specified command: 'git diff --quiet $CACHED_COMMIT_REF $COMMIT_REF . ../netlify.toml'
3:05:11 PM: User-specified ignore command returned exit code 0. Returning early from build.
3:05:11 PM: Creating deploy upload records
3:05:11 PM: Failed during stage 'checking build content for changes': Canceled build due to no content change
Second, the deploy notifications need to be turned off. I think this can only be done in the admin settings on Netlify, rather than config, and only one person currently has access, but hopefully we'll get more soon. sobolevn commented on Dec 8, 2022 Reacted by Hugo van Kemenade CAM-Gerlach commented on Dec 16, 2022 Just to note, we've gotten an substantial and increasing volume of complaints about these comments from the core developers, as it is causing a large amount of noise and annoyance. While we had a fix ready to go within hours of Neltify being initially enabled, unfortunately as mentioned the blocker here is lack of bus-factor on people with appropriate permissions on the Netlify side. If we could get @Mariatta and @hugovk added there, we can not only solve this problem, but also be able to be responsive to any future issues, while reducing excess load on critical, hardworking infrastructure staff. @ewdurbin ?
Also, for the record, this is the config we need under the Notifications settings:
ewdurbin commented on Dec 16, 2022 Settings updated. Unfortunately it looks like netlify’s billing won’t allow me to add additional members at this time.
I’ll email their support to ask if we can get additional users on our plan.
…
On Thursday, Dec 15, 2022 at 10:34 PM, C.A.M. Gerlach ***@***.*** ***@***.***)> wrote:
Just to note, we've gotten an substantial and increasing volume of complaints about these comments from the core developers, as it is causing a large amount of noise and annoyance. While we had a fix ready to go within hours of Neltify being initially enabled, unfortunately as mentioned the blocker here is lack of bus-factor on people with appropriate permissions on the Netlify side. If we could get @Mariatta ( https://github.com/Mariatta) and @hugovk ( https://github.com/hugovk) added there, we can not only solve this problem, but also be able to be responsive to any future issues, while reducing excess load on critical, hardworking infrastructure staff. @ewdurbin ( https://github.com/ewdurbin) ?
Also, for the record, this is the config we need under the Notifications settings:
—
Reply to this email directly, view it on GitHub ( #100072 (comment)), or unsubscribe ( https://github.com/notifications/unsubscribe-auth/AAJFFQAXD7SSWKWKVOWFWALWNPPK7ANCNFSM6AAAAAASWZ45FY).
You are receiving this because you were mentioned.Message ID: ***@***.***>
Reacted by Hugo van Kemenade and C.A.M. Gerlach hugovk commented on Dec 16, 2022 Thank you!
https://www.netlify.com/legal/open-source-policy/ says (emphasis added):
Netlify loves open source! To show our commitment, we’ve created an Open Source plan to enable projects working on open-source-licensed software to host websites containing information directly related to the community around that software. Our Open Source plan has the same features and limits as our Pro plan, with the addition of free unlimited team members.
ewdurbin commented on Dec 16, 2022 Yes, and it seems we may not be upgraded to the latest Open Source plan.
…CAM-Gerlach commented on Dec 16, 2022 Thanks @ewdurbin , hopefully they'll respond soon so we don't have to bother you for anything like this in the future! I think we can close this particular issue now, since both parts are now solved.
MetadataMetadataDevelopmentNo branches or pull requestsIssue actions
|
