Skip to content

networkd: rename GatewayOnlink= to GatewayOnLink=#273

Merged
slyon merged 1 commit intomainfrom
slyon/networkd-onlink
Apr 26, 2022
Merged

networkd: rename GatewayOnlink= to GatewayOnLink=#273
slyon merged 1 commit intomainfrom
slyon/networkd-onlink

Conversation

@slyon
Copy link
Copy Markdown
Contributor

@slyon slyon commented Apr 22, 2022

Description

As of systemd v242 the "GatewayOnlink" spelling was deprecated and replaced by
"GatewayOnLink", while still being available in compat mode.

We've waited long enough and can switch to the proper spelling now.

systemd/systemd@9cb8c55

Checklist

  • Runs make check successfully.
  • Retains 100% code coverage (make check-coverage).
  • New/changed keys in YAML format are documented.
  • (Optional) Adds example YAML for new feature.
  • (Optional) Closes an open bug in Launchpad.

As of systemd v242 the "GatewayOnlink" spelling was deprecated and replaced by
"GatewayOnLink", while still being available in compat mode.

We've waited long enough and can switch to the proper spelling now.

systemd/systemd@9cb8c55
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #273 (1ab87ec) into main (a738597) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #273   +/-   ##
=======================================
  Coverage   99.07%   99.07%           
=======================================
  Files          61       61           
  Lines       10864    10864           
=======================================
  Hits        10764    10764           
  Misses        100      100           
Impacted Files Coverage Δ
tests/generator/test_routing.py 100.00% <ø> (ø)
src/networkd.c 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a738597...1ab87ec. Read the comment docs.

@slyon slyon requested a review from schopin-pro April 25, 2022 16:03
Copy link
Copy Markdown
Contributor

@schopin-pro schopin-pro left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, merging.

@schopin-pro
Copy link
Copy Markdown
Contributor

Well, would have merged but "branch out of date" :)

@slyon slyon merged commit 306600d into main Apr 26, 2022
@slyon slyon deleted the slyon/networkd-onlink branch April 26, 2022 08:18
@slyon
Copy link
Copy Markdown
Contributor Author

slyon commented Apr 26, 2022

Used my admin powers to merge it without rebase.

netdever added a commit to netdever/netplan that referenced this pull request Apr 27, 2022
* ABI compatibility checker (canonical#269)

* abigail: check-in 0.104 ABI

XML serialization generated on Focal via:
abidw libnetplan.so.0.0 --headers-dir include/ > abi-compat/focal_0.104.xml

* abi-compat: use abigail to check ABI compatibility via GA

* [WIP] src:types: move netplan_net_definition into a separate file

This is to allow ABI compatibilty checks of NetplanNetDefinition using the
'abidiff' tool.

* WIP: keep the ABI-specific structs self-contained

Also, only move them in include/ for the ABI check, so that they don't
get shipped as public headers.

* GA CI: update comments

* abi-compat: update XML according to latest state

* GA: upgrade to abigail-tools 2.0

Using PPA version from https://launchpad.net/~slyon/+archive/ubuntu/ci-netplan

* src: cleanup types.h ABI include

Co-authored-by: Simon Chopin <simon.chopin@canonical.com>

* networkd: rename GatewayOnlink= to GatewayOnLink= (canonical#273)

As of systemd v242 the "GatewayOnlink" spelling was deprecated and replaced by
"GatewayOnLink", while still being available in compat mode.

We've waited long enough and can switch to the proper spelling now.

systemd/systemd@9cb8c55

* cli:sriov: fix test coverage for quirk_devices

newer coverage tools seem to be more picky, make them happy

* Makefile: clean src/_features.h.gch file

* Adding VRF and VXLAN support for systemd-networkd. LP: #1764716, #1773522

* Add vxlan/vrf changes to new abi.h file

Co-authored-by: Lukas Märdian <slyon@ubuntu.com>
Co-authored-by: Simon Chopin <simon.chopin@canonical.com>
Co-authored-by: Anthony Timmins <atimmins@datto.com>
netdever added a commit to netdever/netplan that referenced this pull request Apr 27, 2022
* ABI compatibility checker (canonical#269)

* abigail: check-in 0.104 ABI

XML serialization generated on Focal via:
abidw libnetplan.so.0.0 --headers-dir include/ > abi-compat/focal_0.104.xml

* abi-compat: use abigail to check ABI compatibility via GA

* [WIP] src:types: move netplan_net_definition into a separate file

This is to allow ABI compatibilty checks of NetplanNetDefinition using the
'abidiff' tool.

* WIP: keep the ABI-specific structs self-contained

Also, only move them in include/ for the ABI check, so that they don't
get shipped as public headers.

* GA CI: update comments

* abi-compat: update XML according to latest state

* GA: upgrade to abigail-tools 2.0

Using PPA version from https://launchpad.net/~slyon/+archive/ubuntu/ci-netplan

* src: cleanup types.h ABI include

Co-authored-by: Simon Chopin <simon.chopin@canonical.com>

* networkd: rename GatewayOnlink= to GatewayOnLink= (canonical#273)

As of systemd v242 the "GatewayOnlink" spelling was deprecated and replaced by
"GatewayOnLink", while still being available in compat mode.

We've waited long enough and can switch to the proper spelling now.

systemd/systemd@9cb8c55

* cli:sriov: fix test coverage for quirk_devices

newer coverage tools seem to be more picky, make them happy

* Makefile: clean src/_features.h.gch file

Co-authored-by: Lukas Märdian <slyon@ubuntu.com>
Co-authored-by: Simon Chopin <simon.chopin@canonical.com>
Co-authored-by: Anthony Timmins <atimmins@datto.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants