Skip to content
This repository has been archived by the owner on Sep 9, 2024. It is now read-only.

KPM v2.5.0 release #81

Merged
merged 28 commits into from
Jun 4, 2024
Merged

KPM v2.5.0 release #81

merged 28 commits into from
Jun 4, 2024

Conversation

madalazar
Copy link
Contributor

@madalazar madalazar commented May 30, 2024

Summary

This PR contains the following:

  • KPM to use PowerLibrary v1.5.0
  • allow multi node shared pools
  • enable support for E/P cores (*on platforms that support these types of cores)
  • show CRD installation status
  • change the update strategy to a merge-patch command
  • bump K8s, go and 3rd party dependency version to fix dependabot vulnerabilties
  • CI work

madalazar and others added 28 commits May 30, 2024 17:48
This commit will:
* Fix typos in comments
* Updating error messages: using min/max abbreviations
* fix: Allow mutli node shared pools

* profile controller attaches profile information to an exclusive pool
* workload controller retrieves this information
* attaches profile to shared pool

* fix: addressing comments

* changed serviceUnavailable to generic error
* adjustments to comments and logs
* excluding issue as latest image does not resolve issue
This commit will:
* Bump K8s dependencies to 1.28.8
* Bump controller-gen version for K8s 1.28, go v1.22
* chore: bump k8s, go and controller runtime

* k8s bumped to 1.29
* controller runtime bumped to latest

* chore: upping leader ellection timeout and GID
* feat: Control PLane socket integration

* added second socket for control plane
* if cores can't be found this socket is tried

* test: fixing and adding unit tests

* fix: Addressing comments

* tweaked comments and line spacing

* feat: Implementeing library changes

- Integrated function to get frequency ranges
- Adjusted unit tests due to library changes

* fix: addressing further comments

* added nil pointer check on client
* integrated library changes
* added optimizations

* feat: re-alligning with library

* : feat: added retry mechanism for control plane sock

* additional edits from comments also made
* set status error field

* consistent errors

* correct status field ignore predicate

* readme, update only if needed, correct predicate

* typo correction

* updating utests to test multiple object types, fixed multinode errors  with cstates
In execution time, if the controller-manager tries to update the list of
customDevices on the powerNode, the update strategy fails with the message:
"Operation cannot be fulfilled on powernodes.power.intel.com: the object
has been modified; please apply your changes to the latest version and
try again."

This commit changes the update strategy to a merge-patch command, avoiding
an issue during the power node update.

Signed-off-by: Eduardo Juliano Alberti <eduardo.alberti@windriver.com>
Signed-off-by: Madalina Lazar <madalina.lazar@intel.com>
Co-authored-by: Eduardo Juliano Alberti <eduardo.alberti@windriver.com>
Bumps [golang.org/x/net](https://github.com/golang/net) from 0.19.0 to 0.23.0.
- [Commits](golang/net@v0.19.0...v0.23.0)

---
updated-dependencies:
- dependency-name: golang.org/x/net
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [go.uber.org/zap](https://github.com/uber-go/zap) from 1.26.0 to 1.27.0.
- [Release notes](https://github.com/uber-go/zap/releases)
- [Changelog](https://github.com/uber-go/zap/blob/master/CHANGELOG.md)
- [Commits](uber-go/zap@v1.26.0...v1.27.0)

---
updated-dependencies:
- dependency-name: go.uber.org/zap
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [github.com/go-logr/logr](https://github.com/go-logr/logr) from 1.4.1 to 1.4.2.
- [Release notes](https://github.com/go-logr/logr/releases)
- [Changelog](https://github.com/go-logr/logr/blob/master/CHANGELOG.md)
- [Commits](go-logr/logr@v1.4.1...v1.4.2)

---
updated-dependencies:
- dependency-name: github.com/go-logr/logr
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.61.0 to 1.64.0.
- [Release notes](https://github.com/grpc/grpc-go/releases)
- [Commits](grpc/grpc-go@v1.61.0...v1.64.0)

---
updated-dependencies:
- dependency-name: google.golang.org/grpc
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@madalazar madalazar marked this pull request as ready for review May 31, 2024 15:52
@madalazar madalazar requested a review from catblade May 31, 2024 15:56
Copy link
Contributor

@adorney99 adorney99 left a comment

Choose a reason for hiding this comment

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

LGTM

@adorney99 adorney99 merged commit 5d3ddab into master Jun 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants