Skip to content

Commit

Permalink
Merge pull request #52 from smart-edge-open/release-sriov-fec-operato…
Browse files Browse the repository at this point in the history
…r-23.41

SRIOV-FEC Operator Release 23.41
  • Loading branch information
MouliBurla authored Oct 16, 2023
2 parents e5e578e + a5637fb commit be6dc66
Show file tree
Hide file tree
Showing 4 changed files with 30 additions and 8 deletions.
2 changes: 1 addition & 1 deletion Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ export CLI_EXEC?=oc
# Container format for podman. Required to build containers with "ManifestType": "application/vnd.oci.image.manifest.v2+json",
export BUILDAH_FORMAT=docker
# Current Operator version
VERSION ?= 2.7.1
VERSION ?= 2.7.2
# Supported channels
CHANNELS ?= stable
# Default channel
Expand Down
2 changes: 2 additions & 0 deletions pkg/daemon/node_management.go
Original file line number Diff line number Diff line change
Expand Up @@ -159,11 +159,13 @@ func (n *NodeConfigurator) changeAmountOfVFs(driver string, pfPCIAddress string,
return fmt.Errorf("unknown driver %v", driver)
}

n.Log.WithField("op", "VF Enable").Info("Start")
err := writeFileWithTimeout(unbindPath, strconv.Itoa(vfsAmount))
if err != nil {
n.Log.WithError(err).WithField("pf", pfPCIAddress).WithField("vfsAmount", vfsAmount).Error("failed to set new amount of VFs for PF")
return fmt.Errorf("failed to set new amount of VFs (%d) for PF (%s): %w", vfsAmount, pfPCIAddress, err)
}
n.Log.WithField("op", "VF Enable").Info("Complete")
return nil
}

Expand Down
7 changes: 4 additions & 3 deletions pkg/daemon/utils.go
Original file line number Diff line number Diff line change
Expand Up @@ -5,12 +5,13 @@ package daemon

import (
"fmt"
"github.com/sirupsen/logrus"
"os"
"time"

"github.com/sirupsen/logrus"
"sigs.k8s.io/controller-runtime/pkg/event"
"sigs.k8s.io/controller-runtime/pkg/predicate"
"sigs.k8s.io/controller-runtime/pkg/reconcile"
"time"
)

type resourceNamePredicate struct {
Expand Down Expand Up @@ -66,7 +67,7 @@ func writeFileWithTimeout(filename, data string) error {
select {
case <-done:
return err
case <-time.After(5 * time.Second):
case <-time.After(60 * time.Second):
return fmt.Errorf("failed to write to sysfs file. Usually it means that device is in use by other process")
}

Expand Down
27 changes: 23 additions & 4 deletions spec/sriov-fec-operator-release-notes.md
Original file line number Diff line number Diff line change
Expand Up @@ -44,11 +44,16 @@ This document provides high-level system features, issues, and limitations infor
| 2.4.0 | September 2022 | 4.9, 4.10, 4.11 | 4.9.41, 4.10.21, 4.11.2 |
| 2.5.0 | September 2022 | 4.9, 4.10, 4.11 | 4.9.48, 4.10.34, 4.11.5 |
| 2.6.0 | December 2022 | 4.10, 4.11, 4.12 | 4.10.43, 4.11.18, 4.12-rc2 |
| 2.6.1 | January 2023 | 4.10, 4.11, 4.12 | 4.10.43, 4.11.18, 4.12.0-rc.4 | |
| 2.6.1 | January 2023 | 4.10, 4.11, 4.12 | 4.10.43, 4.11.18, 4.12.0-rc.4 |
| 2.7.0 | May 2023 | 4.10, 4.11, 4.12, 4.13 | 4.11.43, 4.12.18, 4.13.0.rc18 |
| 2.7.1 | July 2023 | 4.10, 4.11, 4.12, 4.13 | 4.11.43, 4.12.18, 4.13.7 |
| 2.7.2 | October 2023 | 4.10, 4.11, 4.12, 4.13 | 4.10.67, 4.11.50, 4.12.37, 4.13.15 |

# Features for Release

***v2.7.2***
- Bug fixes

***v2.7.1***
- pf-bb-config updated (22.11 -> 23.03)
- Bug fixes
Expand Down Expand Up @@ -127,6 +132,10 @@ This document provides high-level system features, issues, and limitations infor
- Deploys an instance of K8s SRIOV device plugin which manages the FEC VFs as an OpenShift cluster resource and configures this device plugin to detect the resources

# Changes to Existing Features

***v2.7.2***
- None

***v2.7.1***
- FEC resource names can be controlled through manager environment variables
- igb-uio driver is added to list of support drivers for VF interface
Expand Down Expand Up @@ -200,6 +209,10 @@ This document provides high-level system features, issues, and limitations infor
- There are no unsupported or discontinued features relevant to this release.

# Fixed Issues

***2.7.2***
- Fix for failure in enabling VFs when kernel is overloaded

***2.7.1***
- Fix for supporting multiple FEC devices on same node
- Fix for checking secure boot enabled mode
Expand Down Expand Up @@ -239,6 +252,13 @@ This document provides high-level system features, issues, and limitations infor
- Documentation

# Supported Operating Systems

***v2.7.2***
- OpenShift: 4.13.0
- OS: Red Hat Enterprise Linux CoreOS 413.92.202305191644-0
- Kubernetes: v1.26.3+b404935
- RT Kernel: 5.14.0-284.13.1.el9_2.x86_64

***v2.7.1***
- OpenShift: 4.13.0
- OS: Red Hat Enterprise Linux CoreOS 413.92.202305191644-0
Expand Down Expand Up @@ -381,6 +401,5 @@ This document provides high-level system features, issues, and limitations infor

# Package Versions
Package:
- Golang: 1.18
- pf-bb-config-app: v22.11
- DPDK: 22.11
- Golang: 1.20
- pf-bb-config-app: v23.03

0 comments on commit be6dc66

Please sign in to comment.