Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

release 19.8.0 #1634

Closed
benoitc opened this issue Oct 26, 2017 · 34 comments
Closed

release 19.8.0 #1634

benoitc opened this issue Oct 26, 2017 · 34 comments
Assignees

Comments

@benoitc
Copy link
Owner

benoitc commented Oct 26, 2017

As I would like to release a 19.8 version this week. I don't see anything preventing it right now. Tests pass here but let me know if I missed something.

@benoitc benoitc self-assigned this Oct 26, 2017
@benoitc
Copy link
Owner Author

benoitc commented Oct 26, 2017

cc @tilgovi @berkerpeksag

@berkerpeksag
Copy link
Collaborator

berkerpeksag commented Oct 26, 2017

I would like to get the following PRs merged for 19.8:

@benoitc
Copy link
Owner Author

benoitc commented Oct 26, 2017

@berkerpeksag i commented/approved then. Let me know if you need some help as I may have some cycles tomorrow :)

@tilgovi
Copy link
Collaborator

tilgovi commented Oct 27, 2017

I have open PRs. Do we want to merge any of them?

@berkerpeksag
Copy link
Collaborator

Both PRs good to have if we have time to get them in.

@tilgovi
Copy link
Collaborator

tilgovi commented Oct 30, 2017

Mine are in. Can I help with reviewing anything?

@berkerpeksag
Copy link
Collaborator

#1499 is the only remaining one. I've addressed my review comments.

@benoitc
Copy link
Owner Author

benoitc commented Nov 6, 2017

@berkerpeksag i've commented also, what do you think about it?

@benoitc
Copy link
Owner Author

benoitc commented Nov 12, 2017

@berkerpeksag i didn't get an answer about #1499 . I'm dubious about putting back a eature that have been removed a long itme ago. We should see the pro/cons about it. Or at least makng sure it has no impact.

I think however that before releasing i will disable by defaut the SO_REUSEPORT option of the socket. Thoughts?

@RonRothman
Copy link

RonRothman commented Nov 12, 2017

@benoitc I'm not sure I understand - why would we want to change the default for SO_REUSEPORT based on that PR?

(We do rely on SO_RESUSEPORT to help us efficiently serve tens of thousands of requests/sec, so I'm careful around changes to it.)

@tilgovi
Copy link
Collaborator

tilgovi commented Nov 12, 2017

Having SO_REUSEPORT as a default seems to confused some people, especially those new to Gunicorn or using it in a development environment. It makes it possible to start a new Gunicorn without killing the old one, resulting in multiple versions of an application running. This is not a very friendly experience for a first time user who wonders why their application serves the right response only half of the time.

We will definitely not remove the functionality even if we change the default.

@berkerpeksag
Copy link
Collaborator

@berkerpeksag i didn't get an answer about #1499 . I'm dubious about putting back a eature that have been removed a long itme ago. We should see the pro/cons about it. Or at least makng sure it has no impact.

Looking at #1499 again, I'm fine with closing it as 'wontfix'. Making it compatible with Gunicorn 19.4+ is pretty trivial:

-            '-c', 'airflow.www.gunicorn_config'
+            '-c', 'python:airflow.www.gunicorn_config'

@benoitc
Copy link
Owner Author

benoitc commented Dec 10, 2017

on my todo list before any release I have the following:

i will try to have something for friday now

@robyoung-digital
Copy link

Is there any more news on this?

@berkerpeksag
Copy link
Collaborator

I've just merged PR #1669 and closed PR #1499 so I think the only blocker is now #1653.

@berkerpeksag
Copy link
Collaborator

I think we also need to document the SO_REUSEPORT behavior change (PR #1669) in release notes.

@robyoung-digital
Copy link

Wow, thanks for the quick response. 💯

@Allu2
Copy link

Allu2 commented Feb 8, 2018

Knowing asking ETA's is the greatest sin of all, are there any news on progress of #1653 which seems to be left blocking the release?

@berkerpeksag
Copy link
Collaborator

@benoitc @tilgovi should we leave #1653 to 19.9 and release 19.8 as is? There are some important fixes in master and there are some questions from users on ETA for 19.8 in several issues (for example, #1058) so I'm wondering if we should skip #1653 for now.

@berkerpeksag
Copy link
Collaborator

There's an almost ready PR for #1324:

@tilgovi
Copy link
Collaborator

tilgovi commented Feb 28, 2018

I am fine to skip #1653

@benoitc
Copy link
Owner Author

benoitc commented Mar 1, 2018

I don't see any reason to not release more often :) If I don't came with a patch by today let's release tomorrow in the morning.

About that pyc stuff, i would be for fixing it in the current release and not adding any new option. I would rather deprecate this feature as it breaks the contract of auto reloading the config and such things.

@Allu2
Copy link

Allu2 commented Mar 5, 2018

Pypi seems to be still at 19.7.1 https://pypi.python.org/pypi/gunicorn
As a side thought could the README.md be made to have stable version number?
going trough pypi to check what is the current release feels bit funky, or perhaps I'm missing some obvious version number displayed somewhere 😕

@JonathanHuot
Copy link

Sorry to bump, any news about the 19.8 release ? I'm eager to play with dictConfig :-)

@Code0x58 Code0x58 mentioned this issue Mar 19, 2018
8 tasks
@hugovk
Copy link
Contributor

hugovk commented Mar 19, 2018

(@Allu2 Please see PR #1727 to show the current release's version number in the README)

@unexceptable
Copy link

Any update on this? I'm looking forward to it since it fixes some issues with --reload in alpine containers and would make our dev environments much nicer!

@tilgovi
Copy link
Collaborator

tilgovi commented Apr 17, 2018

I'll prepare a release. @berkerpeksag @benoitc good for you?

@berkerpeksag
Copy link
Collaborator

@tilgovi +1 from me.

@Allu2
Copy link

Allu2 commented Apr 27, 2018

Out of curiosity, what kind of process making a gunicorn release is?
This release seems to have been postponed for the past 5 months with more then one "next (day | week | soon)"

@tilgovi
Copy link
Collaborator

tilgovi commented Apr 28, 2018

@Allu2

  • Update changelog
  • Tag and upload
  • Make release on GitHub

It's not hard, but all of us do this on a volunteer basis.

@tilgovi
Copy link
Collaborator

tilgovi commented Apr 28, 2018

I've updated the changelog and I believe everything is prepped for release.

I'm going to sleep and will cut the tag in the morning, when I can be around in case of any mistake that requires a followup patch release, but I don't expect any issues.

@tilgovi
Copy link
Collaborator

tilgovi commented Apr 28, 2018

19.8.0 is released!

Please comment here with any feedback or issues!

@tilgovi tilgovi closed this as completed Apr 28, 2018
@tilgovi
Copy link
Collaborator

tilgovi commented Apr 28, 2018

We're going to aim for more frequent releases going forward. Your pull requests and reviews help. If you're interested in being a maintainer, let me know!

@benoitc
Copy link
Owner Author

benoitc commented Apr 28, 2018

@tilgovi thank you! and agree :) sorry to not have answered soon I was traveling all the month

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

9 participants