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

Syncing latest changes from master for rook #658

Merged
merged 59 commits into from
May 31, 2024
Merged

Conversation

df-build-team
Copy link

PR containing the latest commits from master branch

subhamkrai and others added 30 commits May 7, 2024 12:46
updating ubuntu image to 22.04 from 20.04 which is used in ci test

Signed-off-by: subhamkrai <[email protected]>
docs: improve wording in BlockPoolRados Namespace CRD doc.
Update the version of Go used for govulncheck to get the latest CVE fix
present in Go v1.22.3.

Signed-off-by: Blaine Gardner <[email protected]>
The CI step that runs markdown lint accidentally got removed as part of
PR updates. Add the CI step back in.

Signed-off-by: Blaine Gardner <[email protected]>
LVM-based OSDs on PVCs are legacy and are not well tested.
Therefore, we add status to the cephcluster CR that at least
gives a clue that the OSDs should be replaced.

Signed-off-by: Travis Nielsen <[email protected]>
osd: Add cephcluster status for deprecated osds to replace
Bumps [golangci/golangci-lint-action](https://github.com/golangci/golangci-lint-action) from 5 to 6.
- [Release notes](https://github.com/golangci/golangci-lint-action/releases)
- [Commits](golangci/golangci-lint-action@v5...v6)

---
updated-dependencies:
- dependency-name: golangci/golangci-lint-action
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
Bumps the github-dependencies group with 1 update: [github.com/aws/aws-sdk-go](https://github.com/aws/aws-sdk-go).


Updates `github.com/aws/aws-sdk-go` from 1.52.2 to 1.53.0
- [Release notes](https://github.com/aws/aws-sdk-go/releases)
- [Commits](aws/aws-sdk-go@v1.52.2...v1.53.0)

---
updated-dependencies:
- dependency-name: github.com/aws/aws-sdk-go
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: github-dependencies
...

Signed-off-by: dependabot[bot] <[email protected]>
…dependencies-58b26b4692

build(deps): bump github.com/aws/aws-sdk-go from 1.52.2 to 1.53.0 in the github-dependencies group
…angci/golangci-lint-action-6

build(deps): bump golangci/golangci-lint-action from 5 to 6
The version v2 was deprecated and was causing warnings.

Signed-off-by: NymanRobin <[email protected]>
Afte pylint was updated to version 3.2.0
a set of new errors appeared. This makes
ci passing by either fixing the problem
or ignoring it in the check

Signed-off-by: NymanRobin <[email protected]>
ci: bump github actions dependency setup-go
Since pacific is no longer supported, we can remove pacific specific
code.

Signed-off-by: Satoru Takeuchi <[email protected]>
since we moved to ubuntu 22.04, ci push image build is failing
due wrong parameter `--platforms linux/amd64 linux/arm64,` it should
be instead `--platforms linux/amd64, linux/arm64`.

Signed-off-by: subhamkrai <[email protected]>
…image

ci: push image ci failing in ubuntu 22.04
after cross-storageclass clone/restore, now we can
clone across different storageclass.
this commit updates the documentation indicating
the same.

Signed-off-by: yati1998 <[email protected]>
The last pr missed removing the extra comma after the platform
so, instead of ` linux/amd64,linux/arm64,` it should be
` linux/amd64,linux/arm64`.

Signed-off-by: subhamkrai <[email protected]>
doc: Update documentation for volume clone
add generated csv changes

Signed-off-by: Ceph Jenkins <[email protected]>
Syncing latest changes from upstream master for rook
update rook csv version to 4.17

Signed-off-by: Nikhil-Ladha <[email protected]>
travisn and others added 13 commits May 18, 2024 10:02
core: remove owner refs in resource cleanup jobs
docs: document how to have StorageClass consume Rados Namespace
currently node predicate watches for resourceVersion and heartbeat
changes, which are not critical things. So with the PR we will add
a early-exit condition for the predicate as an optimization and to
ignore considering for a reconcile of cephcluster if updates are
not crucial

closes: rook#14070

Signed-off-by: parth-gr <[email protected]>
core: do not watch resourceversion and heartbeat node updates
Syncing latest changes from upstream master for rook
When IPv6-only network mode is enabled and when mons are configured to
listen only on msgrV2 port, Rook has to add the port to the mon
--public-bind-addr in order to force mons to stop listening on the
msgrV1 port. This can cause problems for IPv6 clusters where adding
':<port>' to the end of an IP addr looks like a valid IPv6 address.

In IPv6 mode, Rook needs to set this to '[<ip>]:<port>' with the
brackets to be a non-ambiguous IPv6 address.

In IPv4 mode, Rook needs to keep its existing behavior. IPv4 mode should
also continue to be the defualt assumption if the network config isn't
specified.

If 'dualStack: true' is specified, Rook is informed that the environment
is dual stack, but Rook doesn't know which IP family to expect from the
pod IP. In this scenario, it is safest for Rook not to apply the port,
since assuming either IPv4 or IPv6 format could be incorrect. In this
scenario, mons may still listen on msgrV1 ports, but this is preferable
to an environment where mons perpetually crash.

Signed-off-by: Blaine Gardner <[email protected]>
docs: fix broken link to network config
Syncing latest changes from upstream master for rook
mon: fix bind addr when IPv6 and msgr2 are enabled
fixing 'secert' to `secret` and adding workds like
`NotIn, addin` to skip work list.

Signed-off-by: subhamkrai <[email protected]>
This PR adds a `settings` option in the `Module` struct
which can help users to futher customize the `balancer`
module with a differet mode.

Signed-off-by: sp98 <[email protected]>
@df-build-team df-build-team requested a review from a team May 27, 2024 06:35
subhamkrai and others added 11 commits May 27, 2024 12:11
ci: fix codespell ci by skipping some words
This reduces the safe-to-remove retry interval from 1m to 15s.

Closes rook#14247

Signed-off-by: Brian Dowling <[email protected]>
Syncing latest changes from upstream master for rook
Add a note to remove LimitNOFILE setting in containerd service configuration.

This setting causes multiple issues such as slow `ceph status` commands (and failing healthchecks),
various timeouts, and mons failing/timing out.

Upstream containerd already removed the setting. NixOS hasn't incorporated the change.

This note can be removed once NixOS/nixpkgs#313507 makes it to a stable release.

Signed-off-by: Onur Cakmak <[email protected]>
Currently we allow upgrade of other OSDs while migration
of OSD, due to change in backing store, is pending. This
will not work if the updated OSD ceph version does not support
the currently applied backing store. So rook will skip any
OSD upgrade if the OSD migration is pending.

Signed-off-by: sp98 <[email protected]>
…eout

osd: reduce safe-to-destroy retry timeout to 15s
docs: Add a note for NixOS and containerd
mgr: setting balancer mode from cluster spec.
add generated csv changes

Signed-off-by: Ceph Jenkins <[email protected]>
@travisn
Copy link

travisn commented May 31, 2024

/approve
/lgtm

@travisn travisn merged commit f13acdc into release-4.17 May 31, 2024
98 of 100 checks passed
@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label May 31, 2024
Copy link

openshift-ci bot commented May 31, 2024

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: df-build-team, travisn

The full list of commands accepted by this bot can be found 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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.