添加虚拟机备份、编辑磁盘描述 #170
添加虚拟机备份、编辑磁盘描述 #170
Conversation
| IP_cinder = "192.168.190.11" | ||
| PORT_cinder = "8774" | ||
| #读取配置文件 | ||
| cf = ConfigParser.ConfigParser() |
There was a problem hiding this comment.
为什么要读取配置文件啊？不是都在settings里面直接引用吗？
There was a problem hiding this comment.
这些都属于可配置项，在settings里面如果给出的是pyc的文件就不能修改了
There was a problem hiding this comment.
现在配置都提取到yaml文件中了。你可以看看这个issue. #143
该issue已经修复了。
There was a problem hiding this comment.
这个版本里面配置文件还没改，版本合过去后我把这几个配置写到yaml里面去
|
|
||
| pro_path = os.path.split(os.path.realpath(__file__))[0] | ||
| LOG_PATH = os.path.join(pro_path, "log") # 日志路径 | ||
| DB_PATH = os.path.join(pro_path, "db.sqlite3") | ||
| CONFIG_PATH = os.path.join(pro_path,"setting.conf") |
There was a problem hiding this comment.
上面两行:
LOG_PATH = os.path.join(pro_path, "log") # 日志路径
DB_PATH = os.path.join(pro_path, "db.sqlite3")这两个不是全部已经配置了吗？
# Conflicts: # package/Aries/src/functions/Openstack/instances/index.jsx
| :return: | ||
| ''' | ||
| assert self.token != "","not login" | ||
| path = "/v2/%s/volumes/%s"%(self.project_id, volume_id) |
There was a problem hiding this comment.
这里的path。不要写在代码里面了。提取出来放在一个单独的文件中统一管理。
There was a problem hiding this comment.
path会随着用户和操作对象的变化而变化，如果只是把相同的部分放在公用变量里面再拼接起来好像没什么意义
There was a problem hiding this comment.
不用拼接吧。url统一定义成模版的形式使用的时候渲染然后集中放在一个地方比较好管理一点。
例如模版:
opensatckUrl = "/v1/{project_id}/volumes/{volume_id}"使用的时候渲染就可以了。
There was a problem hiding this comment.
这个不是我们程序的url地址，是openstack cinder API的地址，我不确定我们理解的是不是一样的
There was a problem hiding this comment.
你说的openstack cinder API的地址不是可以理解为第3方服务的api地址么?
|
和张鼎那边讨论是暂时先通过。然后后续url作统一的处理,该问题转移到issue #177 |
@qqzxingchen @LuPan2015