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

chore(operator): add openebs 2.11.0 operators #231

Merged
merged 7 commits into from
Jul 17, 2021
Merged

chore(operator): add openebs 2.11.0 operators #231

merged 7 commits into from
Jul 17, 2021

Conversation

prateekpandey14
Copy link
Member

@prateekpandey14 prateekpandey14 commented Jul 8, 2021 •
edited
Loading

Signed-off-by: prateekpandey14 prateek.pandey@mayadata.io

Special notes for your reviewer:

Adds

  • openebs, cstor, jiva operator 2.11.0 (RC1, RC2, GA)
  • lvm operator 0.7.0 (RC1, RC2, GA)
  • ndm operator 1.6.0 (RC1, RC2, GA)
  • zfs operator 1.9.0 (RC1, RC2,GA)
  • Hostpath operator (GA)
  • microk8s cstor operator (GA)

@prateekpandey14
chore(operator): add openebs 2.11.0 operators
2c1f1cc 
Signed-off-by: prateekpandey14 <prateek.pandey@mayadata.io>
@prateekpandey14 prateekpandey14 added this to RC1 - Due: Jul 3 2021 in 2.11 Release Tracker - Due July 15th. Jul 8, 2021
@prateekpandey14 prateekpandey14 moved this from RC1 - Due: Jul 3 2021 to RC2 - Due: Jul 8 2021 in 2.11 Release Tracker - Due July 15th. Jul 8, 2021
@akhilerm
update cstor and ndm yamls
097f87e 
Signed-off-by: Akhil Mohan <akhil.mohan@mayadata.io>
@akhilerm
update openebs operator
e336dd8 
Signed-off-by: Akhil Mohan <akhil.mohan@mayadata.io>
akhilerm
akhilerm reviewed Jul 8, 2021
View reviewed changes
Copy link
Member

@akhilerm akhilerm left a comment

Choose a reason for hiding this comment

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

NDM charts are good. made the necessary changes in NDM, cstor and openebs operator

@akhilerm
update mount change feature gate
a8cc1b9 
Signed-off-by: Akhil Mohan <akhil.mohan@mayadata.io>
@kmova kmova added the pr/hold-merge The PR should not be merged now label Jul 9, 2021
nsathyaseelan
nsathyaseelan suggested changes Jul 12, 2021
View reviewed changes
2.11.0/cstor-operator-RC2.yaml Outdated Show resolved Hide resolved
2.11.0/openebs-operator-RC2.yaml Outdated Show resolved Hide resolved
@prateekpandey14
add RC2 operators yamls
70806a9 
Signed-off-by: prateekpandey14 <prateek.pandey@mayadata.io>
@prateekpandey14 prateekpandey14 moved this from RC2 - Due: Jul 8 2021 to Release Items in 2.11 Release Tracker - Due July 15th. Jul 13, 2021
@akhilerm
Copy link
Member

@prateekpandey14 The yamls at the root directory also needs to be updated.

@kmova
Copy link
Member

kmova commented Jul 15, 2021

@prateekpandey14 -- can you update the NOTICE message in the openebs-operator to fix this isssue: openebs/openebs#3414

Few things we can do is:

  • Mention that the file will be deprecated in 3.0.0
  • Provide links to the YAML files that supersede this YAML - like the cstor-operators and other operator yaml files.

@prateekpandey14
add 2.11.0 GA release operator yamls
09210a5 
Signed-off-by: prateekpandey14 <prateek.pandey@mayadata.io>
@prateekpandey14
Copy link
Member Author

  • Mention that the file will be deprecated in 3.0.0
  • Provide links to the YAML files that supersede this YAML - like the cstor-operators and other operator yaml files.

Done

nsathyaseelan
nsathyaseelan approved these changes Jul 16, 2021
View reviewed changes
Copy link
Contributor

@nsathyaseelan nsathyaseelan left a comment

Choose a reason for hiding this comment

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

lgtm

shubham14bajpai
shubham14bajpai approved these changes Jul 16, 2021
View reviewed changes
Copy link
Member

@shubham14bajpai shubham14bajpai left a comment

Choose a reason for hiding this comment

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

LGTM

akhilerm
akhilerm reviewed Jul 16, 2021
View reviewed changes
Copy link
Member

@akhilerm akhilerm left a comment

Choose a reason for hiding this comment

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

given a comment

@prateekpandey14
update microk8s cstor-operator with NDM changes
54e9110 
Signed-off-by: prateekpandey14 <prateek.pandey@mayadata.io>
akhilerm
akhilerm approved these changes Jul 16, 2021
View reviewed changes
Copy link
Member

@akhilerm akhilerm left a comment

Choose a reason for hiding this comment

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

LGTM

@prateekpandey14 prateekpandey14 removed the pr/hold-merge The PR should not be merged now label Jul 16, 2021
w3aman
w3aman approved these changes Jul 16, 2021
View reviewed changes
Copy link
Contributor

@w3aman w3aman left a comment

Choose a reason for hiding this comment

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

for zfs and lvm operator yamls -- LGTM

@pawanpraka1
Copy link
Contributor

@prateekpandey14 you need to add allowed_topologies as an env for LVM operator yaml. https://github.com/openebs/lvm-localpv/blob/master/deploy/lvm-operator.yaml#L1374.

@kmova kmova mentioned this pull request Jul 17, 2021
Closed
@kmova kmova merged commit f338457 into openebs:gh-pages Jul 17, 2021
2.11 Release Tracker - Due July 15th. automation moved this from Release Items to Done Jul 17, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@kmova kmova kmova left review comments

@akhilerm akhilerm akhilerm approved these changes

@shubham14bajpai shubham14bajpai shubham14bajpai approved these changes

@nsathyaseelan nsathyaseelan nsathyaseelan approved these changes

@w3aman w3aman w3aman approved these changes

@pawanpraka1 pawanpraka1 Awaiting requested review from pawanpraka1

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
7 participants
@prateekpandey14 @akhilerm @kmova @pawanpraka1 @shubham14bajpai @nsathyaseelan @w3aman