RHEL Engineering is moving the tracking of its product development work on RHEL 6 through RHEL 9 to Red Hat Jira (issues.redhat.com). If you're a Red Hat customer, please continue to file support cases via the Red Hat customer portal. If you're not, please head to the "RHEL project" in Red Hat Jira and file new tickets here. Individual Bugzilla bugs in the statuses "NEW", "ASSIGNED", and "POST" are being migrated throughout September 2023. Bugs of Red Hat partners with an assigned Engineering Partner Manager (EPM) are migrated in late September as per pre-agreed dates. Bugs against components "kernel", "kernel-rt", and "kpatch" are only migrated if still in "NEW" or "ASSIGNED". If you cannot log in to RH Jira, please consult article #7032570. That failing, please send an e-mail to the RH Jira admins at rh-issues@redhat.com to troubleshoot your issue as a user management inquiry. The email creates a ServiceNow ticket with Red Hat. Individual Bugzilla bugs that are migrated will be moved to status "CLOSED", resolution "MIGRATED", and set with "MigratedToJIRA" in "Keywords". The link to the successor Jira issue will be found under "Links", have a little "two-footprint" icon next to it, and direct you to the "RHEL project" in Red Hat Jira (issue links are of type "https://issues.redhat.com/browse/RHEL-XXXX", where "X" is a digit). This same link will be available in a blue banner at the top of the page informing you that that bug has been migrated.
Bug 1991787 - RFE: Please add the ability to build external nginx modules
Summary: RFE: Please add the ability to build external nginx modules
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Enterprise Linux 8
Classification: Red Hat
Component: nginx-1.20-module
Version: CentOS Stream
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: beta
: ---
Assignee: Luboš Uhliarik
QA Contact: icesalov
Lenka Špačková
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2021-08-10 02:58 UTC by Neal Gompa
Modified: 2022-05-10 13:51 UTC (History)
18 users (show)

Fixed In Version: nginx-1.20-8060020210910095205.d63f516d
Doc Type: Enhancement
Doc Text:
.A new package: `nginx-mod-devel` A new `nginx-mod-devel` package has been added to the `nginx:1.20` module stream. The package provides all necessary files, including RPM macros and `nginx` source code, for building external dynamic modules for `nginx`.
Clone Of: 1991720
Environment:
Last Closed: 2022-05-10 13:32:23 UTC
Type: Enhancement
Target Upstream Version:
Embargoed:

Attachments (Terms of Use)
Dist-Git patch to resolve this (12.77 KB, application/mbox)
2021-08-10 03:06 UTC, Neal Gompa 		no flags Details
Updated Dist-Git patch to resolve this (12.49 KB, application/mbox)
2021-08-10 16:40 UTC, Neal Gompa 		no flags Details
Show Obsolete (1) View All


Links
System ID Private Priority Status Summary Last Updated
Red Hat Issue Tracker RHELPLAN-92978 0 None None None 2021-08-10 03:01:23 UTC
Red Hat Issue Tracker RHELPLAN-92979 0 None None None 2021-08-10 03:01:26 UTC
Red Hat Product Errata RHEA-2022:1767 0 None None None 2022-05-10 13:32:29 UTC

Description Neal Gompa 2021-08-10 02:58:00 UTC 
+++ This bug was initially created as a clone of Bug #1991720 +++

Description of problem:
Nginx offers the ability to build external dynamic modules[1]; however, we don't have a way to do this in RHEL.

Please a -mod-devel package so that nginx modules can be built for nginx.

[1]: https://www.nginx.com/resources/wiki/extending/converting/#compiling-a-dynamic-module

Version-Release number of selected component (if applicable):
1.20.0-1.module+el8.5.0+10874+8aadcf3f
Comment 1 Neal Gompa 2021-08-10 03:06:52 UTC 
Created attachment 1812657 [details]
Dist-Git patch to resolve this

I've attached a patch to resolve this. It can be applied to the RHEL Dist-Git with "git am -p2".
Comment 2 Neal Gompa 2021-08-10 03:17:47 UTC 
I switched the component to make it clearer this is about the upcoming nginx-1.20 module.
Comment 3 Neal Gompa 2021-08-10 16:40:05 UTC 
Created attachment 1812808 [details]
Updated Dist-Git patch to resolve this

I've refreshed the patch to simplify the generator and make the macros match Fedora and EL9.
Comment 4 Neal Gompa 2021-08-13 11:05:49 UTC 
I've also filed this as a support case: https://access.redhat.com/support/cases/#/case/03007218
Comment 8 Luboš Uhliarik 2021-09-10 01:36:21 UTC 
Hi Neal,

I have already commited changes, so it should be synced soon with c8s.
Comment 9 Neal Gompa 2021-09-10 02:01:24 UTC 
(In reply to Luboš Uhliarik from comment #8)
> Hi Neal,
> 
> I have already commited changes, so it should be synced soon with c8s.

You committed the changes adding the -mod-devel package? Or just the --compat flag?
Comment 10 Branislav Náter 2021-09-10 07:43:42 UTC 
(In reply to Neal Gompa from comment #9)
> (In reply to Luboš Uhliarik from comment #8)
> > Hi Neal,
> > 
> > I have already commited changes, so it should be synced soon with c8s.
> 
> You committed the changes adding the -mod-devel package? Or just the
> --compat flag?

nginx-1.20 with --compat flag is already in c8s. This bug is about -mod-devel.
Comment 11 Neal Gompa 2021-09-10 18:38:44 UTC 
(In reply to Branislav Náter from comment #10)
> (In reply to Neal Gompa from comment #9)
> > (In reply to Luboš Uhliarik from comment #8)
> > > Hi Neal,
> > > 
> > > I have already commited changes, so it should be synced soon with c8s.
> > 
> > You committed the changes adding the -mod-devel package? Or just the
> > --compat flag?
> 
> nginx-1.20 with --compat flag is already in c8s. This bug is about
> -mod-devel.

Any chance we could also get bug 1991720 resolved for RHEL 9.0 too?
Comment 12 Branislav Náter 2021-09-12 08:59:00 UTC 
(In reply to Neal Gompa from comment #11)
> (In reply to Branislav Náter from comment #10)
> > (In reply to Neal Gompa from comment #9)
> > > (In reply to Luboš Uhliarik from comment #8)
> > > > Hi Neal,
> > > > 
> > > > I have already commited changes, so it should be synced soon with c8s.
> > > 
> > > You committed the changes adding the -mod-devel package? Or just the
> > > --compat flag?
> > 
> > nginx-1.20 with --compat flag is already in c8s. This bug is about
> > -mod-devel.
> 
> Any chance we could also get bug 1991720 resolved for RHEL 9.0 too?

Yes, we are planning to do so.
Comment 15 Luboš Uhliarik 2021-09-21 11:05:58 UTC 
Hi Neal,

this bug should be already merged in c8s. Please check it out.

https://git.centos.org/rpms/nginx/c/99dfe574fd6c460c2265a20982a831cbadb9d873?branch=c8s-stream-1.20
Comment 16 Neal Gompa 2021-09-21 12:34:20 UTC 
(In reply to Luboš Uhliarik from comment #15)
> Hi Neal,
> 
> this bug should be already merged in c8s. Please check it out.
> 
> https://git.centos.org/rpms/nginx/c/
> 99dfe574fd6c460c2265a20982a831cbadb9d873?branch=c8s-stream-1.20

Yep, I see it! I'm just waiting for it to be built and released in CentOS Stream 8 to test it out now.
Comment 17 Branislav Náter 2021-09-29 11:48:35 UTC 
(In reply to Neal Gompa from comment #16)
> (In reply to Luboš Uhliarik from comment #15)
> > Hi Neal,
> > 
> > this bug should be already merged in c8s. Please check it out.
> > 
> > https://git.centos.org/rpms/nginx/c/
> > 99dfe574fd6c460c2265a20982a831cbadb9d873?branch=c8s-stream-1.20
> 
> Yep, I see it! I'm just waiting for it to be built and released in CentOS
> Stream 8 to test it out now.

Hi Neal. Packages are present in c8s now [1]. Can you please let us know if everything works as expected and eventually provide us with test scenario?

1. http://mirror.centos.org/centos/8-stream/AppStream/x86_64/os/Packages/
Comment 18 Neal Gompa 2021-09-30 12:55:21 UTC 
(In reply to Branislav Náter from comment #17)
> (In reply to Neal Gompa from comment #16)
> > (In reply to Luboš Uhliarik from comment #15)
> > > Hi Neal,
> > > 
> > > this bug should be already merged in c8s. Please check it out.
> > > 
> > > https://git.centos.org/rpms/nginx/c/
> > > 99dfe574fd6c460c2265a20982a831cbadb9d873?branch=c8s-stream-1.20
> > 
> > Yep, I see it! I'm just waiting for it to be built and released in CentOS
> > Stream 8 to test it out now.
> 
> Hi Neal. Packages are present in c8s now [1]. Can you please let us know if
> everything works as expected and eventually provide us with test scenario?
> 
> 1. http://mirror.centos.org/centos/8-stream/AppStream/x86_64/os/Packages/

Everything seems to work as expected.

Here's a sample container I made for the purpose of testing it: https://pagure.io/nginx-vts-container

You can reproduce and verify this with either CentOS Stream 8 or Fedora and see it works. :)
Comment 29 errata-xmlrpc 2022-05-10 13:32:23 UTC 
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (nginx:1.20 bug fix and enhancement update), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHEA-2022:1767
Note You need to log in before you can comment on or make changes to this bug.