| *** mszilagyi has quit IRC | 00:00 | |
| *** dolphm has joined #openstack-dev | 00:01 | |
| *** flaviamissi has quit IRC | 00:03 | |
| *** jim33 has joined #openstack-dev | 00:03 | |
| *** flaviamissi has joined #openstack-dev | 00:04 | |
| *** jim33 has quit IRC | 00:05 | |
| *** dolphm has quit IRC | 00:07 | |
| *** flaviamissi has quit IRC | 00:08 | |
| *** eglynn__ has joined #openstack-dev | 00:09 | |
| *** jdg has quit IRC | 00:10 | |
| *** eglynn_ has quit IRC | 00:11 | |
| *** eglynn__ has quit IRC | 00:12 | |
| *** shevek_ has joined #openstack-dev | 00:12 | |
| *** eglynn__ has joined #openstack-dev | 00:13 | |
| *** dtroyer has joined #openstack-dev | 00:15 | |
| *** flaviamissi has joined #openstack-dev | 00:15 | |
| *** dtroyer has quit IRC | 00:16 | |
| *** jdg has joined #openstack-dev | 00:18 | |
| *** flaviamissi_ has joined #openstack-dev | 00:19 | |
| *** flaviamissi has quit IRC | 00:19 | |
| *** flaviamissi_ has quit IRC | 00:23 | |
| openstackgerrit | Verification of a change to openstack/nova failed: Remove unused _setup_other_managers method from test case https://review.openstack.org/4844 | 00:24 |
|---|---|---|
| *** jdg has quit IRC | 00:27 | |
| *** flaviamissi has joined #openstack-dev | 00:28 | |
| *** mortman has left #openstack-dev | 00:28 | |
| *** Ryan_Lane has joined #openstack-dev | 00:30 | |
| *** flaviamissi has quit IRC | 00:32 | |
| *** thingee has joined #openstack-dev | 00:32 | |
| *** flaviamissi has joined #openstack-dev | 00:33 | |
| *** Ryan_Lane has quit IRC | 00:35 | |
| *** flaviamissi has quit IRC | 00:37 | |
| *** flaviamissi has joined #openstack-dev | 00:38 | |
| *** rpedde has quit IRC | 00:38 | |
| *** flaviamissi has quit IRC | 00:42 | |
| *** maplebed has quit IRC | 00:51 | |
| *** spiffxp has quit IRC | 00:54 | |
| *** dolphm has joined #openstack-dev | 01:00 | |
| *** dolphm has quit IRC | 01:01 | |
| *** vincentricci has left #openstack-dev | 01:02 | |
| *** madhav has quit IRC | 01:08 | |
| *** creiht has quit IRC | 01:12 | |
| openstackgerrit | Verification of a change to openstack/nova failed: fakes.fake_data_store doesn't exist, so don't reset it https://review.openstack.org/4846 | 01:15 |
| *** littleidea has quit IRC | 01:15 | |
| *** rods has quit IRC | 01:24 | |
| *** x86brandon has quit IRC | 01:26 | |
| *** mikeyp has quit IRC | 01:35 | |
| openstackgerrit | Verification of a change to openstack/nova failed: fix for bug 821252. Smarter default scheduler https://review.openstack.org/4766 | 01:35 |
| uvirtbot` | Launchpad bug 821252 in nova "Default scheduler should be smarter than ChanceScheduler" [Wishlist,In progress] https://launchpad.net/bugs/821252 | 01:35 |
| *** jdurgin has quit IRC | 01:37 | |
| *** PotHix has quit IRC | 01:47 | |
| openstackgerrit | Verification of a change to openstack/horizon failed: add git commit date / sha1 to sphinx html docs https://review.openstack.org/4851 | 01:51 |
| vishy | mtaylor, jeblair we seem to be getting a consistent timeout on a test case | 01:52 |
| vishy | comstud: ^^ | 01:52 |
| comstud | i hate that | 01:52 |
| comstud | that one happened again? | 01:53 |
| comstud | xenapi one | 01:53 |
| *** jog0 has left #openstack-dev | 01:53 | |
| anotherjesse | vishy: if I fix dean's missing of source ./functions in https://review.openstack.org/#patch,sidebyside,4825,2,exercises/bundle.sh is the patch good to go? | 01:53 |
| openstackgerrit | Verification of a change to openstack/nova failed: Display error for invalid CIDR. https://review.openstack.org/4277 | 01:55 |
| *** shevek_ has quit IRC | 01:55 | |
| comstud | vishy: test runs fine for me.. repeatedly. completes in .03 seconds too | 01:55 |
| comstud | i'll look at it tho | 01:55 |
| comstud | not obvious where there could be a timing issue | 01:57 |
| comstud | i see.. another patch that it failed on | 01:58 |
| ayoung | termie, just ran the unit tests https://review.openstack.org/#change,4659 and alot are failing http://admiyo.fedorapeople.org/keystone_test.log | 01:59 |
| ayoung | does some library need to be updated? | 02:00 |
| *** thingee has quit IRC | 02:02 | |
| *** mnewby has quit IRC | 02:02 | |
| openstackgerrit | Verification of a change to openstack/horizon failed: Changed alert message to show actual IP instead of obj_id for 'Disassociate IP' action. https://review.openstack.org/4812 | 02:04 |
| *** littleidea has joined #openstack-dev | 02:05 | |
| *** creiht has joined #openstack-dev | 02:09 | |
| *** ChanServ sets mode: +v creiht | 02:09 | |
| *** madhav has joined #openstack-dev | 02:12 | |
| *** pixelbeat has quit IRC | 02:18 | |
| openstackgerrit | Verification of a change to openstack/nova failed: Remove unused _setup_other_managers method from test case https://review.openstack.org/4844 | 02:25 |
| openstackgerrit | Verification of a change to openstack-dev/devstack failed: Improve exercise robustness https://review.openstack.org/4825 | 02:25 |
| *** mnewby has joined #openstack-dev | 02:29 | |
| vishy | anotherjesse: apparently not quite :) | 02:31 |
| *** creiht has quit IRC | 02:31 | |
| *** creiht has joined #openstack-dev | 02:31 | |
| *** ChanServ sets mode: +v creiht | 02:31 | |
| anotherjesse | vishy: on it | 02:32 |
| anotherjesse | vishy: actually it is better. the devstack gate failure used to go past when the image was not found - https://jenkins.openstack.org/job/gate-integration-tests-devstack-vm/2066/console | 02:33 |
| anotherjesse | vishy: do you know if jeblair / mtaylor have to inject our keys to investigate? | 02:33 |
| vishy | well it didn't pass, so | 02:35 |
| vishy | i'm assuming that there is something off in the tests | 02:35 |
| anotherjesse | vishy: these changes are because it would just fly by when the image 400'd (see bug this review references) | 02:35 |
| anotherjesse | we've had this issue for a bit :( | 02:35 |
| vishy | anotherjesse: so an image 400s? | 02:36 |
| vishy | right now? | 02:36 |
| vishy | so the gate is passing even though it is broke? | 02:36 |
| anotherjesse | it is random | 02:36 |
| anotherjesse | https://bugs.launchpad.net/devstack/+bug/944593 | 02:36 |
| uvirtbot` | Launchpad bug 944593 in devstack "exercise should catch launch failure earlier" [Medium,In progress] | 02:36 |
| anotherjesse | shows that all I had to do is click "retrigger" and the same code passed | 02:37 |
| anotherjesse | the code from dean makes it stop when the image 400s instead of keeping going (and nova show failing) | 02:37 |
| anotherjesse | this isn't yet trying to fix why the image is 400-ing | 02:38 |
| anotherjesse | that is a separate issue | 02:38 |
| openstackgerrit | Verification of a change to openstack/nova failed: Fix test_migrate_disk_and_power_off_exception https://review.openstack.org/4848 | 02:45 |
| *** x86brandon has joined #openstack-dev | 02:51 | |
| *** dayou has joined #openstack-dev | 03:02 | |
| openstackgerrit | Verification of a change to openstack/nova failed: Use self.mox instead of create a new self.mocker https://review.openstack.org/4850 | 03:05 |
| *** danwent has quit IRC | 03:08 | |
| *** letterj has left #openstack-dev | 03:09 | |
| *** davlap has quit IRC | 03:14 | |
| *** davlap has joined #openstack-dev | 03:15 | |
| *** dtroyer has joined #openstack-dev | 03:25 | |
| openstackgerrit | Verification of a change to openstack/horizon failed: Return proper error message when adding security group with no params https://review.openstack.org/4804 | 03:26 |
| *** davlap has quit IRC | 03:29 | |
| openstackgerrit | Verification of a change to openstack/horizon failed: Disable submit action button until any of list item is selected https://review.openstack.org/4799 | 03:30 |
| rmk | Is the milestone ppa deprecated? | 03:33 |
| rmk | https://launchpad.net/~nova-core/+archive/milestone | 03:33 |
| rmk | Looks like it's still e2 packages is why I ask | 03:33 |
| openstackgerrit | Verification of a change to openstack/horizon failed: set image attributes not modifiable as read-only https://review.openstack.org/4797 | 03:35 |
| *** dayou has quit IRC | 03:37 | |
| *** dayou has joined #openstack-dev | 03:39 | |
| *** adjohn has quit IRC | 03:47 | |
| *** littleidea has quit IRC | 03:50 | |
| openstackgerrit | Verification of a change to openstack/horizon failed: Prevents fade-in of modal forms in case of existing previous modal form. Note: This is just a quick fix for the problem specified in the bug. In general there are problems with using overlaying modals. Each subsequent modal form doesn't 'backdrop' the pre https://review.openstack.org/4806 | 03:50 |
| openstackgerrit | Verification of a change to openstack/nova failed: Fix test_migrate_disk_and_power_off_exception https://review.openstack.org/4848 | 04:01 |
| *** mnewby has quit IRC | 04:03 | |
| *** madhav has left #openstack-dev | 04:04 | |
| *** mnewby has joined #openstack-dev | 04:10 | |
| *** maplebed has joined #openstack-dev | 04:20 | |
| *** mdomsch has quit IRC | 04:27 | |
| *** novas0x2a|laptop has joined #openstack-dev | 04:28 | |
| *** danwent has joined #openstack-dev | 04:37 | |
| openstackgerrit | Verification of a change to openstack/nova failed: Fix test_migrate_disk_and_power_off_exception https://review.openstack.org/4848 | 04:39 |
| jeblair | vishy: can you give more detail about "consistent timeout on a test case"? change # or jenkins build #? | 04:42 |
| *** maplebed has quit IRC | 04:43 | |
| jeblair | anotherjesse: we still haven't gotten approval from someone at RS to automatically give failed devstack vms to developers (I'd _really_ like to do that; the code is written!), but I can put your key on one if you want. | 04:43 |
| jeblair | anotherjesse: I've added your keys from https://launchpad.net/~anotherjesse/+sshkeys to 108.166.111.252 | 04:48 |
| jeblair | anotherjesse: ssh jenkins@108.166.111.252 | 04:49 |
| *** danwent has quit IRC | 04:49 | |
| jeblair | anotherjesse: I've also made it so that it won't be automatically deleted, so let me know when you're done with it. :) | 04:49 |
| *** Mandell has quit IRC | 04:54 | |
| *** danwent has joined #openstack-dev | 04:59 | |
| *** ayoung has quit IRC | 05:00 | |
| *** Mandell has joined #openstack-dev | 05:04 | |
| *** littleidea has joined #openstack-dev | 05:06 | |
| *** danwent has quit IRC | 05:09 | |
| *** danwent has joined #openstack-dev | 05:18 | |
| *** mnewby has quit IRC | 05:19 | |
| *** danwent has quit IRC | 05:28 | |
| *** mikal has quit IRC | 05:41 | |
| *** dtroyer has quit IRC | 05:42 | |
| *** mikal has joined #openstack-dev | 05:42 | |
| openstackgerrit | Verification of a change to openstack/horizon failed: Disable submit action button until any of list item is selected https://review.openstack.org/4799 | 05:50 |
| openstackgerrit | Verification of a change to openstack/horizon failed: Return proper error message when adding security group with no params https://review.openstack.org/4804 | 06:04 |
| *** andrewsben has joined #openstack-dev | 06:06 | |
| openstackgerrit | Verification of a change to openstack/nova failed: Fix test_migrate_disk_and_power_off_exception https://review.openstack.org/4848 | 06:22 |
| *** mdomsch_ has quit IRC | 06:23 | |
| rmk | devstack is pretty damn solid | 06:42 |
| *** andrewsben has quit IRC | 06:53 | |
| *** dayou has quit IRC | 07:06 | |
| *** mihgen has joined #openstack-dev | 07:09 | |
| *** mihgen has quit IRC | 07:15 | |
| *** reed has quit IRC | 07:27 | |
| *** mihgen has joined #openstack-dev | 07:56 | |
| *** Vek has quit IRC | 07:59 | |
| *** shevek_ has joined #openstack-dev | 08:09 | |
| *** vizsla_p has joined #openstack-dev | 08:12 | |
| *** anotherjesse is now known as anotherjesse_zz | 08:15 | |
| *** vizsla has quit IRC | 08:16 | |
| *** anotherjesse_zz is now known as anotherjesse | 08:17 | |
| *** littleidea has quit IRC | 08:17 | |
| *** shang has joined #openstack-dev | 08:20 | |
| *** shang has quit IRC | 08:20 | |
| *** armaan has joined #openstack-dev | 08:23 | |
| *** mihgen has quit IRC | 08:27 | |
| *** eglynn__ has quit IRC | 08:36 | |
| *** eglynn__ has joined #openstack-dev | 08:36 | |
| anotherjesse | jeblair: thanks - I think I have it | 08:46 |
| anotherjesse | jeblair: I think it might be https://review.openstack.org/#change,4872 | 08:52 |
| *** anotherjesse is now known as anotherjesse_zz | 09:04 | |
| *** anotherjesse_zz is now known as anotherjesse | 09:04 | |
| *** anotherjesse is now known as anotherjesse_zz | 09:05 | |
| *** anotherjesse_zz is now known as anotherjesse | 09:07 | |
| *** berendt has joined #openstack-dev | 09:10 | |
| *** anotherjesse is now known as anotherjesse_zz | 09:14 | |
| *** armaan has left #openstack-dev | 09:20 | |
| *** pixelbeat has joined #openstack-dev | 09:44 | |
| *** apevec has joined #openstack-dev | 09:48 | |
| *** Vek has joined #openstack-dev | 09:54 | |
| *** berendt has quit IRC | 10:12 | |
| *** Mandell has quit IRC | 10:13 | |
| *** apevec has quit IRC | 10:22 | |
| *** berendt has joined #openstack-dev | 10:28 | |
| *** berendt1 has joined #openstack-dev | 10:30 | |
| *** berendt has quit IRC | 10:32 | |
| *** berendt2 has joined #openstack-dev | 10:37 | |
| *** berendt1 has quit IRC | 10:37 | |
| *** berendt1 has joined #openstack-dev | 10:39 | |
| *** berendt2 has quit IRC | 10:39 | |
| *** berendt1 has quit IRC | 10:40 | |
| *** prectechCharlie has joined #openstack-dev | 10:45 | |
| *** prectechCharlie has left #openstack-dev | 10:45 | |
| *** mihgen has joined #openstack-dev | 11:06 | |
| *** mihgen has quit IRC | 12:41 | |
| *** n0ano has quit IRC | 13:04 | |
| *** tryggvil__ has joined #openstack-dev | 13:14 | |
| *** tryggvil_ has quit IRC | 13:15 | |
| *** paulormg has joined #openstack-dev | 13:42 | |
| *** bengrue has quit IRC | 13:51 | |
| *** bengrue has joined #openstack-dev | 13:51 | |
| *** dtroyer has joined #openstack-dev | 14:13 | |
| *** crobinso has quit IRC | 14:23 | |
| *** littleidea has joined #openstack-dev | 15:16 | |
| *** davidkranz has quit IRC | 15:43 | |
| *** brooksbp has joined #openstack-dev | 15:43 | |
| *** vricci has joined #openstack-dev | 15:44 | |
| *** vricci has left #openstack-dev | 15:44 | |
| *** hhoover has joined #openstack-dev | 15:45 | |
| *** eglynn has joined #openstack-dev | 15:51 | |
| *** eglynn_ has joined #openstack-dev | 15:52 | |
| *** eglynn__ has quit IRC | 15:54 | |
| *** eglynn has quit IRC | 15:56 | |
| *** eglynn__ has joined #openstack-dev | 16:04 | |
| *** rods has joined #openstack-dev | 16:04 | |
| *** eglynn_ has quit IRC | 16:07 | |
| *** pixelbeat has quit IRC | 16:08 | |
| *** eglynn has joined #openstack-dev | 16:09 | |
| *** brooksbp has quit IRC | 16:11 | |
| *** eglynn__ has quit IRC | 16:13 | |
| *** vricci has joined #openstack-dev | 16:13 | |
| *** ywu has joined #openstack-dev | 16:14 | |
| *** bencherian has joined #openstack-dev | 16:26 | |
| *** danwent has joined #openstack-dev | 16:45 | |
| *** ywu has quit IRC | 16:47 | |
| *** stuntmachine has joined #openstack-dev | 16:51 | |
| *** ywu has joined #openstack-dev | 16:53 | |
| *** Mandell has joined #openstack-dev | 16:55 | |
| *** vricci has quit IRC | 16:55 | |
| *** LinuxJedi has quit IRC | 16:55 | |
| *** littleidea has quit IRC | 16:56 | |
| *** eglynn has quit IRC | 17:11 | |
| *** stuntmachine has quit IRC | 17:12 | |
| *** Mandell has quit IRC | 17:16 | |
| *** eglynn has joined #openstack-dev | 17:16 | |
| *** eglynn has quit IRC | 17:21 | |
| openstackgerrit | Verification of a change to openstack/openstack-planet failed: Added oubiwann (Duncan McGreggor) to Planet OpenStack. https://review.openstack.org/4826 | 17:32 |
| openstackgerrit | Verification of a change to openstack/openstack-ci failed: Add bundle support. https://review.openstack.org/4047 | 17:38 |
| *** Mkenneth has quit IRC | 17:46 | |
| *** Mkenneth has joined #openstack-dev | 17:47 | |
| *** eglynn has joined #openstack-dev | 17:48 | |
| *** jdg_ has joined #openstack-dev | 17:48 | |
| *** Mkenneth has quit IRC | 17:51 | |
| *** dneary has joined #openstack-dev | 17:52 | |
| *** Mkenneth has joined #openstack-dev | 17:53 | |
| *** eglynn_ has joined #openstack-dev | 17:57 | |
| *** eglynn__ has joined #openstack-dev | 17:59 | |
| *** dneary has quit IRC | 17:59 | |
| *** Mkenneth has quit IRC | 17:59 | |
| *** eglynn has quit IRC | 18:00 | |
| *** eglynn_ has quit IRC | 18:01 | |
| *** Mkenneth has joined #openstack-dev | 18:06 | |
| *** dneary has joined #openstack-dev | 18:06 | |
| *** dneary has quit IRC | 18:06 | |
| *** dneary has joined #openstack-dev | 18:06 | |
| *** eglynn has joined #openstack-dev | 18:06 | |
| *** eglynn__ has quit IRC | 18:08 | |
| *** anotherjesse_zz is now known as anotherjesse | 18:11 | |
| *** anotherjesse is now known as anotherjesse_zz | 18:13 | |
| openstackgerrit | Verification of a change to openstack/glance failed: Added first step of babel-based translations. https://review.openstack.org/4326 | 18:13 |
| *** Mandell has joined #openstack-dev | 18:14 | |
| *** dneary has quit IRC | 18:18 | |
| *** reed has joined #openstack-dev | 18:18 | |
| *** dneary has joined #openstack-dev | 18:21 | |
| *** dneary has quit IRC | 18:21 | |
| *** dneary has joined #openstack-dev | 18:21 | |
| *** eglynn has quit IRC | 18:22 | |
| openstackgerrit | Verification of a change to openstack/openstack-ci failed: Add bundle support. https://review.openstack.org/4047 | 18:22 |
| *** pixelbeat has joined #openstack-dev | 18:25 | |
| *** dneary has quit IRC | 18:29 | |
| *** Mandell has quit IRC | 18:34 | |
| *** paulormg has quit IRC | 18:35 | |
| *** Mandell has joined #openstack-dev | 18:35 | |
| *** bsza has joined #openstack-dev | 18:42 | |
| *** LinuxJedi has joined #openstack-dev | 18:48 | |
| *** bsza has quit IRC | 18:54 | |
| *** eglynn has joined #openstack-dev | 18:54 | |
| *** jdg_ has quit IRC | 18:59 | |
| *** adjohn has joined #openstack-dev | 19:04 | |
| *** eglynn has quit IRC | 19:07 | |
| *** Mandell has quit IRC | 19:17 | |
| *** Mkenneth has quit IRC | 19:29 | |
| *** Mkenneth has joined #openstack-dev | 19:34 | |
| *** Mkenneth has quit IRC | 19:47 | |
| *** adjohn has quit IRC | 19:49 | |
| *** cjwilson has quit IRC | 19:51 | |
| *** pquerna has quit IRC | 20:01 | |
| *** dtroyer has quit IRC | 20:08 | |
| *** dtroyer has joined #openstack-dev | 20:08 | |
| *** pquerna has joined #openstack-dev | 20:09 | |
| *** bencherian has quit IRC | 20:44 | |
| *** eglynn has joined #openstack-dev | 20:45 | |
| *** bencherian has joined #openstack-dev | 20:56 | |
| *** dwalleck has joined #openstack-dev | 21:01 | |
| *** dwalleck has quit IRC | 21:04 | |
| *** sniperd has quit IRC | 21:04 | |
| *** sniperd has joined #openstack-dev | 21:05 | |
| *** eglynn has quit IRC | 21:08 | |
| *** sniperd has quit IRC | 21:09 | |
| *** eglynn has joined #openstack-dev | 21:11 | |
| *** sdake has quit IRC | 21:12 | |
| *** Mandell has joined #openstack-dev | 21:16 | |
| *** eglynn has quit IRC | 21:20 | |
| devcamcar | https://bugs.launchpad.net/keystone/+bug/945868 | 21:23 |
| uvirtbot` | Launchpad bug 945868 in keystone "Listen interface can only be 0.0.0.0" [Undecided,Confirmed] | 21:23 |
| *** sniperd has joined #openstack-dev | 21:28 | |
| *** sniperd has quit IRC | 21:33 | |
| *** x86brandon has quit IRC | 21:34 | |
| *** sniperd has joined #openstack-dev | 21:34 | |
| *** hashar has joined #openstack-dev | 21:38 | |
| *** sniperd has quit IRC | 21:38 | |
| *** hashar has quit IRC | 21:43 | |
| *** bencherian has quit IRC | 21:46 | |
| *** x86brandon has joined #openstack-dev | 21:50 | |
| *** eglynn has joined #openstack-dev | 21:51 | |
| *** bencherian has joined #openstack-dev | 22:03 | |
| *** eglynn has quit IRC | 22:05 | |
| *** danwent has quit IRC | 22:07 | |
| devcamcar | https://review.openstack.org/#change,4880 | 22:08 |
| *** novas0x2a|laptop has quit IRC | 22:15 | |
| *** x86brandon has quit IRC | 22:27 | |
| *** cjwilson has joined #openstack-dev | 22:33 | |
| chmouel | jeblair, mtaylor: is uploading reviews as draft disalowed?, ie: http://pastie.org/private/wutpqiyiowywv4atraxaw | 22:38 |
| *** hhoover has left #openstack-dev | 22:39 | |
| *** littleidea has joined #openstack-dev | 22:46 | |
| jeblair | chmouel: we haven't upgrade to 2.2.2 yet (coming soon -- we've forward ported our patches, just need to test it a bit and then do the upgrade) | 23:00 |
| chmouel | jeblair: cool, thanks. | 23:00 |
| *** utlemming has quit IRC | 23:03 | |
| *** utlemming has joined #openstack-dev | 23:03 | |
| *** troytoman-away is now known as troytoman | 23:15 | |
| *** novas0x2a|laptop has joined #openstack-dev | 23:17 | |
| *** x86brandon has joined #openstack-dev | 23:37 | |
| notmyname | chmouel: still here? | 23:39 |
| chmouel | notmyname: yes | 23:39 |
| notmyname | I just saw the bug you reported | 23:40 |
| notmyname | at frist glance, I don't think it's a bug | 23:40 |
| chmouel | for the swift3 bug? | 23:40 |
| notmyname | ya | 23:40 |
| chmouel | seems like euca-upload-bundle expecting a 404 and not a 400 | 23:41 |
| notmyname | caveat: if s3 return 404 on bad bucket name, then it's a bug and swift3 should translate | 23:41 |
| notmyname | however, 404 certainly isn't what should be returned for an invalid container name | 23:41 |
| notmyname | and a client should check for a 4xx series, not a specific code (normally) | 23:42 |
| chmouel | well so i am looking over the unittests | 23:42 |
| notmyname | same with 2xx vs 200 or 202 | 23:42 |
| chmouel | and it seem that invalidbucketname should be 404 in that test https://github.com/openstack/swift/blob/master/test/unit/common/middleware/test_swift3.py#L263 | 23:43 |
| chmouel | but i am not sure why it didn't catch it | 23:43 |
| chmouel | and maybe i misunderstand the code | 23:44 |
| notmyname | hmm | 23:44 |
| chmouel | this is btw what I am getting when using euca-upload-bundle http://pastie.org/private/y9o5rxvwsaeh27vksuwkaq | 23:46 |
| notmyname | chmouel: looks like that checks that a 404 returns InvalidBucketName. your bug is the opposite (InvalidBucketName means 404) | 23:46 |
| *** sdake has joined #openstack-dev | 23:47 | |
| chmouel | notmyname: it's not the same thing? | 23:48 |
| *** littleidea has quit IRC | 23:48 | |
| notmyname | the swift3 test you reference tests if a 404 returns InvalidBucketName. your code is assuming that InvalidBucketName is always a 404 (if I'm reading your code right) | 23:49 |
| notmyname | however, I think there might be a bug, but it's not the one you reported | 23:50 |
| notmyname | I don't know why a 404 is returning InvalidBucketName on a GET rather than NoSuchBucket (line 243 in swift3.py) | 23:50 |
| *** eglynn has joined #openstack-dev | 23:50 | |
| notmyname | that ^ is what the unit test you mentioned is testing | 23:51 |
| chmouel | yep okay i get what you say | 23:52 |
| chmouel | i'll update the bug report | 23:53 |
| notmyname | I've got to go run an errand. I'll try to look at it later | 23:53 |
| notmyname | thanks | 23:53 |
| chmouel | thanks | 23:54 |
| *** eglynn has quit IRC | 23:56 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!