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.

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

sync downstream main with upstream main #81

Merged
merged 25 commits into from
Jul 6, 2023

Conversation

Madhu-1
Copy link
Member

@Madhu-1 Madhu-1 commented Jul 5, 2023

Regular sync of downstream main with upstream main.

nixpanic and others added 25 commits June 22, 2023 05:42
With recent changes to detect Podman/Docker, the
`docker-generate-protobuf` make target was broken. The parameters to
the container build command were passed like `$^`, and not only the
Containerfile filename.

Fixes: #372
Signed-off-by: Niels de Vos <ndevos@ibm.com>
This commit adds lastsyncbytes and lastsyncduration
to volume replication status and generates updated crds.

Signed-off-by: yati1998 <ypadia@redhat.com>
This commit updates the getvolumereplicationinfo
rpc to include lastsyncbytes and lastsyncduration.

Signed-off-by: yati1998 <ypadia@redhat.com>
This commit updates the vendor directory to
include lastest spec.

Signed-off-by: yati1998 <ypadia@redhat.com>
This commit updates the reconcile logic to
get more info and update the volrep status.

Signed-off-by: yati1998 <ypadia@redhat.com>
This also moves to use the quay.io mirror of the Golang container-image
from docker.io. Docker Hub has pull restrictions, that can prevent
pulling the image if it is done too often. Quay.io does not enforce
restrictions like that.

Signed-off-by: Niels de Vos <ndevos@ibm.com>
tools.go is moved to another module tools to
separate all dependencies used for build from
main go.mod.

Signed-off-by: Rakshith R <rar@redhat.com>
The `/tools` directory was added with commit 342d457. It is
needed to update the dependencies of the tools separately.

Signed-off-by: Niels de Vos <ndevos@ibm.com>
Bumps [google.golang.org/grpc](https://github.com/grpc/grpc-go) from 1.55.0 to 1.56.1.
- [Release notes](https://github.com/grpc/grpc-go/releases)
- [Commits](grpc/grpc-go@v1.55.0...v1.56.1)

---
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>
this commit update the kubernetes pkgs to latest 0.27.2
and controller runtime version to 1.15.0.

Signed-off-by: subhamkrai <srai@redhat.com>

Signed-off-by: subhamkrai <srai@redhat.com>
Bumps [k8s.io/api](https://github.com/kubernetes/api) from 0.27.2 to 0.27.3.
- [Commits](kubernetes/api@v0.27.2...v0.27.3)

---
updated-dependencies:
- dependency-name: k8s.io/api
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [github.com/kubernetes-csi/csi-lib-utils](https://github.com/kubernetes-csi/csi-lib-utils) from 0.13.0 to 0.14.0.
- [Release notes](https://github.com/kubernetes-csi/csi-lib-utils/releases)
- [Commits](kubernetes-csi/csi-lib-utils@v0.13.0...v0.14.0)

---
updated-dependencies:
- dependency-name: github.com/kubernetes-csi/csi-lib-utils
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps [k8s.io/client-go](https://github.com/kubernetes/client-go) from 0.26.3 to 0.27.3.
- [Changelog](https://github.com/kubernetes/client-go/blob/master/CHANGELOG.md)
- [Commits](kubernetes/client-go@v0.26.3...v0.27.3)

---
updated-dependencies:
- dependency-name: k8s.io/client-go
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Bumps google.golang.org/protobuf from 1.30.0 to 1.31.0.

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

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: Rakshith R <rar@redhat.com>
Bumps google.golang.org/protobuf from 1.30.0 to 1.31.0.

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: Rakshith R <rar@redhat.com>
Signed-off-by: saranyareddy24 <saranyareddipalle@yahoo.in>
The `Makefile` now contains logic to provide a version (git tag) and the
current git commit hash in a `-version` argument to executables. However
the container-images did not use `make`, and therefor the executables
would not have the version information set.

While updating the building of the executables in the container, the
names of the executables has been prefixed with `csi-addons-`, so that a
listing of running processes on a Kubernetes node does not show
`manager` for the CSI-Addons Operator, but `csi-addons-manager`.

Updates: #385
Signed-off-by: Niels de Vos <ndevos@ibm.com>
Just like for the manager and sidecar executables, add `-version` as an
option to the `csi-addons` tool.

Signed-off-by: Niels de Vos <ndevos@ibm.com>
Disable building with CGo support, so that only native Golang functions
are used. This makes it possible to build the executables in one
container, and run them in a distroless container.

Fore rebuilding all components with `-a` in the `Makefile`, this
prevents including artifacts that may have been built outside the
contains with CGo enabled.

Signed-off-by: Niels de Vos <ndevos@ibm.com>
This commit switches back to docker for
base image golang:1.20 since github actions
does not seem to be able to pull from quay
intermittently.

Signed-off-by: Rakshith R <rar@redhat.com>
Signed-off-by: karthik-us <ksubrahm@redhat.com>
Logic used for determining reclaimspace annotation
based on ns annotation and driver support had a bug
which caused all the PVCs regardless of driversupport
being annotated.
This commit makes sure only pvc with driver which
support reclaimspace is annotated/ requeued.

Signed-off-by: Rakshith R <rar@redhat.com>
@Madhu-1
Copy link
Member Author

Madhu-1 commented Jul 5, 2023

/assign @nixpanic

Copy link
Member

@Rakshith-R Rakshith-R left a comment

Choose a reason for hiding this comment

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

/lgtm

@openshift-ci openshift-ci bot added the lgtm label Jul 6, 2023
@openshift-ci
Copy link

openshift-ci bot commented Jul 6, 2023

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Madhu-1, Rakshith-R

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

@openshift-merge-robot openshift-merge-robot merged commit a08b342 into red-hat-storage:main Jul 6, 2023
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants