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

feat(pci-block-storage): add exten beta #13489

Merged
merged 11 commits into from
Oct 31, 2024
Merged

feat(pci-block-storage): add exten beta #13489

merged 11 commits into from
Oct 31, 2024

Conversation

SimonChaumet
Copy link
Contributor

@SimonChaumet SimonChaumet commented Oct 8, 2024 •
edited
Loading

Question Answer
Branch? develop
Bug fix? no
New feature? yes
Breaking change? no (if #13505 is merged)
Tickets Epic TAPC-1694
License BSD 3-Clause
  • Try to keep pull requests small so they can be easily reviewed.
  • Commits are signed-off
  • Only FR translations have been updated
  • Branch is up-to-date with target branch
  • Lint has passed locally
  • Standalone app was ran and tested locally
  • Ticket reference is mentioned in linked commits (internal only)
  • Breaking change is mentioned in relevant commits

Description

  • extend volume capacity to 12To
  • add banner for exten beta
  • implement dependent regions on block tiles

Related

@SimonChaumet SimonChaumet requested review from a team as code owners October 8, 2024 08:06
@SimonChaumet SimonChaumet changed the title Feat/exten beta feat(pci-block-storage): add exten beta Oct 8, 2024
@SimonChaumet SimonChaumet changed the base branch from master to develop October 8, 2024 08:07
@SimonChaumet SimonChaumet marked this pull request as draft October 8, 2024 10:18
@SimonChaumet SimonChaumet force-pushed the feat/exten-beta branch 3 times, most recently from 0a9b9ac to 633d4be Compare October 8, 2024 15:20
@SimonChaumet SimonChaumet marked this pull request as ready for review October 11, 2024 12:05
ppprevost
ppprevost previously approved these changes Oct 30, 2024
View reviewed changes
Copy link
Contributor

@ppprevost ppprevost left a comment

Choose a reason for hiding this comment

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

Lot of functions but think to test them more,

@sonarcloud SonarCloud
Copy link

sonarcloud bot commented Oct 30, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@SimonChaumet
Copy link
Contributor Author

Lot of functions but think to test them more,

@ppprevost the created functions are temporary due to the catalog not replicating the backend state, it should fixed with the new product availability route or when the feature flipping is fully deployed

@anooparveti anooparveti changed the base branch from develop to release/public-cloud-w45 October 31, 2024 16:37
@anooparveti anooparveti mentioned this pull request Oct 31, 2024
Merged
8 tasks
@anooparveti anooparveti merged commit d217168 into release/public-cloud-w45 Oct 31, 2024
15 of 18 checks passed
@anooparveti anooparveti deleted the feat/exten-beta branch October 31, 2024 17:00
@SimonChaumet SimonChaumet mentioned this pull request Nov 4, 2024
Closed
8 tasks
anooparveti pushed a commit that referenced this pull request Nov 4, 2024
@SimonChaumet @ovh-cds @anooparveti
feat(pci-block-storage): add exten beta (#13489)
6b9b3f6 
Extend volume capacity to 12To
Add banner for beta extension
Implement dependent regions on block tiles
ref: TAPC-1609, TAPC-2071, TAPC-2072, TAPC-1139, 
TAPC-1505, TAPC-2063, TAPC-1711, TAPC-1716, TAPC-1728,
TAPC-1938

Signed-off-by: Simon Chaumet <[email protected]>
Co-authored-by: Yann Lojewski <[email protected]>
Co-authored-by: CDS Translator Agent <[email protected]>
anooparveti pushed a commit that referenced this pull request Nov 5, 2024
@SimonChaumet @ovh-cds @anooparveti
feat(pci-block-storage): add exten beta (#13489)
b6a2723 
Extend volume capacity to 12To
Add banner for beta extension
Implement dependent regions on block tiles
ref: TAPC-1609, TAPC-2071, TAPC-2072, TAPC-1139, 
TAPC-1505, TAPC-2063, TAPC-1711, TAPC-1716, TAPC-1728,
TAPC-1938

Signed-off-by: Simon Chaumet <[email protected]>
Co-authored-by: Yann Lojewski <[email protected]>
Co-authored-by: CDS Translator Agent <[email protected]>
ppprevost pushed a commit that referenced this pull request Nov 15, 2024
@SimonChaumet @ovh-cds
feat(pci-block-storage): add exten beta (#13489)
f8f0c9f 
Extend volume capacity to 12To
Add banner for beta extension
Implement dependent regions on block tiles
ref: TAPC-1609, TAPC-2071, TAPC-2072, TAPC-1139, 
TAPC-1505, TAPC-2063, TAPC-1711, TAPC-1716, TAPC-1728,
TAPC-1938

Signed-off-by: Simon Chaumet <[email protected]>
Co-authored-by: Yann Lojewski <[email protected]>
Co-authored-by: CDS Translator Agent <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fredericvilcot fredericvilcot fredericvilcot approved these changes

@Eric-ciccotti Eric-ciccotti Eric-ciccotti approved these changes

@Tsiorifamonjena Tsiorifamonjena Tsiorifamonjena approved these changes

@sidlynx sidlynx Awaiting requested review from sidlynx sidlynx was automatically assigned from ovh/su-digital-tools-core-stack

@kqesar kqesar Awaiting requested review from kqesar kqesar was automatically assigned from ovh/su-digital-tools-core-stack

@qpavy qpavy Awaiting requested review from qpavy qpavy was automatically assigned from ovh/team-manager-dev-angularjs

@ghyenne ghyenne Awaiting requested review from ghyenne ghyenne was automatically assigned from ovh/team-manager-dev-angularjs

@sachinrameshn sachinrameshn Awaiting requested review from sachinrameshn sachinrameshn was automatically assigned from ovh/team-manager-dev-angularjs

@ppprevost ppprevost Awaiting requested review from ppprevost

Assignees
No one assigned
Labels
feature New feature quality check: ok
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@SimonChaumet @ppprevost @Tsiorifamonjena @Eric-ciccotti @fredericvilcot @y4nnL @anooparveti