should not fetch runtime-info of vm from cached_nodes #40655
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Why are these changes needed?
Power-on-off status is runtime info of VM, should not fetch it from cached-nodes, which is probably dirty data.
It should query by pyvmomi_sdk every time.
Test
We have internal automation pipelines passed. Also team tested the GPU functionalities manually.
![image](https://private-user-images.githubusercontent.com/85480625/277941400-40154468-91e0-41f8-bb56-d0aa305c549e.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjA1NzM1MjgsIm5iZiI6MTcyMDU3MzIyOCwicGF0aCI6Ii84NTQ4MDYyNS8yNzc5NDE0MDAtNDAxNTQ0NjgtOTFlMC00MWY4LWJiNTYtZDBhYTMwNWM1NDllLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MTAlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzEwVDAxMDAyOFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTg3ZDM4MTU4ZTBmYTRjZjVlMjg5ZjEzM2Q0OTQ0M2Q2MzBkYWQwMWJkYTdhMDgwNDg2MjBkY2MxZDMxNGEzZDAmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.Rva33pQK2cnuPaa0M1QVOLWZQc_1GqIawiMPxtMjvD8)
![image](https://private-user-images.githubusercontent.com/85480625/277941676-52bab37c-0349-42d9-9070-a06688af0a7c.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MjA1NzM1MjgsIm5iZiI6MTcyMDU3MzIyOCwicGF0aCI6Ii84NTQ4MDYyNS8yNzc5NDE2NzYtNTJiYWIzN2MtMDM0OS00MmQ5LTkwNzAtYTA2Njg4YWYwYTdjLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNDA3MTAlMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjQwNzEwVDAxMDAyOFomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTQ4Y2MwNDIzNTIzZTM3OWExNGZiMWZiYzA5ZGQ1NWM5ODFhOTRhYmFhMTdkZTFiY2M4OGYzY2EyYmQ3MmZmMTkmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JmFjdG9yX2lkPTAma2V5X2lkPTAmcmVwb19pZD0wIn0.ziKLCoETWCN-AE4Jp0OrHqUgTRsXOCnbVRPweyGAfmI)
Checks
git commit -s
) in this PR.scripts/format.sh
to lint the changes in this PR.method in Tune, I've added it in
doc/source/tune/api/
under thecorresponding
.rst
file.