Re: [PATCH v4 3/8] Replace migrate_get_connect_uri inplace of migrate_get_socket_address

[Het Gala]

On 11/03/24 11:49 pm, Fabiano Rosas wrote:

> Het Gala <het.gala@nutanix.com> writes:
>
> >  bool migrate_watch_for_events(QTestState *who, const char *name,
> > @@ -130,7 +140,7 @@ void migrate_qmp(QTestState *who, QTestState *to, const char *uri,
> >  
> >      g_assert(!qdict_haskey(args, "uri"));
> >      if (!uri) {
> > -        connect_uri = migrate_get_socket_address(to, "socket-address");
> > +        connect_uri = migrate_get_connect_uri(to, "socket-address");
> >       
> What's the point of the "socket-address" argument here? Seems a bit
> nonsensical to me to call: migrate_get_socket_address(..., "socket-address").
>
> What about we just suppress this throughout the stack and directly call:
>
>     object = qdict_get(rsp, "socket-address");
>

Fabiano, I didn't get clearly understand your point here. From what I understand,
you want to call just
1. migrate_get_connect_uri(to) and migrate_get_connect_qdict(to)
2. delete migrate_get_socket_address(..., "socket-address") altogether 3. Just call qdict_get(rsp, "socket-address") which will return an object 4. Then convert this object into qdict and uri string respectively ?

Hmm, If that's the case, converting to qdict shouldn't be a problem. But for uri string is there a simpler method or writing a parsing function would be needed ?

Regards,

Het Gala