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.

Sign up for GitHub

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

Jump to bottom

Powerscale v2.8.0 release #239

Closed
wants to merge 1 commit into from
Closed

Powerscale v2.8.0 release #239

wants to merge 1 commit into from

Conversation

HarishH-DELL
Copy link
Contributor

Is this a new chart?

Yes

What this PR does / why we need it:

Powerscale 2.8.0 version

Which issue(s) is this PR associated with:

Special notes for your reviewer:

Checklist:

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • Chart Version bumped
  • Variables are documented in the chart README.md
  • Title of the PR starts with the chart name (e.g. [charts_dir/mychartname]) if applicable

niranjan-n1
niranjan-n1 reviewed Jul 24, 2023
View reviewed changes
Copy link
Contributor

@niranjan-n1 niranjan-n1 left a comment

Choose a reason for hiding this comment

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

Should this PR be raised against 1.8 branch?

@HarishH-DELL
Copy link
Contributor Author

yes

chaganti-rajitha
chaganti-rajitha requested changes Jul 24, 2023
View reviewed changes
Copy link
Contributor

@chaganti-rajitha chaganti-rajitha left a comment

Choose a reason for hiding this comment

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

We need to merge the changes to release-1.8.0 branch, please change it and also we need to update the changes in container-storage-modules chart under charts folder. Please update that chart as well.

@HarishH-DELL HarishH-DELL deleted the powerscale_v2.8.0_release_changes branch July 24, 2023 12:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@niranjan-n1 niranjan-n1 niranjan-n1 left review comments

@chaganti-rajitha chaganti-rajitha chaganti-rajitha requested changes

@gallacher gallacher Awaiting requested review from gallacher gallacher is a code owner

@tdawe tdawe Awaiting requested review from tdawe tdawe is a code owner

@alikdell alikdell Awaiting requested review from alikdell alikdell is a code owner

@atye atye Awaiting requested review from atye atye is a code owner

@hoppea2 hoppea2 Awaiting requested review from hoppea2 hoppea2 is a code owner

@coulof coulof Awaiting requested review from coulof coulof is a code owner

@meggm meggm Awaiting requested review from meggm meggm is a code owner

@shaynafinocchiaro shaynafinocchiaro Awaiting requested review from shaynafinocchiaro shaynafinocchiaro is a code owner

@sharmilarama sharmilarama Awaiting requested review from sharmilarama sharmilarama is a code owner

@tssushma tssushma Awaiting requested review from tssushma tssushma is a code owner

@taohe1012 taohe1012 Awaiting requested review from taohe1012 taohe1012 is a code owner

@P-Cao P-Cao Awaiting requested review from P-Cao P-Cao is a code owner

@shanmydell shanmydell Awaiting requested review from shanmydell shanmydell is a code owner

@baoy1 baoy1 Awaiting requested review from baoy1 baoy1 is a code owner

@YianZong YianZong Awaiting requested review from YianZong YianZong is a code owner

@forrestxia forrestxia Awaiting requested review from forrestxia forrestxia is a code owner

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

Successfully merging this pull request may close these issues.
3 participants
@HarishH-DELL @niranjan-n1 @chaganti-rajitha