cloudstack-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Mike Tutkowski (JIRA)" <j...@apache.org>
Subject [jira] [Created] (CLOUDSTACK-9383) Hyper-V communication broken by change in variable names
Date Sat, 21 May 2016 03:05:12 GMT
Mike Tutkowski created CLOUDSTACK-9383:
------------------------------------------

             Summary: Hyper-V communication broken by change in variable names
                 Key: CLOUDSTACK-9383
                 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-9383
             Project: CloudStack
          Issue Type: Bug
      Security Level: Public (Anyone can view this level - this is the default.)
          Components: Management Server
    Affects Versions: 4.9.0
         Environment: N/A
            Reporter: Mike Tutkowski
             Fix For: 4.9.0


Per e-mails to dev@:

Tutkowski, Mike <Mike.Tutkowski@netapp.com>
Thu 5/19/2016 10:53 PM

Thanks for sending out this e-mail, Anshul.

This is a bit of a strange situation because we need to make sure people are either aware
of the fact that properties in Command classes are serialized (and not change existing variable
names) or come up with a less fragile way of choosing property names when sending data (perhaps
using annotations).

At the very least, we should have comments in these classes indicating the dangers of changing
property names. It might also be beneficial to have unit tests in place that expect certain
variable names and assert if they are not as expected.

In the meanwhile, I plan to change the variable names back that were changed in PR #816.

Additional thoughts on how this should be addressed long term?

Thanks!
Mike

Anshul Gangwar <anshul.gangwar@accelerite.com>
Thu 5/19/2016 10:47 PM
To:
dev@cloudstack.apache.org;
You replied on 5/19/2016 10:53 PM.

Hi,

We should not allow renaming of variables in classes which ends with Command and TO. As these
objects are meant to be consumed by Agents.

Agents may not be written in java so relying on these variable names to get the info. One
such example is Hyper-V agent.

Hyper-V support is currently broken as there are some variables renamed in PR https://github.com/apache/cloudstack/pull/816.

Regards,
Anshul



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Mime
View raw message