Skip to content

HDDS-11188. Initial setup for new UI layout and enable users to switch to new UI #6953

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
Merged
tanvipenumudy merged 10 commits into from
Jul 23, 2024
Merged

HDDS-11188. Initial setup for new UI layout and enable users to switch to new UI #6953

tanvipenumudy merged 10 commits into from
Jul 23, 2024

Conversation

@devabhishekpal
Copy link
Contributor

@devabhishekpal devabhishekpal commented Jul 16, 2024

What changes were proposed in this pull request?

HDDS-11188. Initial setup for new UI layout and enable users to switch to new UI

Please describe your PR in detail:

  • As a part of HDDS-11153 a new UI is being worked on for Ozone Recon
  • However since this will be done is steps we do not want to make changes in the existing stable UI
  • Hence we should allow switching of the UI to the users so that they can access the existing UI, while the work for the new UI can be done in the background before migrating.
  • This PR provides the changes required to allow users to switch to the new UI and also the initial layout for the pages

What is the link to the Apache JIRA

https://issues.apache.org/jira/browse/HDDS-11188

How was this patch tested?

Patch was tested manually.

@adoroszlai adoroszlai marked this pull request as draft July 19, 2024 08:28
@adoroszlai
Copy link
Contributor

adoroszlai commented Jul 19, 2024 •
edited
Loading

Thanks @devabhishekpal for the patch. Changed it to draft as there are some build and rat errors.

https://github.com/devabhishekpal/ozone/actions/runs/10003885766/job/27651557054#step:7:3388
https://github.com/devabhishekpal/ozone/actions/runs/10003885766/job/27651557367#step:8:11

@ArafatKhan2198 @smitajoshi12 please review

@devabhishekpal devabhishekpal marked this pull request as ready for review July 22, 2024 08:44
@devabhishekpal
Copy link
Contributor Author

devabhishekpal commented Jul 22, 2024

Hi @adoroszlai , thanks for pointing out the build issues.
I have fixed those, removed the files which are currently not required for this PR, and tested out the patch locally.
@smitajoshi12 @ArafatKhan2198 it would be great if you could take a look at this PR.
Thanks

@tanvipenumudy tanvipenumudy self-requested a review July 23, 2024 11:34
tanvipenumudy
tanvipenumudy approved these changes Jul 23, 2024
View reviewed changes
Copy link
Contributor

@tanvipenumudy tanvipenumudy left a comment

Choose a reason for hiding this comment

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

Thank you @devabhishekpal for working on the patch, +1.

ArafatKhan2198
ArafatKhan2198 approved these changes Jul 23, 2024
View reviewed changes
Copy link
Contributor

@ArafatKhan2198 ArafatKhan2198 left a comment

Choose a reason for hiding this comment

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

Thanks for working on this patch @devabhishekpal LGTM +1

@tanvipenumudy tanvipenumudy merged commit 2eed61c into apache:master Jul 23, 2024
@tanvipenumudy
Copy link
Contributor

tanvipenumudy commented Jul 23, 2024

Thank you @devabhishekpal for the patch, thank you @adoroszlai and @ArafatKhan2198 for the reviews.

@devabhishekpal devabhishekpal deleted the HDDS-11188 branch July 23, 2024 12:45
errose28 added a commit to errose28/ozone that referenced this pull request Jul 30, 2024
@errose28
Merge branch 'HDDS-10239-container-reconciliation' into HDDS-10926-mt…
be95b85 
…-delete

* HDDS-10239-container-reconciliation: (184 commits)
  HDDS-10373. Implement framework for capturing Merkle Tree Metrics. (apache#6864)
  HDDS-11188. Initial setup for new UI layout and enable users to switch to new UI (apache#6953)
  HDDS-11120. Rich rebalancing status info (apache#6911)
  HDDS-11187. Fix Event Handling in Recon OMDBUpdatesHandler to Prevent ClassCastException. (apache#6950)
  HDDS-11213. Bump commons-daemon to 1.4.0 (apache#6971)
  HDDS-11212. Bump commons-net to 3.11.1 (apache#6973)
  HDDS-11211. Bump assertj-core to 3.26.3 (apache#6972)
  HDDS-11210. Bump log4j2 to 2.23.1 (apache#6970)
  HDDS-11150. Recon Overview page crashes due to failed API Calls (apache#6944)
  HDDS-11183. Keys from DeletedTable and DeletedDirTable of AOS should be deleted on batch operation while creating a snapshot (apache#6946)
  HDDS-11198. Fix Typescript configs for Recon (apache#6961)
  HDDS-11180. Simplify HttpServer2#inferMimeType return statement (apache#6963)
  HDDS-11194. OM missing audit log for upgrade (apache#6958)
  HDDS-10389. Implement a search feature for users to locate open keys within the Open Keys Insights section. (apache#6231)
  HDDS-10561. Dashboard for delete key metrics (apache#6948)
  HDDS-11192. Increase SPNEGO URL test coverage (apache#6956)
  HDDS-11179. DBConfigFromFile#readFromFile result of toIOException not thrown (apache#6957)
  HDDS-11186. First container log missing from bundle (apache#6952)
  HDDS-10844. Clarify snapshot create error message. (apache#6955)
  HDDS-11166. Switch to Rocky Linux-based ozone-runner (apache#6942)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tanvipenumudy tanvipenumudy tanvipenumudy approved these changes

@ArafatKhan2198 ArafatKhan2198 ArafatKhan2198 approved these changes

Assignees

No one assigned

Labels

recon UI

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@devabhishekpal @adoroszlai @tanvipenumudy @ArafatKhan2198