[ https://issues.apache.org/jira/browse/CLOUDSTACK-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16700575#comment-16700575 ]
ASF GitHub Bot commented on CLOUDSTACK-3049: -------------------------------------------- bwsw commented on issue #3058: CLOUDSTACK-3049: update dynamic role for an account URL: https://github.com/apache/cloudstack/pull/3058#issuecomment-442096532 > It's end of my day, let me get back to you soon. In short, my hesitation stems from the fact that the operation in question has some security and privileges implications. As a general rule such an update param/operation must be privileged and checked. It's better to have this as separate api. Take example of other apis which change ownership, transfer relationships etc all cases which could too be solved by an update api as well. @rhtyd Well, if we decide that only global admin (not domain admin) can change that, then you are right, it's better to implement as a separate API call. ---------------------------------------------------------------- This is an automated message from the Apache Git Service. To respond to the message, please log on GitHub and use the URL above to go to the specific comment. For queries about this service, please contact Infrastructure at: us...@infra.apache.org > marvin-nose plugin causes other nose plugins to fail when not enabled > --------------------------------------------------------------------- > > Key: CLOUDSTACK-3049 > URL: https://issues.apache.org/jira/browse/CLOUDSTACK-3049 > Project: CloudStack > Issue Type: Bug > Security Level: Public(Anyone can view this level - this is the > default.) > Components: Test Tools > Affects Versions: 4.1.0, 4.2.0 > Reporter: Prasanna Santhanam > Assignee: Prasanna Santhanam > Priority: Major > Fix For: 4.2.0 > > > enabled by default. Running regular nosetests with plugins other than marvin > will fail because of this. Failure manifests itself as nose looking for the > config file when using the marivn-nose plugin > ~/workspace/cloudstack/incubator-cloudstack(branch:master*) ยป nosetests > test.py > Traceback (most recent call last): > File "/usr/local/bin/nosetests", line 8, in <module> > load_entry_point('nose==1.3.0', 'console_scripts', 'nosetests')() > File "/Library/Python/2.7/site-packages/nose/core.py", line 118, in __init__ > **extra_args) > File > "/System/Library/Frameworks/Python.framework/Versions/2.7/lib/python2.7/unittest/main.py", > line 94, in __init__ > self.parseArgs(argv) > File "/Library/Python/2.7/site-packages/nose/core.py", line 135, in > parseArgs > self.config.configure(argv, doc=self.usage()) > File "/Library/Python/2.7/site-packages/nose/config.py", line 344, in > configure > self.plugins.configure(options, self) > File "/Library/Python/2.7/site-packages/nose/plugins/manager.py", line 284, > in configure > cfg(options, config) > File "/Library/Python/2.7/site-packages/nose/plugins/manager.py", line 99, > in __call__ > return self.call(*arg, **kw) > File "/Library/Python/2.7/site-packages/nose/plugins/manager.py", line 167, > in simple > result = meth(*arg, **kw) > File "/Library/Python/2.7/site-packages/marvin/marvinPlugin.py", line 61, > in configure > deploy = deployDataCenter.deployDataCenters(options.config) > File "/Library/Python/2.7/site-packages/marvin/deployDataCenter.py", line > 32, in __init__ > raise IOError("config file %s not found. please specify a valid config > file"%cfgFile) > IOError: config file ./datacenter.cfg not found. please specify a valid > config file -- This message was sent by Atlassian JIRA (v7.6.3#76005)