Skip to content

Release prep v11.0.0 #2464

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

Merged
merged 1 commit into from
Sep 22, 2023
Merged

Release prep v11.0.0 #2464

merged 1 commit into from
Sep 22, 2023

Conversation

github-actions[bot]
Copy link

@github-actions github-actions bot commented Sep 21, 2023

Automated release-prep through pdk-templates from commit 502a891.
Please verify before merging:

Copy link
Collaborator

@ekohl ekohl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please clean up more deprecations if this is a major version. Various parameters are deprecated and can be removed.

@ekohl
Copy link
Collaborator

ekohl commented Sep 21, 2023

#2465 & #2466 for example.

@Ramesh7
Copy link
Contributor

Ramesh7 commented Sep 22, 2023

#2465 & #2466 for example.

Thanks @ekohl for your quick PR's, have checked and they have included into this release.
Can you please re-review?

Ramesh7
Ramesh7 previously approved these changes Sep 22, 2023
Copy link
Collaborator

@ekohl ekohl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Those were just some examples. There's still more deprecated code in the module:

$ rg -i depre manifests/ lib/
manifests/vhost.pp
1099:#   > **Note**: This parameter is deprecated in favor of the `aliases` parameter.<br />
2705:    deprecation('apache::vhost::wsgi_daemon_process_options', 'This parameter is deprecated. Please add values inside Hash `wsgi_daemon_process`.')

lib/puppet/functions/bool2httpd.rb
3:# @summary DEPRECATED.  Use the namespaced function [`apache::bool2httpd`](#apachebool2httpd) instead.
5:  dispatch :deprecation_gen do
8:  def deprecation_gen(*args)
9:    call_function('deprecation', 'bool2httpd', 'This function is deprecated, please use apache::bool2httpd instead.')

lib/puppet/functions/apache_pw_hash.rb
3:# @summary DEPRECATED.  Use the namespaced function [`apache::pw_hash`](#apachepw_hash) instead.
5:  dispatch :deprecation_gen do
8:  def deprecation_gen(*args)
9:    call_function('deprecation', 'apache_pw_hash', 'This function is deprecated, please use apache::pw_hash instead.')

manifests/init.pp
110:#   determine an appropriate `content-type`. This directive is deprecated in Apache 2.4 and 
429:#   > **Note**: This parameter is deprecated in favor of the `purge_configs` parameter.<br />
602:  # Deprecated backwards-compatibility
604:    warning('Class[\'apache\'] parameter purge_vdir is deprecated in favor of purge_configs')

lib/puppet/functions/apache/apache_pw_hash.rb
3:# @summary DEPRECATED.  Use the function [`apache::pw_hash`](#apachepw_hash) instead.
5:  dispatch :deprecation_gen do
8:  def deprecation_gen(*args)
9:    call_function('deprecation', 'apache::apache_pw_hash', 'This function is deprecated, please use apache::pw_hash instead.')

manifests/mod/auth_cas.pp
25:#   Whether to validate the presented certificate. This has been deprecated and

manifests/mod/remoteip.pp
16:#   *Deprecated*: use `$internal_proxy` instead.
43:#   *Deprecated*: use `$trusted_proxy` instead.
67:    deprecation('apache::mod::remoteip::proxy_ips', 'This parameter is deprecated, please use `internal_proxy`.')
76:    deprecation('apache::mod::remoteip::trusted_proxy_ips', 'This parameter is deprecated, please use `trusted_proxy`.')

manifests/mod/passenger.pp
263:#   Deprecated in 3.0.0 in favor of PassengerBaseURI.
281:#   Deprecated in 3.0.0 in favor of PassengerBaseURI.
284:#   Deprecated in 3.0.0 in favor of PassengerDefaultUser
294:#   Deprecated in 3.0.0 in favor of PassengerRuby.
297:#   Deprecated in 3.0.0 in favor of PassengerSpawnMethod.
300:#   Deprecated in 3.0.0 in favor of PassengerUserSwitching.
314:# The current set of server configurations settings were taken directly from the Passenger Reference. To enable deprecation warnings 
318:#   - As of 08/13/2017 there are 84 available/deprecated/removed settings.
320:#   - As of 08/20/2017 there are 77 available/deprecated/removed settings.
480:        warning('DEPRECATED PASSENGER OPTION :: passenger_debug_log_file :: This option has been renamed in version 5.0.5 to PassengerLogFile.')
778:        warning('DEPRECATED PASSENGER OPTION :: rack_base_uri :: Deprecated in 3.0.0 in favor of PassengerBaseURI.')
788:        warning("DEPRECATED PASSENGER OPTION :: rails_allow_mod_rewrite :: This option doesn't do anything anymore in since version 4.0.0.")
803:        warning('DEPRECATED PASSENGER OPTION :: rails_base_uri :: Deprecated in 3.0.0 in favor of PassengerBaseURI.')
808:        warning('DEPRECATED PASSENGER OPTION :: rails_default_user :: Deprecated in 3.0.0 in favor of PassengerDefaultUser.')
823:        warning('DEPRECATED PASSENGER OPTION :: rails_ruby :: Deprecated in 3.0.0 in favor of PassengerRuby.')
828:        warning('DEPRECATED PASSENGER OPTION :: rails_spawn_method :: Deprecated in 3.0.0 in favor of PassengerSpawnMethod.')
833:        warning('DEPRECATED PASSENGER OPTION :: rails_user_switching :: Deprecated in 3.0.0 in favor of PassengerUserSwitching.')
912:  # - $passenger_debug_log_file : since unkown. Deprecated in 5.0.5.
963:  # - $passenger_use_global_queue : since 2.0.4.Deprecated in 4.0.0.
966:  # - $rack_auto_detect : since unkown. Deprecated in 4.0.0.
967:  # - $rack_base_uri : since unkown. Deprecated in 3.0.0.
969:  # - $rails_allow_mod_rewrite : since unkown. Deprecated in 4.0.0.
970:  # - $rails_app_spawner_idle_time : since unkown. Deprecated in 4.0.0.
971:  # - $rails_auto_detect : since unkown. Deprecated in 4.0.0.
972:  # - $rails_base_uri : since unkown. Deprecated in 3.0.0.
973:  # - $rails_default_user : since unkown. Deprecated in 3.0.0.
975:  # - $rails_framework_spawner_idle_time : since unkown. Deprecated in 4.0.0.
976:  # - $rails_ruby : since unkown. Deprecated in 3.0.0.
977:  # - $rails_spawn_method : since unkown. Deprecated in 3.0.0.
978:  # - $rails_user_switching : since unkown. Deprecated in 3.0.0.
979:  # - $wsgi_auto_detect : since unkown. Deprecated in 4.0.0.

I'd look at Passenger and see what common minimum versions are these days. For example, there's still code for mod_passenger 0.x but I doubt that's still shipped anywhere.

@Ramesh7
Copy link
Contributor

Ramesh7 commented Sep 22, 2023

Those were just some examples. There's still more deprecated code in the module:

$ rg -i depre manifests/ lib/
manifests/vhost.pp
1099:#   > **Note**: This parameter is deprecated in favor of the `aliases` parameter.<br />
2705:    deprecation('apache::vhost::wsgi_daemon_process_options', 'This parameter is deprecated. Please add values inside Hash `wsgi_daemon_process`.')

lib/puppet/functions/bool2httpd.rb
3:# @summary DEPRECATED.  Use the namespaced function [`apache::bool2httpd`](#apachebool2httpd) instead.
5:  dispatch :deprecation_gen do
8:  def deprecation_gen(*args)
9:    call_function('deprecation', 'bool2httpd', 'This function is deprecated, please use apache::bool2httpd instead.')

lib/puppet/functions/apache_pw_hash.rb
3:# @summary DEPRECATED.  Use the namespaced function [`apache::pw_hash`](#apachepw_hash) instead.
5:  dispatch :deprecation_gen do
8:  def deprecation_gen(*args)
9:    call_function('deprecation', 'apache_pw_hash', 'This function is deprecated, please use apache::pw_hash instead.')

manifests/init.pp
110:#   determine an appropriate `content-type`. This directive is deprecated in Apache 2.4 and 
429:#   > **Note**: This parameter is deprecated in favor of the `purge_configs` parameter.<br />
602:  # Deprecated backwards-compatibility
604:    warning('Class[\'apache\'] parameter purge_vdir is deprecated in favor of purge_configs')

lib/puppet/functions/apache/apache_pw_hash.rb
3:# @summary DEPRECATED.  Use the function [`apache::pw_hash`](#apachepw_hash) instead.
5:  dispatch :deprecation_gen do
8:  def deprecation_gen(*args)
9:    call_function('deprecation', 'apache::apache_pw_hash', 'This function is deprecated, please use apache::pw_hash instead.')

manifests/mod/auth_cas.pp
25:#   Whether to validate the presented certificate. This has been deprecated and

manifests/mod/remoteip.pp
16:#   *Deprecated*: use `$internal_proxy` instead.
43:#   *Deprecated*: use `$trusted_proxy` instead.
67:    deprecation('apache::mod::remoteip::proxy_ips', 'This parameter is deprecated, please use `internal_proxy`.')
76:    deprecation('apache::mod::remoteip::trusted_proxy_ips', 'This parameter is deprecated, please use `trusted_proxy`.')

manifests/mod/passenger.pp
263:#   Deprecated in 3.0.0 in favor of PassengerBaseURI.
281:#   Deprecated in 3.0.0 in favor of PassengerBaseURI.
284:#   Deprecated in 3.0.0 in favor of PassengerDefaultUser
294:#   Deprecated in 3.0.0 in favor of PassengerRuby.
297:#   Deprecated in 3.0.0 in favor of PassengerSpawnMethod.
300:#   Deprecated in 3.0.0 in favor of PassengerUserSwitching.
314:# The current set of server configurations settings were taken directly from the Passenger Reference. To enable deprecation warnings 
318:#   - As of 08/13/2017 there are 84 available/deprecated/removed settings.
320:#   - As of 08/20/2017 there are 77 available/deprecated/removed settings.
480:        warning('DEPRECATED PASSENGER OPTION :: passenger_debug_log_file :: This option has been renamed in version 5.0.5 to PassengerLogFile.')
778:        warning('DEPRECATED PASSENGER OPTION :: rack_base_uri :: Deprecated in 3.0.0 in favor of PassengerBaseURI.')
788:        warning("DEPRECATED PASSENGER OPTION :: rails_allow_mod_rewrite :: This option doesn't do anything anymore in since version 4.0.0.")
803:        warning('DEPRECATED PASSENGER OPTION :: rails_base_uri :: Deprecated in 3.0.0 in favor of PassengerBaseURI.')
808:        warning('DEPRECATED PASSENGER OPTION :: rails_default_user :: Deprecated in 3.0.0 in favor of PassengerDefaultUser.')
823:        warning('DEPRECATED PASSENGER OPTION :: rails_ruby :: Deprecated in 3.0.0 in favor of PassengerRuby.')
828:        warning('DEPRECATED PASSENGER OPTION :: rails_spawn_method :: Deprecated in 3.0.0 in favor of PassengerSpawnMethod.')
833:        warning('DEPRECATED PASSENGER OPTION :: rails_user_switching :: Deprecated in 3.0.0 in favor of PassengerUserSwitching.')
912:  # - $passenger_debug_log_file : since unkown. Deprecated in 5.0.5.
963:  # - $passenger_use_global_queue : since 2.0.4.Deprecated in 4.0.0.
966:  # - $rack_auto_detect : since unkown. Deprecated in 4.0.0.
967:  # - $rack_base_uri : since unkown. Deprecated in 3.0.0.
969:  # - $rails_allow_mod_rewrite : since unkown. Deprecated in 4.0.0.
970:  # - $rails_app_spawner_idle_time : since unkown. Deprecated in 4.0.0.
971:  # - $rails_auto_detect : since unkown. Deprecated in 4.0.0.
972:  # - $rails_base_uri : since unkown. Deprecated in 3.0.0.
973:  # - $rails_default_user : since unkown. Deprecated in 3.0.0.
975:  # - $rails_framework_spawner_idle_time : since unkown. Deprecated in 4.0.0.
976:  # - $rails_ruby : since unkown. Deprecated in 3.0.0.
977:  # - $rails_spawn_method : since unkown. Deprecated in 3.0.0.
978:  # - $rails_user_switching : since unkown. Deprecated in 3.0.0.
979:  # - $wsgi_auto_detect : since unkown. Deprecated in 4.0.0.

I'd look at Passenger and see what common minimum versions are these days. For example, there's still code for mod_passenger 0.x but I doubt that's still shipped anywhere.

@ekohl agreed, was going through today the deprication thing and we have long list for that, I thought to take that as new items and deliver those in upcoming releaes.
The module was long due as we have added many features recently and some bug fixes for those the community is waiting.
Looking for thoughts?

@Ramesh7
Copy link
Contributor

Ramesh7 commented Sep 22, 2023

@ekohl saying that I have created backlog items for tracking the same.

@ekohl
Copy link
Collaborator

ekohl commented Sep 22, 2023

I created #2469 for the scriptaliases parameter.

@Ramesh7
Copy link
Contributor

Ramesh7 commented Sep 22, 2023

I created #2469 for the scriptaliases parameter.

Created #2470, as of now tried to cover up vhost and passenger. The work is still in progress.

@Ramesh7
Copy link
Contributor

Ramesh7 commented Sep 22, 2023

Just now saw you already covered for scriptaliases.

@Ramesh7 Ramesh7 merged commit 5d47675 into main Sep 22, 2023
@Ramesh7 Ramesh7 deleted the release-prep branch September 22, 2023 18:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants