Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

chore(deps): update all dependencies on main (main) #326

Merged
merged 10 commits into from
Aug 19, 2024
Merged

chore(deps): update all dependencies on main (main) #326

merged 10 commits into from
Aug 19, 2024

Conversation

redhat-renovate-bot
Copy link
Collaborator

@redhat-renovate-bot redhat-renovate-bot commented Aug 15, 2024 •
edited by phoracek
Loading

This is a major bump of Go, Go dependencies and GitHub Actions dependencies. It was originally issued by Renovate, but making this pass required lot of manual work too.

I recommend going through this commit by commit. Some of the changes are pure bumps, others replace deprecated function calls. The last change is adding an explicit 'state up' configuration to the container netns veth end, since the function used to create the veth pair is no longer doing it for us.

Original commit message from Renovate:

This PR contains the following updates:

Package Type Update Change
actions/checkout action major v2 -> v4
docker/build-push-action action major v5 -> v6
github.com/Mellanox/sriovnet require minor v1.0.2 -> v1.2.0
github.com/containernetworking/cni require minor v1.0.1 -> v1.2.3
github.com/containernetworking/plugins require minor v1.0.1 -> v1.5.1
github.com/golang/glog require minor v1.1.0 -> v1.2.2
github.com/imdario/mergo require major v0.3.12 -> v1.0.0
github.com/j-keck/arping require patch v1.0.2 -> v1.0.3
github.com/k8snetworkplumbingwg/network-attachment-definition-client require major v0.0.0-20200626054723-37f83d1996bc -> v1.7.1
github.com/onsi/ginkgo require major v1.16.4 -> v2.20.0
github.com/onsi/gomega require minor v1.31.0 -> v1.34.1
github.com/ovn-org/libovsdb require minor v0.6.0 -> v0.7.0
github.com/vishvananda/netlink require minor v1.1.1-0.20210330154013-f5de75959ad5 -> v1.2.1-beta.2
go (source) toolchain minor 1.22.5 -> 1.23.0
k8s.io/api replace minor v0.30.3 -> v0.31.0
k8s.io/api require minor v0.30.3 -> v0.31.0
k8s.io/apiextensions-apiserver replace minor v0.30.3 -> v0.31.0
k8s.io/apimachinery replace minor v0.30.3 -> v0.31.0
k8s.io/apimachinery require minor v0.30.3 -> v0.31.0
k8s.io/apiserver replace minor v0.30.3 -> v0.31.0
k8s.io/cli-runtime replace minor v0.30.3 -> v0.31.0
k8s.io/client-go replace minor v0.30.3 -> v0.31.0
k8s.io/client-go require minor v0.30.3 -> v0.31.0
k8s.io/cloud-provider replace minor v0.30.3 -> v0.31.0
k8s.io/cluster-bootstrap replace minor v0.30.3 -> v0.31.0
k8s.io/code-generator replace minor v0.30.3 -> v0.31.0
k8s.io/component-base replace minor v0.30.3 -> v0.31.0
k8s.io/cri-api replace minor v0.30.3 -> v0.31.0
k8s.io/csi-translation-lib replace minor v0.30.3 -> v0.31.0
k8s.io/kube-aggregator replace minor v0.30.3 -> v0.31.0
k8s.io/kube-controller-manager replace minor v0.30.3 -> v0.31.0
k8s.io/kube-proxy replace minor v0.30.3 -> v0.31.0
k8s.io/kube-scheduler replace minor v0.30.3 -> v0.31.0
k8s.io/kubectl replace minor v0.30.3 -> v0.31.0
k8s.io/kubelet replace minor v0.30.3 -> v0.31.0
k8s.io/legacy-cloud-providers replace patch v0.30.3 -> v0.30.4
k8s.io/metrics replace minor v0.30.3 -> v0.31.0
k8s.io/sample-apiserver replace minor v0.30.3 -> v0.31.0
kubevirt.io/qe-tools require patch v0.1.6 -> v0.1.8

Release Notes

actions/checkout (actions/checkout)

v4

Compare Source

v3

Compare Source

docker/build-push-action (docker/build-push-action)

v6

Compare Source

Mellanox/sriovnet (github.com/Mellanox/sriovnet)

v1.2.0

Compare Source

What's Changed

Full Changelog: k8snetworkplumbingwg/sriovnet@v1.1.0...v1.2.0

v1.1.0

Compare Source

What's Changed

Full Changelog: k8snetworkplumbingwg/sriovnet@v1.0.3...v1.1.0

v1.0.3

Compare Source

  • Extend smartnic support #​37
  • Skip non representors netdev #​38
  • Rename GetRepresentorMacAddress #​39
  • Use devlink for smartnic #​40
  • an API to check if VF is bound to vfio-pci driver #​41
  • an API to set the host VF's mac through its representor #​42
  • Change default controller ID to 1 #​43
  • Refactoring - Change Smart-NIC to DPU #​44
  • Extend GetUplinkRepresentor to work with PF argument #​46
containernetworking/cni (github.com/containernetworking/cni)

v1.2.3: libcni v1.2.3

Compare Source

This is a minor release to correct a divergence between the specification and libcni. In this case, the specification was updated, as it concerns a new feature, GC, that is not yet deployed.

What's Changed

v1.2.2: libcni v1.2.2

Compare Source

What's Changed

Full Changelog: containernetworking/cni@v1.2.1...v1.2.2

v1.2.1: CNI Spec v1.1 / libcni v1.2.1

Compare Source

This is a minor release that fixes some small bugs:

Improvements
  • (#​1098). SPEC: add warning about preserving shared resources for GC
  • (#​1090). Spec, libcni: add disableGC flag This allows administrators to disable garbage collection in exceptional circumstances, such as multiple runtimes sharing a network configuration.
Bug fixes:
  • (#​1097). Fix faulty json.Marshal behavior for embeds types.NetConf Fix #​1096
  • (#​1095). libcni: always delete the cache on conflist for CNI DEL This aligns the call with DelNetwork, and allows CRIO tests to bump

v1.2.0: CNI spec v1.1 / libcni v1.2.0

Compare Source

This is libcni v1.2.0, which releases the new CNI spec version v1.1.0.

Major changes:

GC verb

The GC verb allows runtimes to specify the set of known-good attachments, allowing plugins to clean up stale and leaked resources such as IPAM reservations. Libcni will also synthesize a CNI DEL for any stale cached attachments, so all users will have a form of GC, even if their plugins do not support CNI v1.1

STATUS verb

The STATUS verb allows a plugin to report its readiness to accept ADD requests. Runtimes such as containerd and cri-o will no longer have to rely merely on the presence of a CNI configuration file to determine network readiness.

Version negotiation

CNI configurations may now contain multiple versions, so that an administrator can opportunistically update the protocol version without breaking older runtimes:

{
  "cniVersion": "1.0.0",
  "cniVersions": ["1.0.0", "1.1.0"]
}

New fields

A number of new fields have been added to the result type:

Other improvements

  • (#​911). libcni: add specific type for CHECK not supported
  • (#​1072). tolerate invalid cni caches for deletion
  • (#​1054). Add Version() to CNI interface

v1.1.2: CNI v1.1.2

Compare Source

This is a minor update to the CNI libraries and tooling. This does not bump the protocol / spec version, which remains at v1.0.0.

What's Changed

New Contributors

Full Changelog: containernetworking/cni@v1.1.1...v1.1.2

v1.1.1: CNI v1.1.1

Compare Source

This is a minor update to the CNI libraries and tooling. This does not bump the protocol / spec version, which remains at v1.0.0.

Changes include:

New Contributors:

Full Changelog: containernetworking/cni@v1.1.0...v1.1.1

v1.1.0: CNI v1.1.0

Compare Source

This is a minor update to the CNI libraries and tooling. This does not bump the protocol / spec version, which remains at v1.0.0.

Changes include:

  • libcni: handle empty version when parsing version (#​893). Without this, Delete failed for empty-version configs, which was a regression from v0.8.0.
  • Fix incorrect pointer inputs to json.Unmarshal (#​880).
  • [exec-plugins]: support plugin lists (#​865).
  • skel: remove superfluous err nil check in (*dispatcher).pluginMain (#​864).
  • skel: print out CNI versions supported in help text (#​860).
containernetworking/plugins (github.com/containernetworking/plugins)

v1.5.1

Compare Source

What's Changed

Full Changelog: containernetworking/plugins@v1.5.0...v1.5.1

v1.5.0: CNI Plugins v1.5.0

Compare Source

What's Changed

New Features:

Fixes:

New Contributors

Full Changelog: containernetworking/plugins@v1.4.1...v1.5.0

v1.4.1

Compare Source

v1.4.0: CNI Plugins v1.4.0

Compare Source

New features:
  • (#​832). tap: allow for a tap device to be created as a bridge port
  • (#​914). [tuning] add ability to set tx queue len
Improvements:
  • (#​969). Add CNI_NETNS_OVERRIDE
  • (#​979). Add ndisc_notify in ipvlan for ipv6 ndp
  • (#​974). macvlan: enable ipv6 ndisc_notify
  • (#​950). Create IPAM files with 0600 permissions
  • (#​924). More efficient iptables usage.
  • (#​902). spoofcheck: Make use of go-nft's ApplyConfigEcho(). This is much faster
  • (#​874). Add routes propagation for VRF plugin
Build:
  • (#​982). Bump to golang:1.21-alpine
  • (#​948). build: Use POSIX sh for shell scripts
Bug fixes:
  • (#​954). macvlan cmdDel: handle deletion when master has been deleted
  • (#​927). vrf: fix route filter to use output iface

v1.3.0: CNI Plugins v1.3.0

Compare Source

This release introduces a new plugin: tap. Thanks to @​mmirecki for contributing this

New features:

  • (#​784). tap: This PR adds a plugin to create tap devices.
  • (#​829). bridge: add vlan trunk support
  • (#​875). bridge: Add parameter to disable default vlan
  • (#​814). macvlan: Add support for in-container master
  • (#​813). ipvlan: Add support for in-container master
  • (#​781). vlan: Add support for in-container master

Improvements:

  • (#​880). bridge: read only required chain on cni del instead of the entire ruleset
  • (#​873). bridge, spoof check: remove drop rule index

Bug fixes:

  • (#​892). sbr: Ignore LinkNotFoundError during cmdDel null
  • (#​887). ptp: Fix ValidateExpectedRoute with non default routes and nil GW
  • (#​885). tuning: fix cmdCheck when using IFNAME
  • (#​831). Fix overwritten error var in getMTUByName
  • (#​821). Only check or del ipv6 when an IPv6 is configured

v1.2.0: CNI Plugins v1.2.0

Compare Source

Changelog:

New plugins & features

  • (#​743). dummy: Create a Dummy CNI plugin that creates a virtual interface
  • (#​725). V2 API support for win-overlay CNI
  • (#​693). tuning Add sysctl allowList

Bug fixes

  • (#​809). bridge: refresh host-veth mac after port add
  • (#​802). Add IPv6 support for AddDefaultRoute
  • (#​779). Fix path substitution to enable setting sysctls on vlan interfaces
  • (#​782). host-local: fix bug on getting NextIP of addresses with first byte
  • (#​709). dhcp: Fix client id in renew/release

Improvements & Cleanups:

  • (#​772). portmap support masquerade all
  • (#​733). bridge: support IPAM DNS settings
  • (#​702). bridge: call ipam.ExecDel after clean up device in netns #​702
  • (#​768). dhcp: Cleanup Socket and Pidfile on exit
  • (#​792). dhcp: Update Allocate method to reuse lease if present
  • (#​755). dhcp: Use the same options for acquiring, renewing lease
  • (#​730). tuning Check for duplicated sysctl keys
  • (#​739). build: support riscv64
  • (#​712). bug: return errors when iptables and ip6tables are unusable
  • (#​719). Make description for static plugin more exact

As always, many thanks to our contributors.

v1.1.1: CNI plugins v1.1.1

Compare Source

Plugins release v1.1.1

This is a patch release that fixes the following bugs in v1.1.0:

  • #​702 bridge: call ipam.ExecDel after clean up device in netns
  • #​709 ipam/dhcp: Fix client id in renew/release

v1.1.0 Changelog:

One minor-but-major change is that we no longer wait for IPv6 Duplicate
Address Detection to complete. This reduces execution time by 2 seconds.

New features:

  • firewall: support ingressPolicy=(open|same-bridge) for isolating bridges as in Docker (#​584)
  • dhcp ipam: support customizing dhcp options from CNI args (#​670)
  • Allow setting sysctls on a particular interface (#​669)
  • bridge: Add macspoofchk support (#​639).

Bug fixes:

  • portmap: fix bug that new udp connection deletes all existing conntrack entries (#​705)
  • portmap: fix checkPorts result when chain does not exist (#​707)
  • dhcp: fixed DHCP problem that broke when fast retry was added (#​681)
  • ipvlan: Send Gratuitous ARP after IPs are set (#​675)

Improvements

  • host-device: Bring interfaces up after moving into container (#​679)
  • Explicitly Disable Duplicate Address Detection For Container Side Veth (#​695)
  • Replace arping package with arp_notify (#​687)
  • host-device: add ipam support for dpdk device (#​642)

Other changes

  • Ignore NetNS path errors on delete (#​686)
  • Fix confusing error msg invalid cidr (#​638)

v1.1.0: CNI Plugins v1.1.0

Compare Source

This release brings a number of new features, along with the usual
smattering of bug fixes and cleanups.

One minor-but-major change is that we no longer wait for IPv6 Duplicate
Address Detection to complete. This reduces execution time by 2 seconds.

New features:

  • firewall: support ingressPolicy=(open|same-bridge) for isolating bridges as in Docker (#​584)
  • dhcp ipam: support customizing dhcp options from CNI args (#​670)
  • Allow setting sysctls on a particular interface (#​669)
  • bridge: Add macspoofchk support (#​639).

Bug fixes:

  • portmap: fix bug that new udp connection deletes all existing conntrack entries (#​705)
  • portmap: fix checkPorts result when chain does not exist (#​707)
  • dhcp: fixed DHCP problem that broke when fast retry was added (#​681)
  • ipvlan: Send Gratuitous ARP after IPs are set (#​675)

Improvements

  • host-device: Bring interfaces up after moving into container (#​679)
  • Explicitly Disable Duplicate Address Detection For Container Side Veth (#​695)
  • Replace arping package with arp_notify (#​687)
  • host-device: add ipam support for dpdk device (#​642)

Other changes

  • Ignore NetNS path errors on delete (#​686)
  • Fix confusing error msg invalid cidr (#​638)
golang/glog (github.com/golang/glog)

v1.2.2

Compare Source

What's Changed

Full Changelog: golang/glog@v1.2.1...v1.2.2

v1.2.1

Compare Source

What's Changed

Full Changelog: golang/glog@v1.2.0...v1.2.1

v1.2.0

Compare Source

What's Changed

Full Changelog: golang/glog@v1.1.2...v1.2.0

v1.1.2

Compare Source

Bugfix release.

What's Changed

Full Changelog: golang/glog@v1.1.1...v1.1.2

v1.1.1

Compare Source

Bugfixes since the larger v1.1.0, which have been addressed.

imdario/mergo (github.com/imdario/mergo)

v1.0.0: 1.0.0 released with new module URL: dario.cat/mergo

Compare Source

This PR is a release containing 0.3.15 features but changing the module URL to dario.cat/mergo.

What's Changed

Full Changelog: darccio/mergo@v0.3.16...v1.0.0

v0.3.16: Announcement: v1.0.0 will be released on June 18th

Compare Source

This release doesn't contain code changes.

After 10 years, with many corner cases covered, very few issues pending (at least, comparing them with the usage of the library as part of Docker, Kubernetes, Datadog's agent, etc.), and a very stable API, I think it's time to release a 1.0.0 version.

This version will be released under a vanity URL: dario.cat/mergo

PS: although I'll make sure that github.com/imdario/mergo will be available, I'm going to also change my GitHub handle, so expect for a few minutes to not be able to pull from github.com/imdario as I fork it from the new handle to the old one.

PS2: I'm creating a discussion for this release to make sure we can have a conversation around the topic, and anything else about Mergo that you care about.

v0.3.15: Issues #​131, #​170, #​220, and #​202 fixed

Compare Source

v0.3.14: fix: Respect overwriteWithEmptySrc when merging maps #​231

Compare Source

v0.3.13: Merging same-type slices supported, YAML upgraded to v3 (only used in tests), and more flexibility for transformers

Compare Source

What's Changed

New Contributors

Full Changelog: darccio/mergo@0.3.12...v0.3.13

j-keck/arping (github.com/j-keck/arping)

v1.0.3

Compare Source

k8snetworkplumbingwg/network-attachment-definition-client (github.com/k8snetworkplumbingwg/network-attachment-definition-client)

v1.7.1

Compare Source

Adds the CreateNetworkStatuses method to properly set the network-status annotation when a CNI result includes multiple interfaces in the return.

For details of the before/after, see this gist: https://gist.github.com/dougbtv/1eb8ac2d61d494b56d65a6b236a86e61

For the related suggested update to the NPWG net-attach-def specification (for clarity only), see this proposal @​ https://docs.google.com/document/d/1DUTV-o6E6zlRTKZkxeDhAeyGrmq03qPgPbU580Rm7-g/edit

v1.7.0

Compare Source

Adds MTU to network-status

v1.6.0

Compare Source

Adds IPAMClaimReference to network-selection-elements

v1.5.0

Compare Source

What's Changed

New Contributors

Full Changelog: k8snetworkplumbingwg/network-attachment-definition-client@v1.4.0...v1.5.0

v1.4.0

Compare Source

What's Changed

Full Changelog: k8snetworkplumbingwg/network-attachment-definition-client@v1.3.0...v1.4.0

v1.3.0

Compare Source

Major change:

  • Update Kubernetes API to 1.22.8
  • Remove obsolated network status annotation k8s.v1.cni.cncf.io/networks-status

v1.2.0: CNI version 1.0.1 support

Compare Source

This version supports CNI v1.0.1

v1.1.1

Compare Source

This makes a change to allow CNI v1.0 formatted IP information (which omits the version parameter) previously, we'd require that the version was there, however, this is changed in recently releases, and needs update in v1.1.

onsi/ginkgo (github.com/onsi/ginkgo)

v2.20.0

Compare Source

2.20.0

Features
Maintenance
  • Add update-deps to makefile [d303d14]
  • bump all dependencies [7a50221]

v2.19.1

Compare Source

2.19.1

Fixes
  • update supported platforms for race conditions [63c8c30]
  • [build] Allow custom name for binaries. [ff41e27]
Maintenance

v2.19.0

Compare Source

2.19.0

Features

Label Sets allow for more expressive and flexible label filtering.

v2.18.0

Compare Source

2.18.0

Features
  • Add --slience-skips and --force-newlines [f010b65]
  • fail when no tests were run and --fail-on-empty was set [d80eebe]
Fixes
  • Fix table entry context edge case [42013d6]
Maintenance

v2.17.3

Compare Source

2.17.3

Fixes

ginkgo watch now ignores hidden files [bde6e00]

v2.17.2

Compare Source

2.17.2

Fixes
  • fix: close files [32259c8]
  • fix github output log level for skipped specs [780e7a3]
Maintenance

v2.17.1

Compare Source

2.17.1

Fixes
  • If the user sets --seed=0, make sure all parallel nodes get the same seed [af0330d]

v2.17.0

Compare Source

2.17.0

Features
  • add --github-output for nicer output in github actions [e8a2056]
Maintenance

v2.16.0

Compare Source

2.16.0

Features
  • add SpecContext to reporting nodes
Fixes
Maintenance

v2.15.0

Compare Source

2.15.0

Features
  • JUnit reports now interpret Label(owner:X) and set owner to X. [8f3bd70]
  • include cancellation reason when cancelling spec context [96e915c]
Fixes
  • emit output of failed go tool cover invocation so users can try to debug things for themselves [c245d09]
  • fix outline when using nodot in ginkgo v2 [dca77c8]
  • Document areas where GinkgoT() behaves differently from testing.T [dbaf18f]
  • bugfix(docs): use Unsetenv instead of Clearenv (#​1337) [6f67a14]
Maintenance

v2.14.0

Compare Source

2.14.0

Features

You can now use GinkgoTB() when you need an instance of testing.TB to pass to a library.

Prior to this release table testing only supported generating individual Its for each test entry. DescribeTableSubtree extends table testing support to entire testing subtrees - under the hood DescrieTableSubtree generates a new container for each entry and invokes your function to fill our the container. See the docs to learn more.

Fixes
Maintenance

v2.13.2

Compare Source

2.13.2

Fixes

v2.13.1

Compare Source

2.13.1

Fixes

  • 1296 fix(precompiled test guite): exec bit check omitted on Windows (#​1301) [26eea01]

Maintenance

v2.13.0

Compare Source

2.13.0

Features

Add PreviewSpect() to enable programmatic preview access to the suite report (fixes #​1225)

v2.12.1

Compare Source

2.12.1

Fixes
Maintenance

v2.12.0

[Compare Source](https://togithub.com/onsi/ginkgo/compare

Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

  • If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

@redhat-renovate-bot redhat-renovate-bot added the release-note-none Denotes a PR that doesn't merit a release note. label Aug 15, 2024
@redhat-renovate-bot
Copy link
Collaborator Author

⚠️ Artifact update problem

Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.

♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

  • any of the package files in this branch needs updating, or
  • the branch becomes conflicted, or
  • you click the rebase/retry checkbox if found above, or
  • you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:

File name: go.sum
Command failed: go get -d -t ./...
go: github.com/Mellanox/[email protected]: parsing go.mod:
	module declares its path as: github.com/k8snetworkplumbingwg/sriovnet
	        but was required as: github.com/Mellanox/sriovnet

@kubevirt-bot kubevirt-bot added the dco-signoff: yes Indicates the PR's author has DCO signed all their commits. label Aug 15, 2024
@redhat-renovate-bot
Copy link
Collaborator Author

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.

@phoracek phoracek force-pushed the renovate/main-all-dependencies-on-main branch 2 times, most recently from a6f861a to 94c3790 Compare August 16, 2024 08:52
@phoracek
Copy link
Member

Many tests now fail due to:

       Expected
              
          <netlink.LinkOperState>: 2
      to equal
          <netlink.LinkOperState>: 6

meaning that we expect interfaces to be UP, while they are DOWN.

@phoracek phoracek force-pushed the renovate/main-all-dependencies-on-main branch 2 times, most recently from a9f1059 to 4b732c4 Compare August 16, 2024 09:56
@phoracek
Copy link
Member

@SchSeba @ykulazhenkov hi. I made a major bump of the dependencies and address some of the compatibility issues. Would be great if you could take a look - I recommend scanning through this commit by commit - there are very few changes to our code.

SchSeba
SchSeba reviewed Aug 18, 2024
View reviewed changes
Copy link
Collaborator

@SchSeba SchSeba left a comment

Choose a reason for hiding this comment

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

amazing work thanks @phoracek!

I left just two small comments and I think we should be good

go.mod Outdated
)

go 1.22.0

toolchain go1.22.5
toolchain go1.23.0
Copy link
Collaborator

Choose a reason for hiding this comment

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

we will have an issue on this one I think it should remain 1.22 for now no?

go.mod Outdated
sigs.k8s.io/json v0.0.0-20221116044647-bc3834ca7abd // indirect
sigs.k8s.io/structured-merge-diff/v4 v4.4.1 // indirect
sigs.k8s.io/yaml v1.3.0 // indirect
sigs.k8s.io/yaml v1.4.0 // indirect
)

// Pinned to kubernetes-1.30.3
Copy link
Collaborator

Choose a reason for hiding this comment

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

the comment is not right here

redhat-renovate-bot and others added 10 commits August 19, 2024 13:28
@redhat-renovate-bot @phoracek
chore(deps): update all dependencies on main
1a0b390 
Signed-off-by: redhat-renovate-bot <[email protected]>
@phoracek
chore(deps): Fix import paths after bump
85967a0 
Some of the imported modules has changed their paths since they were
introduced to the project.

This commit reflects their current path.

Without this change, go mod tidy fails.

Signed-off-by: Petr Horacek <[email protected]>
@phoracek
Fix ovsdb signatures after bump
61b9a4d 
OVSDB has changed some of its signatures. This commit reflects that
to allow compilation of ovsdb.go.

Signed-off-by: Petr Horacek <[email protected]>
@phoracek
Install Go detected from go.mod
a3dd261 
Replace duplicated Go version requirement from the script and use the
one specified in the go.mod instead.

Without this change, linter fails on OOM due to incompatible version:
running lint of 1.22 codebase with 1.21 Go.

Signed-off-by: Petr Horacek <[email protected]>
@phoracek
Bump version of the go linter
3acf166 
Signed-off-by: Petr Horacek <[email protected]>
@phoracek
chore(deps): Fix cni testlib singnatures after bump
f094d5f 
Some of the CNI testlib functions has changed their signatures. This
commits reflects that in our unit tests.

Signed-off-by: Petr Horacek <[email protected]>
@phoracek
chore(deps): Switch to ginkgo v2
bd8f03b 
The QE reporter now seems to rely on Ginkgo v2. Using Ginkgo v1 results
in a compilation failure.

This commit bumps the used Ginkgo version to v2.

Signed-off-by: Petr Horacek <[email protected]>
@phoracek
Fix linter deprecation warnings
af76242 
With the recent bump, some of the importent functions became
deprecated.

This commit replaces deprecated functions with their replacements.

Signed-off-by: Petr Horacek <[email protected]>
@phoracek
Remove minimal Go requirement from SR-IOV
1bb7b53 
This explicit requirement for Go >=1.10 is ancient and not needed
anymore since all the compilation is happening with 1.22+ already.

This requirement also started preventing compilation, thus removing it.

Signed-off-by: Petr Horacek <[email protected]>
@phoracek
Explicitly bring container veth up
d9bcdb5 
containernetwork/plugins changed the behavior of SetupVeth in 1.1,
to not set the container-side of the veth up.

This commit makes sure we set it up after the veth is created.

Signed-off-by: Petr Horacek <[email protected]>
@phoracek phoracek force-pushed the renovate/main-all-dependencies-on-main branch from 4b732c4 to d9bcdb5 Compare August 19, 2024 11:32
SchSeba
SchSeba approved these changes Aug 19, 2024
View reviewed changes
Copy link
Collaborator

@SchSeba SchSeba left a comment

Choose a reason for hiding this comment

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

/lgtm
/approve

@kubevirt-bot kubevirt-bot added the lgtm Indicates that a PR is ready to be merged. label Aug 19, 2024
@kubevirt-bot
Copy link
Collaborator

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: redhat-renovate-bot, SchSeba

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubevirt-bot kubevirt-bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Aug 19, 2024
@kubevirt-bot kubevirt-bot merged commit 1c2b841 into main Aug 19, 2024
6 of 7 checks passed
@phoracek
Copy link
Member

Thanks a lot @SchSeba!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@SchSeba SchSeba SchSeba approved these changes

Assignees

@SchSeba SchSeba

Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. dco-signoff: yes Indicates the PR's author has DCO signed all their commits. lgtm Indicates that a PR is ready to be merged. release-note-none Denotes a PR that doesn't merit a release note.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@redhat-renovate-bot @phoracek @kubevirt-bot @SchSeba