[
https://issues.apache.org/jira/browse/CLOUDSTACK-3049?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16700558#comment-16700558
]
ASF GitHub Bot commented on CLOUDSTACK-3049:
--------------------------------------------
rhtyd commented on issue #3058: CLOUDSTACK-3049: update dynamic role for an
account
URL: https://github.com/apache/cloudstack/pull/3058#issuecomment-442093555
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.
----------------------------------------------------------------
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)
