ravening commented on pull request #5227:
URL: https://github.com/apache/cloudstack/pull/5227#issuecomment-895183913


   > @ravening sorry for not being able to understand but how this will help 
admins against adding source host details for event entries?
   > Even with changes admin will have to use SQL query. Similar details can be 
fetched using `event` table as well and changes can be made to add source host 
in the event description:
   > 
   > ```
   > > SELECT * FROM event WHERE description LIKE '%VM Id: 104%' AND 
type='VM.MIGRATE' ORDER BY id DESC\G
   > *************************** 1. row ***************************
   >          id: 7114
   >        uuid: 0a63c788-7ee9-4612-81ad-57c45bc0d54c
   >        type: VM.MIGRATE
   >       state: Completed
   > description: Successfully completed migrating VM. VM Id: 104 to host Id: 2
   >     user_id: 2
   >  account_id: 2
   >   domain_id: 1
   >     created: 2021-08-02 07:11:41
   >       level: INFO
   >    start_id: 7112
   >  parameters: NULL
   >    archived: 0
   >     display: 1
   > *************************** 2. row ***************************
   >          id: 7113
   >        uuid: 0b23a367-2187-4415-a7c9-624b8e956733
   >        type: VM.MIGRATE
   >       state: Started
   > description: migrating VM. VM Id: 104 to host Id: 2
   >     user_id: 2
   >  account_id: 2
   >   domain_id: 1
   >     created: 2021-08-02 07:11:27
   >       level: INFO
   >    start_id: 7112
   >  parameters: NULL
   >    archived: 0
   >     display: 1
   > *************************** 3. row ***************************
   >          id: 7112
   >        uuid: 98c658eb-5fed-4f27-b94b-cac85c9e612a
   >        type: VM.MIGRATE
   >       state: Scheduled
   > description: Attempting to migrate VM id: 104 to host Id: 2
   >     user_id: 2
   >  account_id: 2
   >   domain_id: 1
   >     created: 2021-08-02 07:11:27
   >       level: INFO
   >    start_id: 0
   >  parameters: NULL
   >    archived: 0
   >     display: 1
   > ```
   > 
   > Also, why names of hosts and IDs?
   
   @shwstppr 
   
   ```
   *************************** 24. row ***************************
                 id: 24
              state: Completed
        instance_id: 17
      instance_name: i-2-17-VM
        description: Completed migration of vm i-2-17-VM
            vm_type: User
        source_host: node42
   destination_host: node41
            created: 2021-07-20 15:31:07
          user_name: admin
         account_id: 2
          domain_id: 1
   ```
   
   Im adding a new table `vm_migration` which has fewer details related to 
migration of vm which can be easily using instance_name, search for only failed 
migrations. search for all migrations related to a particular host.
   
   I know it can be obtained from either events table or from listEvents api 
but the information available there is not easy to read. 
   
   
   This will make getting data easier but if you guys think that this feature 
is not needed then I can close this pr.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to