From msehnout at redhat.com Tue Apr 10 07:40:10 2018 From: msehnout at redhat.com (Martin Sehnoutka) Date: Tue, 10 Apr 2018 09:40:10 +0200 Subject: [Dnssec-trigger] Reducing dnssec-trigger-script in favor of riggerd Message-ID: Hello, I am working on a project of migrating the functionality currently provided by the script into the daemon. The build system was modified in a way, that you can select which implementation you want during build time. It, of course, defaults to the Python implementation. I have also added support for simple unit testing and automatic RPM package build on push event. Regarding the code base, the "update" command from the script is available in the daemon as "update_all", so that they can live side by side: https://github.com/InfrastructureServices/dnssec-trigger/blob/master-fedora/riggerd/svr.c#L806 Now, I would like to ask you, is it acceptable for you to receive this project as a series of patches over the mailing-list? I know there was a question about migration to Git some time ago, is it being considered? Best regards, -- Martin Sehnoutka | Associate Software Engineer PGP: 5FD64AF5 UTC+1 (CET) RED HAT | TRIED. TESTED. TRUSTED. From wouter at nlnetlabs.nl Tue Apr 10 07:54:20 2018 From: wouter at nlnetlabs.nl (W.C.A. Wijngaards) Date: Tue, 10 Apr 2018 09:54:20 +0200 Subject: [Dnssec-trigger] Reducing dnssec-trigger-script in favor of riggerd In-Reply-To: References: Message-ID: Hi Martin, On 10/04/18 09:40, Martin Sehnoutka wrote: > Hello, > > I am working on a project of migrating the functionality currently > provided by the script into the daemon. The build system was modified in > a way, that you can select which implementation you want during build > time. It, of course, defaults to the Python implementation. I have also > added support for simple unit testing and automatic RPM package build on > push event. Regarding the code base, the "update" command from the > script is available in the daemon as "update_all", so that they can live > side by side: > https://github.com/InfrastructureServices/dnssec-trigger/blob/master-fedora/riggerd/svr.c#L806 > > Now, I would like to ask you, is it acceptable for you to receive this > project as a series of patches over the mailing-list? I know there was a Sure just send the patches. Sounds like it could be a nice improvement! > question about migration to Git some time ago, is it being considered? The repo is replicated to github by script, I believe. Best regards, Wouter > > Best regards, > -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 833 bytes Desc: OpenPGP digital signature URL: From msehnout at redhat.com Thu Apr 19 12:23:56 2018 From: msehnout at redhat.com (Martin Sehnoutka) Date: Thu, 19 Apr 2018 12:23:56 -0000 Subject: [Dnssec-trigger] Reducing dnssec-trigger-script in favor of riggerd In-Reply-To: References: Message-ID: <30645bd2-00f4-b842-ec1e-d21a064f8eea@redhat.com> On 04/10/2018 09:54 AM, W.C.A. Wijngaards wrote: > Hi Martin, > > On 10/04/18 09:40, Martin Sehnoutka wrote: >> Hello, >> >> I am working on a project of migrating the functionality currently >> provided by the script into the daemon. The build system was modified in >> a way, that you can select which implementation you want during build >> time. It, of course, defaults to the Python implementation. I have also >> added support for simple unit testing and automatic RPM package build on >> push event. Regarding the code base, the "update" command from the >> script is available in the daemon as "update_all", so that they can live >> side by side: >> https://github.com/InfrastructureServices/dnssec-trigger/blob/master-fedora/riggerd/svr.c#L806 >> >> Now, I would like to ask you, is it acceptable for you to receive this >> project as a series of patches over the mailing-list? I know there was a > > Sure just send the patches. Sounds like it could be a nice improvement! > In order to reduce amount of patches, I tried to divide my work according to files, that were modified. I hope this approach is acceptable for you. Regards, Martin Sehnoutka >> question about migration to Git some time ago, is it being considered? > > The repo is replicated to github by script, I believe. > > Best regards, Wouter > >> >> Best regards, >> > > > > > _______________________________________________ > dnssec-trigger mailing list > dnssec-trigger at NLnetLabs.nl > https://open.nlnetlabs.nl/mailman/listinfo/dnssec-trigger > -- Martin Sehnoutka | Associate Software Engineer PGP: 5FD64AF5 UTC+1 (CET) RED HAT | TRIED. TESTED. TRUSTED. -------------- next part -------------- A non-text attachment was scrubbed... Name: 0001-Modify-the-build-system.patch Type: text/x-patch Size: 12506 bytes Desc: not available URL: -------------- next part -------------- A non-text attachment was scrubbed... Name: 0002-Introduce-string_buffer-and-string_list-types.patch Type: text/x-patch Size: 11536 bytes Desc: not available URL: -------------- next part -------------- A non-text attachment was scrubbed... Name: 0003-Import-JSON-parsing-library.patch Type: text/x-patch Size: 58025 bytes Desc: not available URL: -------------- next part -------------- A non-text attachment was scrubbed... Name: 0004-Connection-list-module.patch Type: text/x-patch Size: 16276 bytes Desc: not available URL: -------------- next part -------------- A non-text attachment was scrubbed... Name: 0005-Function-to-parse-JSON-into-connection-list.patch Type: text/x-patch Size: 4849 bytes Desc: not available URL: -------------- next part -------------- A non-text attachment was scrubbed... Name: 0006-Lock-module-used-to-serialize-execution-in-the-scrip.patch Type: text/x-patch Size: 3154 bytes Desc: not available URL: -------------- next part -------------- A non-text attachment was scrubbed... Name: 0007-Store-module-persistent-storage-used-by-the-script.patch Type: text/x-patch Size: 5182 bytes Desc: not available URL: -------------- next part -------------- A non-text attachment was scrubbed... Name: 0008-Testing-suite-for-previously-introduced-modules.patch Type: text/x-patch Size: 17200 bytes Desc: not available URL: -------------- next part -------------- A non-text attachment was scrubbed... Name: 0009-New-configuration-options-that-were-in-the-script.patch Type: text/x-patch Size: 2032 bytes Desc: not available URL: -------------- next part -------------- A non-text attachment was scrubbed... Name: 0010-Hook-unbound-control.patch Type: text/x-patch Size: 7991 bytes Desc: not available URL: -------------- next part -------------- A non-text attachment was scrubbed... Name: 0011-Reimplement-update-command-from-script-in-riggerd.patch Type: text/x-patch Size: 12263 bytes Desc: not available URL: -------------- next part -------------- A non-text attachment was scrubbed... Name: 0012-add-testing-file-for-global-forwarders-cache.patch Type: text/x-patch Size: 527 bytes Desc: not available URL: -------------- next part -------------- A non-text attachment was scrubbed... Name: signature.asc Type: application/pgp-signature Size: 833 bytes Desc: OpenPGP digital signature URL: