nokia / danm

26%

master: 44%

LAST BUILD ON BRANCH tenantnet_setup

branch: tenantnet_setup

CHANGE BRANCH

x

Reset

• tenantnet_setup
• fixin_uts
• master
• cni_del_fix
• v3.2.0
• validation_type_change
• cmd_intro
• syncher_ut
• ipam_uts
• cnidel_steroid
• cnidel_uts
• prefix_fix
• fix_netype_bug
• webhook
• networktype_separation
• configure_naming
• v3.3.0
• new_apis
• logo-with-name
• validate_new_apis
• netwatcher_fourdoto
• fix_ip_delegation
• metacni_fourdoto
• slack-links
• svcwatcher_fourdoto
• fix_add_gc
• readme_facelift
• v4.0.0
• netdel_ut
• tconf_hook_ut
• akraino-install-guide-link
• fix_none_ip
• ipam_for_everyone
• category_all
• netadmit_uts
• device_handling_fixes
• netwatcher_modify
• update_delegation
• central_ipam_fixes
• validate_max_netmask
• rutas
• go_12_9
• perf_fix
• typo-correction
• wording/changes
• intel_dpdk
• binaries
• intel_vf_wa
• deploy-guide
• formatting
• fix_confman_core
• v4.1.0
• versioning
• concurrent_confman
• store_client
• ip6conf_for_none
• alloc6_api
• v6_alloc
• ipam_store_last_ip
• pod_id_to_ep
• ipv6_cudo
• outage
• svcw_update_fix
• svcwatcher_ns
• v4.2.0-RC1
• svcwatcher_update_eps
• route_man_fix
• v4.2.0-rc2
• ipam_prefix_change
• netwatcher_healthcheck
• dnet_apitype_fix
• svcwatcher_healthcheck
• v4.2.0-rc3
• dummy_encsa
• dis_dad
• uid_weeding
• set_ndisc_noti
• v4.2.0
• damn-danm
• add-copyright-headers
• route_device_fix
• dummy_cosmetics
• v4.2.1
• y_u_no_free
• v6_stat
• netwatcher_nad
• api_group_migration
• k8s_1.21_fix
• cni_res_fix

Build # 450

Build Type

Pull #94

travis-ci

Committed by web-flow

Commit Message

Fixing bugs related to Tenantconfig handling.

1: There was a nil pointer reference error in metacni during creation of DANM client
2: IsDynamicBackend incorrectly did not recognize ipvlan as such
3: Accidentally used shallow copy was changed to a (manual, sigh) deep copy when we save the original manifest during net, and conf validation
4: HostDevices list was not properly patched. When changed, the whole JSON list is now put together, and sent for replacement
5: When a host_device or devicepool is explicitly requested, but is not configured, we throw a validation error
6: hostDevices are now filtered, so DevicePools are not accidentally randomly allocated as host_device for a TenantNetwork
7: Mutating VLAN and VxLAN types for accidentally switched up due to wrong condition
8: NetworkIDs were not mutated into the networks because we haven't checked whether they even changed
9: NEtworkID pre-validation was accidentally removed from ClusterNetwork, not from TenantNetwork :)

DISCLAIMER: Freeing VNIs in TenantConfig does not work because K8s does not fill ANY Objects in DELETE requests!!!
Fortunetaly correction is merged, but we need to wait for 1.15.
Til then - no freeing of VNIs.

Pull Request Pull Request #94: DANM 4.0 EP4: Adding the TenantNetwork mutating code to the Webhook.

Run Details

491 of 491 new or added lines in 9 files covered. (100.0%)

541 of 2042 relevant lines covered (26.49%)

6.74 hits per line