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

Rebase 03/06/24 #52

Merged
merged 10 commits into from
Mar 6, 2024
Merged

Rebase 03/06/24 #52

merged 10 commits into from
Mar 6, 2024

Conversation

jakobmoellerdev
Copy link

No description provided.

daichimukai and others added 10 commits February 26, 2024 14:53
The Watch RPC of lvmd's VGService is expected to notify VG information
periodically to its watchers. However, when lvmd is embedded into
topolvm-node, Watch RPC sends VG info only once on startup time and does
nothing after that unless LVService invokes LVM commands for RPCs. This
commit addresses the issue.

Fixes: c5f17a4 ("feat: implement embed-lvmd")
Signed-off-by: Daichi Mukai <daichi-mukai@cybozu.co.jp>
lvmd: fix Watch RPC to notify VG info periodically when embedded
pkg           -> cmd
e2e           -> test/e2e
util/testing  -> test/util
controllers   -> internal/controller
other sources -> internal/*

ref: https://book.kubebuilder.io/migration/migration_guide_gov3_to_gov4

Signed-off-by: Yuji Ito <llamerada.jp@gmail.com>
The same test is executed on the test target in the Makefile.
Therefore, this one is removed.

Signed-off-by: Yuji Ito <llamerada.jp@gmail.com>
ref: #840 (comment)
ref: #840 (comment)

Signed-off-by: Yuji Ito <llamerada.jp@gmail.com>
Bumps the github-actions-update group with 1 update: [azure/setup-helm](https://github.com/azure/setup-helm).


Updates `azure/setup-helm` from 3 to 4
- [Release notes](https://github.com/azure/setup-helm/releases)
- [Commits](Azure/setup-helm@v3...v4)

---
updated-dependencies:
- dependency-name: azure/setup-helm
  dependency-type: direct:production
  update-type: version-update:semver-major
  dependency-group: github-actions-update
...

Signed-off-by: dependabot[bot] <support@github.com>
Change directory tree to the new kubebilder template
…-actions-update-ec2bd254e3

build(deps): bump the github-actions-update group with 1 update
Signed-off-by: Jakob Möller <jmoller@redhat.com>
Introduce 4096 byte Sector Size Minimum for Rounding LV Sizes on Creation and Resizing
@suleymanakbas91
Copy link

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Mar 6, 2024
@jakobmoellerdev jakobmoellerdev merged commit 90f16b4 into openshift:main Mar 6, 2024
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.

None yet

8 participants