Update patch set 2
Patch Set 2: (2 inline comments) So #1 & #2 I think are clear. +1 #3 is state of the world today, but given the new efforts that are spinning up around that, I'm sort of +0 on this. I think convention carries us forward, and we don't need policy. I'm pretty strongly -1 on #4. I feel like the burden is on Heat at this point to get more OpenStack integrated, not the other way around. For instance, they shipped a broken i2 release because they didn't have enough integration testing to block a keystoneclient change. #5 I'm -0. I feel like there might be enough exceptions to this that I think it's part of the horizon team to make priorities during their cycle which new integrated projects with stable APIs they want to pull in. I'm not sure TC is the right place to dictate. Patch-set: 2
This commit is contained in:
parent
5a183ec0f8
commit
c940830789
|
@ -1,5 +1,22 @@
|
|||
{
|
||||
"comments": [
|
||||
{
|
||||
"key": {
|
||||
"uuid": "AAAAUH/+hHY\u003d",
|
||||
"filename": "reference/incubation-integration-requirements",
|
||||
"patchSetId": 2
|
||||
},
|
||||
"lineNbr": 84,
|
||||
"author": {
|
||||
"id": 2750
|
||||
},
|
||||
"writtenOn": "2014-01-24T20:46:27Z",
|
||||
"side": 1,
|
||||
"message": "I\u0027m not so sure we want to codify this now, especially with SDK possibly going in a different set of projects.",
|
||||
"revId": "fd1389a51d5f38c1dd3055836f555856e5a20f66",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": false
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "AAAAUH/+zho\u003d",
|
||||
|
@ -52,6 +69,23 @@
|
|||
"revId": "fd1389a51d5f38c1dd3055836f555856e5a20f66",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": false
|
||||
},
|
||||
{
|
||||
"key": {
|
||||
"uuid": "AAAAUH/+hHk\u003d",
|
||||
"filename": "reference/incubation-integration-requirements",
|
||||
"patchSetId": 2
|
||||
},
|
||||
"lineNbr": 85,
|
||||
"author": {
|
||||
"id": 2750
|
||||
},
|
||||
"writtenOn": "2014-01-24T20:46:27Z",
|
||||
"side": 1,
|
||||
"message": "Heat isn\u0027t yet really in the integrated gate (it\u0027s there in such a light way that it shipped a broken i2 release). Until that time requiring Heat is way beyond scope.",
|
||||
"revId": "fd1389a51d5f38c1dd3055836f555856e5a20f66",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": false
|
||||
}
|
||||
]
|
||||
}
|
Loading…
Reference in New Issue