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

#PR1- CP-47869-Clean up unused XAPI code. #5667

Merged
merged 23 commits into from
Jun 13, 2024

Conversation

ashwin9390
Copy link
Contributor

@ashwin9390 ashwin9390 commented Jun 3, 2024

Main ticket:CP-47869

Split from #5662

This PR is about clean up of XAPI code .

Reverted rrdd-example.py as it is required .

There is no major changes in the code for this PR ,test done in PR 5662 holds good.

Test result -198930 (Dev Run)
Ring0 BST - 8.4.0-206415 (pbb-ashwinh-xapiclean) (repo) run 30 May 2024
Tests Executed (excludes blocked): 47
Tests Failed: 1
Tests Errored: 0
Tests Blocked: 0
Tests Not Executed: 0
Test failed due to production issues: 0
Pass rate (excluding failures due to non product issues): 100.0%
Pass rate (of testcases XenRT was able to resource): 97.9%
Pass rate (of all test cases): 97.9%
Percentage of nightly complete: 100.0%
Tests excluded from numbers: 0

Signed-off-by: Ashwinh ashwin.h@cloud.com

@stephenchengCloud @liulinC @psafont

@lindig
Copy link
Contributor

lindig commented Jun 4, 2024

Are we sure this code is unused @edwintorok

@stephenchengCloud
Copy link
Contributor

Before removing these files, Ashwin had a talk with Pau and Edwin.
And I double-checked the files below, and they are not installed in Dom0:

ocaml/message-switch/core_test/message_switch_test.py
ocaml/message-switch/python/message_switch.py
ocaml/tests/has_vendor_device_test.py
scripts/hatests
scripts/mtcerrno-to-ocaml.py

@ashwin9390 Did you miss some files such as ocaml/events/event_listen.py, and scripts/time-vm-boots.py?

@ashwin9390
Copy link
Contributor Author

Before removing these files, Ashwin had a talk with Pau and Edwin. And I double-checked the files below, and they are not installed in Dom0:

ocaml/message-switch/core_test/message_switch_test.py
ocaml/message-switch/python/message_switch.py
ocaml/tests/has_vendor_device_test.py
scripts/hatests
scripts/mtcerrno-to-ocaml.py

@ashwin9390 Did you miss some files such as ocaml/events/event_listen.py, and scripts/time-vm-boots.py?

@stephenchengCloud i did not remove scripts/time-vm-boots.py as this is an example file which will be required for debugging. Pushed commit where event_listen.py has been removed.

@stephenchengCloud
Copy link
Contributor

stephenchengCloud commented Jun 12, 2024

  1. Might need to rebase to the latest code to pass the CI check
  2. Removing scripts requires also removing them from Makefile. https://github.com/xapi-project/xen-api/blob/feature/py3/scripts/Makefile#L135

…ge_switch.py

Signed-off-by: Ashwinh <ashwin.h@cloud.com>
Signed-off-by: Ashwinh <ashwin.h@cloud.com>
Signed-off-by: Ashwinh <ashwin.h@cloud.com>
Signed-off-by: Ashwinh <ashwin.h@cloud.com>
Signed-off-by: Ashwinh <ashwin.h@cloud.com>
Signed-off-by: Ashwinh <ashwin.h@cloud.com>
Signed-off-by: Ashwinh <ashwin.h@cloud.com>
Signed-off-by: Ashwinh <ashwin.h@cloud.com>
…toml

Signed-off-by: Ashwinh <ashwin.h@cloud.com>
This reverts commit 515a8b2.

Signed-off-by: Ashwinh <ashwin.h@cloud.com>
Signed-off-by: Ashwinh <ashwin.h@cloud.com>
Signed-off-by: Ashwinh <ashwin.h@cloud.com>
Signed-off-by: Ashwinh <ashwin.h@cloud.com>
Signed-off-by: Ashwinh <ashwin.h@cloud.com>
Signed-off-by: Ashwinh <ashwin.h@cloud.com>
Signed-off-by: Ashwinh <ashwin.h@cloud.com>
Signed-off-by: Ashwinh <ashwin.h@cloud.com>
Signed-off-by: Ashwinh <ashwin.h@cloud.com>
Signed-off-by: Ashwinh <ashwin.h@cloud.com>
Signed-off-by: Ashwinh <ashwin.h@cloud.com>
Signed-off-by: Ashwinh <ashwin.h@cloud.com>

This reverts commit a1b06ec.
@ashwin9390 ashwin9390 force-pushed the private/ashwin/CP-47869 branch 2 times, most recently from 79f9da6 to 3885e39 Compare June 12, 2024 10:44
@ashwin9390
Copy link
Contributor Author

  1. Might need to rebase to the latest code to pass the CI check
  2. Removing scripts requires also removing them from Makefile. https://github.com/xapi-project/xen-api/blob/feature/py3/scripts/Makefile#L135
    @stephenchengCloud : Need to figure out why ocaml test fails when iovirt is removed from scripts and Makefile .

Reverted to last successful commit which passed all the test.

Copy link
Member

@psafont psafont left a comment

Choose a reason for hiding this comment

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

I don't see any particular reason to keep these

@liulinC liulinC merged commit e56686e into xapi-project:feature/py3 Jun 13, 2024
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants