psafont / xen-api

80%

master: 78%

LAST BUILD ON BRANCH dev/pau/ipv6-autoconf

branch: dev/pau/ipv6-autoconf

CHANGE BRANCH

x

Reset

Sync Branches

• dev/pau/ipv6-autoconf
• 52
• bump
• certdocs
• ci
• cifix
• codecov-fallback
• copam
• deb/pau/important-backtraces-8
• dev/pau-vdi-revert-alpha
• dev/pau/53
• dev/pau/541
• dev/pau/bye-yangtze
• dev/pau/cmdliner
• dev/pau/cross-pool-fix
• dev/pau/errors-ha
• dev/pau/errors-ha-master
• dev/pau/format
• dev/pau/ha_errs
• dev/pau/haxn
• dev/pau/host-ipv6
• dev/pau/important-backtraces
• dev/pau/ipv6-autoconf-backport
• dev/pau/listless
• dev/pau/majmin
• dev/pau/migracap
• dev/pau/no-exns
• dev/pau/nocov
• dev/pau/ocaml54
• dev/pau/oformat.28.1
• dev/pau/opam
• dev/pau/opam-metadata
• dev/pau/prepof28.1
• dev/pau/quality
• dev/pau/random-fixes
• dev/pau/release-8.3
• dev/pau/revert-cds
• dev/pau/revlog
• dev/pau/revlog-9
• dev/pau/schemabump
• dev/pau/smplugins
• dev/pau/smplugins-9
• dev/pau/snaptags
• dev/pau/sr-description
• dev/pau/sr-description-83
• dev/pau/sr-suspend
• dev/pau/stringless
• dev/pau/strings
• dev/pau/stunnel
• dev/pau/system-status
• dev/pau/tweaks
• dev/pau/vdi-revert
• dev/pau/vdi-revert-alpha
• dev/pau/vdi-revert-doc
• dev/pau/version
• dev/pau/vm-revert-master
• dev/pau/zerotest
• dev/paus/delays
• dev/paus/platformdata
• dev/psafont/arg
• dev/psafont/arm64
• dev/psafont/dcmi
• dev/psafont/format
• dev/psafont/makemake
• dev/psafont/onlink
• dev/psafont/python
• dev/psafont/ssh-design
• dev/vdi-revert-prepare
• dev/vdi-revert-prepare-9
• dune-mswitch
• dune-vhd-tool
• endcover
• feature/perf-edvint
• fix-pkgs
• fixes-next
• forkexec+
• forktests
• getallwhere
• ghperms
• gtn-check-all-changes-during-sr-scan
• hold
• info
• instafix
• last_async
• late-licenses
• logconn
• makemake
• master
• meta
• metacap
• no-async
• nosystemd
• nowarns
• nuuid
• omore
• packs
• parops
• pau/dev/cross-pool-fix
• pkg-fix
• private/mingl/CP-44103
• private/paus/accept-rrd
• private/paus/async-mig
• private/paus/async_mig
• private/paus/bashcheck
• private/paus/console_api
• private/paus/datasource-confusion
• private/paus/dev/setup
• private/paus/dns
• private/paus/dopon
• private/paus/duneless-release
• private/paus/fingers-crossed
• private/paus/forktime
• private/paus/gencertg
• private/paus/grandates
• private/paus/hold-name-label
• private/paus/ign
• private/paus/ipmissing
• private/paus/ipv6
• private/paus/ipv6-hack
• private/paus/its-about-mtime
• private/paus/late-socket
• private/paus/logs
• private/paus/logsig
• private/paus/minions
• private/paus/netconf
• private/paus/new-crypto
• private/paus/new-numa
• private/paus/nopes
• private/paus/numa
• private/paus/numa-claim
• private/paus/numa-default
• private/paus/numa-hard-feature
• private/paus/numa-mem-metrics
• private/paus/numa-share
• private/paus/numa-xs8
• private/paus/numaybe
• private/paus/numnummaybe
• private/paus/one-unlock
• private/paus/op-ehm
• private/paus/order_some_plugs
• private/paus/osandbox
• private/paus/paraxenopers
• private/paus/parops
• private/paus/parsed-pems
• private/paus/paused-metrics
• private/paus/pdates
• private/paus/perfuffle
• private/paus/pimpampem
• private/paus/pre-numa
• private/paus/pvhterm
• private/paus/random-fixes
• private/paus/scrub
• private/paus/sd_notify
• private/paus/smint
• private/paus/stdless
• private/paus/storage-errors
• private/paus/stringy-crypto
• private/paus/tears-in-the-rain
• private/paus/timesout
• private/paus/undate
• private/paus/unknown-error
• private/paus/upup
• private/paus/validation
• private/paus/vhd-cleanup
• private/paus/xenops-long-waits
• private/paus/xpool
• private/paus/xs-up
• qcheck-core
• qgate
• release-acquire
• rrd-unpack
• runs
• sad-dco
• scheduled-yangtze
• stressdune
• strip_xfm
• test-new-setup
• tests
• trunc_claims
• tryshut
• unit-test
• unzurich
• up
• utop
• v25.1.0
• v25.10.0
• v25.11.0
• v25.12.0
• v25.13.0
• v25.14.0
• v25.15.0
• v25.16.0
• v25.17.0
• v25.18.0
• v25.19.0
• v26.1.3-8.3
• xen_types

Build # 17643203587

Build Type

push

github

Committed by psafont

Commit Message

xapi-idl/network: Remove code duplication for DNS persistence decisions

Previously both xapi and networkd had to inspect the IP configuration to decide
whether the DNS values should be persistend into /etc/resolv.conf. This
actually lead to a mismatch in them. Instead use an option value for DNS that
simply means that if there's a value, it must be persisted.

Now xapi decides the instances where these values are written.

Treat a couple of empty lists as a lack of value to avoid writing empty
resolv.conf files. This can happen when updating a host from previous
versions, which use empty lists when using DHCP.

Tested manually by installing a version with this change and restarting
the toolstack. The file is kept intact, unlike the previous version of
the change that did not take into account the update behaviour.

Signed-off-by: Pau Ruiz Safont <pau.safont@vates.tech>

Coverage Stats

3496 of 4350 relevant lines covered (80.37%)

0.8 hits per line