rtr: Fix incomplete undo after failed table update#281
rtr: Fix incomplete undo after failed table update#281carl-tud wants to merge 2 commits intortrlib:masterfrom
Conversation
Continuous Integration Result: FAILEDSee below for issues. This is a comment from an automated CI system. Get source / Pull Request: SuccessfulBuilding Stage: FailedUbuntu 20.04 arm8: Failed (click for details)Make Debug failed for Ubuntu 20.04 arm8: Ubuntu 18.04 ppc64le: Failed (click for details)Make Debug failed for Ubuntu 18.04 ppc64le: Ubuntu 18.04 arm8: Failed (click for details)Make Debug failed for Ubuntu 18.04 arm8: OpenBSD 7 amd64: Failed (click for details)Make Debug failed for OpenBSD 7 amd64: Debian 11 amd64: Failed (click for details)Make Debug failed for Debian 11 amd64: FreeBSD 12 amd64: Failed (click for details)Make Debug failed for FreeBSD 12 amd64: Debian 10 amd64: Failed (click for details)Make Debug failed for Debian 10 amd64: FreeBSD 11 amd64: Failed (click for details)Make Debug failed for FreeBSD 11 amd64: Redhat 8 amd64: Failed (click for details)Make Debug failed for Redhat 8 amd64: Fedora 35 amd64: Failed (click for details)__Unittests (make test) failed for Fedora 35 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-411/artifact/FEDORA35/ErrorLog/log_make_test_debug.txtNetBSD 9 amd64: Failed (click for details)Make Debug failed for NetBSD 9 amd64: Redhat 9 amd64: Failed (click for details)__Unittests (make test) failed for Redhat 9 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-411/artifact/RH9BUILD/ErrorLog/log_make_test_debug.txtFedora 31 amd64: Failed (click for details)Make Debug failed for Fedora 31 amd64: Fedora 29 amd64: Failed (click for details)Make Debug failed for Fedora 29 amd64: Ubuntu 18.04 amd64: Failed (click for details)Make Debug failed for Ubuntu 18.04 amd64: Ubuntu 18.04 i386: Failed (click for details)Make Debug failed for Ubuntu 18.04 i386: CentOS 7 amd64: Failed (click for details)Make Debug failed for CentOS 7 amd64: Ubuntu 18.04 arm7: Failed (click for details)Make Debug failed for Ubuntu 18.04 arm7: |
370ebc4 to
34be75e
Compare
Continuous Integration Result: FAILEDSee below for issues. This is a comment from an automated CI system. Get source / Pull Request: SuccessfulBuilding Stage: FailedUbuntu 20.04 arm8: Failed (click for details)__Unittests (make test) failed for Ubuntu 20.04 arm8__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-412/artifact/U2004ARM8/ErrorLog/log_make_test_debug.txtUbuntu 18.04 arm8: Failed (click for details)__Unittests (make test) failed for Ubuntu 18.04 arm8__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-412/artifact/U1804ARM8/ErrorLog/log_make_test_debug.txtUbuntu 18.04 ppc64le: Failed (click for details)__Unittests (make test) failed for Ubuntu 18.04 ppc64le__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-412/artifact/U1804PPC64LE/ErrorLog/log_make_test_debug.txtUbuntu 18.04 amd64: Failed (click for details)__Unittests (make test) failed for Ubuntu 18.04 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-412/artifact/U1804AMD64/ErrorLog/log_make_test_debug.txtDebian 11 amd64: Failed (click for details)__Unittests (make test) failed for Debian 11 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-412/artifact/DEB11AMD64/ErrorLog/log_make_test_debug.txtFreeBSD 12 amd64: Failed (click for details)__Unittests (make test) failed for FreeBSD 12 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-412/artifact/FBSD12AMD64/ErrorLog/log_make_test_debug.txtOpenBSD 7 amd64: Failed (click for details)__Unittests (make test) failed for OpenBSD 7 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-412/artifact/OBSD6AMD64/ErrorLog/log_make_test_debug.txtRedhat 8 amd64: Failed (click for details)__Unittests (make test) failed for Redhat 8 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-412/artifact/REDHAT8/ErrorLog/log_make_test_debug.txtFreeBSD 11 amd64: Failed (click for details)__Unittests (make test) failed for FreeBSD 11 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-412/artifact/FBSD11AMD64/ErrorLog/log_make_test_debug.txtNetBSD 9 amd64: Failed (click for details)__Unittests (make test) failed for NetBSD 9 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-412/artifact/NBSD7AMD64/ErrorLog/log_make_test_debug.txtFedora 35 amd64: Failed (click for details)__Unittests (make test) failed for Fedora 35 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-412/artifact/FEDORA35/ErrorLog/log_make_test_debug.txtRedhat 9 amd64: Failed (click for details)__Unittests (make test) failed for Redhat 9 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-412/artifact/RH9BUILD/ErrorLog/log_make_test_debug.txtFedora 31 amd64: Failed (click for details)__Unittests (make test) failed for Fedora 31 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-412/artifact/FED31BUILD/ErrorLog/log_make_test_debug.txtDebian 10 amd64: Failed (click for details)__Unittests (make test) failed for Debian 10 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-412/artifact/DEB10AMD64/ErrorLog/log_make_test_debug.txtUbuntu 18.04 arm7: Failed (click for details)__Unittests (make test) failed for Ubuntu 18.04 arm7__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-412/artifact/U1804ARM7/ErrorLog/log_make_test_debug.txtFedora 29 amd64: Failed (click for details)__Unittests (make test) failed for Fedora 29 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-412/artifact/FEDORA29AMD64/ErrorLog/log_make_test_debug.txtCentOS 7 amd64: Failed (click for details)__Unittests (make test) failed for CentOS 7 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-412/artifact/CENTOS7AMD64/ErrorLog/log_make_test_debug.txtUbuntu 18.04 i386: Failed (click for details)__Unittests (make test) failed for Ubuntu 18.04 i386__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-412/artifact/U18I386/ErrorLog/log_make_test_debug.txt |
|
|
|
thanks @carl-tud. yes, rpki-validator.realmv6.org is down. i will check later. |
|
brief update: we will set up a new validator, but it will take some days due to internal clearance procedures. @Nimuedaertya will take care of this. |
mroethke
left a comment
There was a problem hiding this comment.
Looks good on a cursory glance and does seem to work, but I won't have time to do a thorough review for at least several weeks.
34be75e to
1bbb302
Compare
Continuous Integration Result: FAILEDSee below for issues. This is a comment from an automated CI system. Get source / Pull Request: SuccessfulBuilding Stage: FailedFedora 35 amd64: Failed (click for details)__Unittests (make test) failed for Fedora 35 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-421/artifact/FEDORA35/ErrorLog/log_make_test_debug.txtUbuntu 18.04 arm7: Failed (click for details)__Unittests (make test) failed for Ubuntu 18.04 arm7__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-421/artifact/U1804ARM7/ErrorLog/log_make_test_debug.txtRedhat 8 amd64: Failed (click for details)__Unittests (make test) failed for Redhat 8 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-421/artifact/REDHAT8/ErrorLog/log_make_test_debug.txtFedora 29 amd64: Failed (click for details)__Unittests (make test) failed for Fedora 29 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-421/artifact/FEDORA29AMD64/ErrorLog/log_make_test_debug.txtNetBSD 9 amd64: Failed (click for details)__Unittests (make test) failed for NetBSD 9 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-421/artifact/NBSD7AMD64/ErrorLog/log_make_test_debug.txtUbuntu 18.04 i386: Failed (click for details)__Unittests (make test) failed for Ubuntu 18.04 i386__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-421/artifact/U18I386/ErrorLog/log_make_test_debug.txtDebian 12 amd64: Failed (click for details)__Unittests (make test) failed for Debian 12 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-421/artifact/D12AMD64/ErrorLog/log_make_test_debug.txtUbuntu 20.04 arm8: Failed (click for details)__Unittests (make test) failed for Ubuntu 20.04 arm8__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-421/artifact/U2004ARM8/ErrorLog/log_make_test_debug.txtUbuntu 18.04 arm8: Failed (click for details)__Unittests (make test) failed for Ubuntu 18.04 arm8__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-421/artifact/U1804ARM8/ErrorLog/log_make_test_debug.txtOpenBSD 7 amd64: Failed (click for details)__Unittests (make test) failed for OpenBSD 7 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-421/artifact/OBSD6AMD64/ErrorLog/log_make_test_debug.txtUbuntu 18.04 amd64: Failed (click for details)__Unittests (make test) failed for Ubuntu 18.04 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-421/artifact/U1804AMD64/ErrorLog/log_make_test_debug.txtCentOS 7 amd64: Failed (click for details)__Unittests (make test) failed for CentOS 7 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-421/artifact/CENTOS7AMD64/ErrorLog/log_make_test_debug.txtFedora 31 amd64: Failed (click for details)__Unittests (make test) failed for Fedora 31 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-421/artifact/FED31BUILD/ErrorLog/log_make_test_debug.txtRedhat 9 amd64: Failed (click for details)__Unittests (make test) failed for Redhat 9 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-421/artifact/RH9BUILD/ErrorLog/log_make_test_debug.txtDebian 11 amd64: Failed (click for details)__Unittests (make test) failed for Debian 11 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-421/artifact/DEB11AMD64/ErrorLog/log_make_test_debug.txtUbuntu 18.04 ppc64le: Failed (click for details)__Unittests (make test) failed for Ubuntu 18.04 ppc64le__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-421/artifact/U1804PPC64LE/ErrorLog/log_make_test_debug.txtFreeBSD 12 amd64: Failed (click for details)__Unittests (make test) failed for FreeBSD 12 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-421/artifact/FBSD12AMD64/ErrorLog/log_make_test_debug.txtFreeBSD 11 amd64: Failed (click for details)__Unittests (make test) failed for FreeBSD 11 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-421/artifact/FBSD11AMD64/ErrorLog/log_make_test_debug.txtDebian 10 amd64: Failed (click for details)__Unittests (make test) failed for Debian 10 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-421/artifact/DEB10AMD64/ErrorLog/log_make_test_debug.txt |
|
CI:rerun |
|
CI:rerun |
|
CI:rerun |
2 similar comments
|
CI:rerun |
|
CI:rerun |
Continuous Integration Result: FAILEDTest incomplete. See below for issues. This is a comment from an automated CI system. Get source / Pull Request: SuccessfulBuilding Stage: IncompleteFedora 35 amd64: Incomplete(check logs for details)Ubuntu 18.04 i386: Failed (click for details)__Unittests (make test) failed for Ubuntu 18.04 i386__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-424/artifact/U18I386/ErrorLog/log_make_test_debug.txtUbuntu 18.04 arm7: Failed (click for details)__Unittests (make test) failed for Ubuntu 18.04 arm7__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-424/artifact/U1804ARM7/ErrorLog/log_make_test_debug.txtUbuntu 18.04 ppc64le: Failed (click for details)__Unittests (make test) failed for Ubuntu 18.04 ppc64le__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-424/artifact/U1804PPC64LE/ErrorLog/log_make_test_debug.txtSuccessful on other platforms/tests
|
|
the results of the CI tests are very confusing. rerunning the CI leads to different results. sometimes the live test_live_validation and test_dynamic_groups fail, sometimes they pass. if we should have stable CI tests before merging. |
Continuous Integration Result: FAILEDTest incomplete. See below for issues. This is a comment from an automated CI system. Get source / Pull Request: SuccessfulBuilding Stage: IncompleteRedhat 8 amd64: Failed (click for details)__Unittests (make test) failed for Redhat 8 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-425/artifact/REDHAT8/ErrorLog/log_make_test_debug.txtFedora 29 amd64: Failed (click for details)__Unittests (make test) failed for Fedora 29 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-425/artifact/FEDORA29AMD64/ErrorLog/log_make_test_debug.txtRedhat 9 amd64: Failed (click for details)__Unittests (make test) failed for Redhat 9 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-425/artifact/RH9BUILD/ErrorLog/log_make_test_debug.txtFedora 31 amd64: Failed (click for details)__Unittests (make test) failed for Fedora 31 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-425/artifact/FED31BUILD/ErrorLog/log_make_test_debug.txtCentOS 7 amd64: Failed (click for details)__Unittests (make test) failed for CentOS 7 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-425/artifact/CENTOS7AMD64/ErrorLog/log_make_test_debug.txtOpenBSD 7 amd64: Failed (click for details)__Unittests (make test) failed for OpenBSD 7 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-425/artifact/OBSD6AMD64/ErrorLog/log_make_test_debug.txtUbuntu 18.04 arm7: Failed (click for details)__Unittests (make test) failed for Ubuntu 18.04 arm7__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-425/artifact/U1804ARM7/ErrorLog/log_make_test_debug.txtUbuntu 18.04 ppc64le: Failed (click for details)__Unittests (make test) failed for Ubuntu 18.04 ppc64le__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-425/artifact/U1804PPC64LE/ErrorLog/log_make_test_debug.txtUbuntu 20.04 arm8: Failed (click for details)__Unittests (make test) failed for Ubuntu 20.04 arm8__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-425/artifact/U2004ARM8/ErrorLog/log_make_test_release.txtUbuntu 18.04 arm8: Failed (click for details)__Unittests (make test) failed for Ubuntu 18.04 arm8__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-425/artifact/U1804ARM8/ErrorLog/log_make_test_release.txtDebian 11 amd64: Failed (click for details)__Unittests (make test) failed for Debian 11 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-425/artifact/DEB11AMD64/ErrorLog/log_make_test_debug.txtFreeBSD 12 amd64: Failed (click for details)__Unittests (make test) failed for FreeBSD 12 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-425/artifact/FBSD12AMD64/ErrorLog/log_make_test_debug.txtDebian 10 amd64: Failed (click for details)__Unittests (make test) failed for Debian 10 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-425/artifact/DEB10AMD64/ErrorLog/log_make_test_debug.txtFreeBSD 11 amd64: Failed (click for details)__Unittests (make test) failed for FreeBSD 11 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-425/artifact/FBSD11AMD64/ErrorLog/log_make_test_debug.txtFedora 35 amd64: Incomplete(check logs for details)Successful on other platforms/tests
|
|
my suggestion is that @Nimuedaertya creates a dummy PR that doesn't include any code changes to better understand what is going on with the CI. |
Continuous Integration Result: FAILEDTest incomplete. See below for issues. This is a comment from an automated CI system. Get source / Pull Request: SuccessfulBuilding Stage: IncompleteDebian 12 amd64: Failed (click for details)__Unittests (make test) failed for Debian 12 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-426/artifact/D12AMD64/ErrorLog/log_make_test_debug.txtFedora 31 amd64: Failed (click for details)__Unittests (make test) failed for Fedora 31 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-426/artifact/FED31BUILD/ErrorLog/log_make_test_debug.txtRedhat 9 amd64: Failed (click for details)__Unittests (make test) failed for Redhat 9 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-426/artifact/RH9BUILD/ErrorLog/log_make_test_debug.txtUbuntu 18.04 amd64: Failed (click for details)__Unittests (make test) failed for Ubuntu 18.04 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-426/artifact/U1804AMD64/ErrorLog/log_make_test_debug.txtUbuntu 18.04 i386: Failed (click for details)__Unittests (make test) failed for Ubuntu 18.04 i386__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-426/artifact/U18I386/ErrorLog/log_make_test_debug.txtFedora 29 amd64: Failed (click for details)__Unittests (make test) failed for Fedora 29 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-426/artifact/FEDORA29AMD64/ErrorLog/log_make_test_debug.txtCentOS 7 amd64: Failed (click for details)__Unittests (make test) failed for CentOS 7 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-426/artifact/CENTOS7AMD64/ErrorLog/log_make_test_debug.txtUbuntu 18.04 arm7: Failed (click for details)__Unittests (make test) failed for Ubuntu 18.04 arm7__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-426/artifact/U1804ARM7/ErrorLog/log_make_test_debug.txtUbuntu 18.04 ppc64le: Failed (click for details)__Unittests (make test) failed for Ubuntu 18.04 ppc64le__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-426/artifact/U1804PPC64LE/ErrorLog/log_make_test_debug.txtUbuntu 20.04 arm8: Failed (click for details)__Unittests (make test) failed for Ubuntu 20.04 arm8__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-426/artifact/U2004ARM8/ErrorLog/log_make_test_release.txtUbuntu 18.04 arm8: Failed (click for details)__Unittests (make test) failed for Ubuntu 18.04 arm8__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-426/artifact/U1804ARM8/ErrorLog/log_make_test_release.txtOpenBSD 7 amd64: Failed (click for details)__Unittests (make test) failed for OpenBSD 7 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-426/artifact/OBSD6AMD64/ErrorLog/log_make_test_debug.txtDebian 10 amd64: Failed (click for details)__Unittests (make test) failed for Debian 10 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-426/artifact/DEB10AMD64/ErrorLog/log_make_test_debug.txtDebian 11 amd64: Failed (click for details)__Unittests (make test) failed for Debian 11 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-426/artifact/DEB11AMD64/ErrorLog/log_make_test_debug.txtFreeBSD 12 amd64: Failed (click for details)__Unittests (make test) failed for FreeBSD 12 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-426/artifact/FBSD12AMD64/ErrorLog/log_make_test_debug.txtRedhat 8 amd64: Failed (click for details)__Unittests (make test) failed for Redhat 8 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-426/artifact/REDHAT8/ErrorLog/log_make_test_debug.txtFreeBSD 11 amd64: Failed (click for details)__Unittests (make test) failed for FreeBSD 11 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-426/artifact/FBSD11AMD64/ErrorLog/log_make_test_debug.txtFedora 35 amd64: Incomplete(check logs for details)NetBSD 9 amd64: Failed (click for details)__Unittests (make test) failed for NetBSD 9 amd64__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-426/artifact/NBSD7AMD64/ErrorLog/log_make_test_debug.txt |
|
At least for me, locally, |
yes, the validator was not running stable. we fixed this today morning. |
|
CI:rerun |
Continuous Integration Result: FAILEDTest incomplete. See below for issues. This is a comment from an automated CI system. Get source / Pull Request: SuccessfulBuilding Stage: IncompleteUbuntu 18.04 ppc64le: Failed (click for details)__Unittests (make test) failed for Ubuntu 18.04 ppc64le__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-432/artifact/U1804PPC64LE/ErrorLog/log_make_test_debug.txtUbuntu 18.04 arm7: Failed (click for details)__Unittests (make test) failed for Ubuntu 18.04 arm7__ see Unittest log at https://ci1.netdef.org/browse/RPKI-RTRLIBPR-432/artifact/U1804ARM7/ErrorLog/log_make_test_debug.txtFedora 35 amd64: Incomplete(check logs for details)Successful on other platforms/tests
|
|
@carl-tud what is the status of this PR? |
|
The changes I‘ve made for this PR are included in #285 . This PR was just to get some of the changes not directly related ASPA, but the update logic out before the final ASPA PR. Do you think we should still merge this PR first? |
|
@carl-tud as you prefer :-) |
If a table update fails, changes made to the prefix and spki table must be reverted/undone. If a table undo operation fails, changes to the other table should still be undone for a complete undo.
fixes a leak in `recv_loop_cleanup` where PDU pointers in `recv_loop_cleanup_args` are `NULL`.
Sorry for the late reply, in theory I'd be totally okay with that, but, reading #285 it looks like ASPA is going to be introduced as part of a new version (with potentially source-breaking changes), so this might be needed as bugfix for those temporarily staying on the older version..? |
|
CI:rerun |
|
CI:rerun |
|
CI:rerun edit: |
Contribution description
This PR restructures
rtr_sync_receive_and_store_pdusin order to fully complete an undo operation after a failed update to the prefix or spki table. Currently, changes to the respective tables are not entirely reverted if a single table undo fails. The update and undo logic was moved into separate functions, makingrtr_sync_receive_and_store_pdusmore comprehensible.Additionally, this PR fixes a memory leak involving
recv_loop_cleanup_argswhere the references to the PDUs saved temporarily aren't stored properly.