*** Oneiroi has quit IRC | 00:00 | |
*** openstackjenkins has quit IRC | 00:11 | |
*** andrewbogott has quit IRC | 00:11 | |
*** openstackjenkins has joined #openstack-dev | 00:12 | |
*** gyee has quit IRC | 00:16 | |
openstackgerrit | Gabriel Hurley proposed a change to openstack/horizon: Remove problematic .pyc file for local_settings after test run completion. https://review.openstack.org/2635 | 00:25 |
---|---|---|
openstackgerrit | James E. Blair proposed a change to openstack/openstack-ci-puppet: test change https://review.openstack.org/2636 | 00:27 |
*** kbringard has joined #openstack-dev | 00:27 | |
openstackgerrit | Verification of a change to openstack/openstack-ci-puppet failed: test change https://review.openstack.org/2636 | 00:38 |
*** ncode has quit IRC | 00:49 | |
*** kbringard has quit IRC | 00:54 | |
*** andrewbogott has joined #openstack-dev | 00:57 | |
*** zns has joined #openstack-dev | 01:37 | |
*** jakedahn has joined #openstack-dev | 01:47 | |
jeblair | Jenkins upgrade is complete, gerrit and jenkins are back online. | 01:53 |
mtaylor | jeblair++ | 01:54 |
*** spiffxp has quit IRC | 02:03 | |
openstackgerrit | James E. Blair proposed a change to openstack/openstack-ci: Document "Approved" changes to gerrit/jenkins. https://review.openstack.org/2637 | 02:09 |
*** deshantm has joined #openstack-dev | 02:09 | |
*** PaulM has joined #openstack-dev | 02:17 | |
*** zykes- has joined #openstack-dev | 02:27 | |
yamahata | ping blamar | 02:33 |
*** zns has quit IRC | 02:36 | |
*** zns has joined #openstack-dev | 02:36 | |
openstackgerrit | ZhongYue Luo proposed a change to openstack/python-novaclient: python-novaclient missing pep8 in pip-requires https://review.openstack.org/2638 | 02:42 |
bcwaldon | yamahata: blamar is probably offline for the night (we're in EST) | 02:42 |
bcwaldon | yamahata: send him an email at brian.lamar@rackspace.com if you need him | 02:42 |
*** mnewby has quit IRC | 02:48 | |
*** andrewbogott has quit IRC | 02:51 | |
*** andrewbogott has joined #openstack-dev | 02:51 | |
*** bcwaldon has quit IRC | 02:54 | |
openstackgerrit | Paul McMillan proposed a change to openstack/horizon: Added IP addresses by network group to instance detail page. https://review.openstack.org/2639 | 03:02 |
openstackgerrit | Paul McMillan proposed a change to openstack/horizon: Added IP addresses by network group to instance detail page. https://review.openstack.org/2639 | 03:07 |
*** hugokuo has joined #openstack-dev | 03:07 | |
*** ben_duyujie has joined #openstack-dev | 03:07 | |
*** spiffxp has joined #openstack-dev | 03:22 | |
yamahata | bcwaldon, I see thank you. | 03:27 |
*** mjfork has quit IRC | 03:41 | |
*** dtroyer has joined #openstack-dev | 04:32 | |
*** dtroyer has quit IRC | 04:47 | |
*** dtroyer has joined #openstack-dev | 04:53 | |
*** jakedahn has quit IRC | 05:04 | |
*** andrewbogott has quit IRC | 05:05 | |
*** ben_duyujie has left #openstack-dev | 05:13 | |
*** ben_duyujie has quit IRC | 05:13 | |
*** Mandell has joined #openstack-dev | 05:18 | |
openstackgerrit | guang-yee proposed a change to openstack/keystone: Add HP-IDM extension to fix Bug 890411. https://review.openstack.org/2640 | 05:23 |
openstackgerrit | guang-yee proposed a change to openstack/keystone: Add HP-IDM extension to fix Bug 890411. https://review.openstack.org/2624 | 05:23 |
uvirtbot | Launchpad bug 890411 in keystone "Tenant role conflicts/overlaps can be a security issue" [Medium,In progress] https://launchpad.net/bugs/890411 | 05:23 |
openstackgerrit | guang-yee proposed a change to openstack/keystone: Add HP-IDM extension to fix Bug 890411 https://review.openstack.org/2623 | 05:23 |
*** dweimer has quit IRC | 05:46 | |
*** dtroyer has quit IRC | 06:04 | |
openstackgerrit | Ziad Sawalha proposed a change to openstack/python-keystoneclient: Support for version discovery https://review.openstack.org/2641 | 06:24 |
*** spiffxp has quit IRC | 06:25 | |
openstackgerrit | Ziad Sawalha proposed a change to openstack/python-keystoneclient: Support for version discovery https://review.openstack.org/2641 | 06:26 |
*** deshantm has quit IRC | 06:29 | |
openstackgerrit | Ziad Sawalha proposed a change to openstack/python-keystoneclient: Support for version discovery https://review.openstack.org/2641 | 06:29 |
openstackgerrit | sleepsonthefloor proposed a change to openstack-dev/devstack: make python-keystoneclient configurable via devstack, remove pip requirement for horizon. https://review.openstack.org/2642 | 06:37 |
openstackgerrit | Nilanjan Roy proposed a change to openstack/nova: Date 28/12/2011 12:05PM IST https://review.openstack.org/2643 | 06:40 |
openstackgerrit | Nilanjan Roy proposed a change to openstack/nova: Changed some comments https://review.openstack.org/2644 | 06:40 |
*** Mandell has quit IRC | 06:43 | |
*** Mandell has joined #openstack-dev | 06:56 | |
openstackgerrit | Ewan Mellor proposed a change to openstack/keystone: Bug #909255: Endpoint handling broken on SQL backend by portable-identifiers changes https://review.openstack.org/2645 | 06:58 |
uvirtbot | Launchpad bug 909255 in keystone "Endpoint handling broken on SQL backend by portable-identifiers changes" [Undecided,In progress] https://launchpad.net/bugs/909255 | 06:58 |
*** sleepsonthefloo has quit IRC | 07:05 | |
*** sleepsonthefloo has joined #openstack-dev | 07:06 | |
*** GheRivero has joined #openstack-dev | 07:08 | |
*** feiyuliu has quit IRC | 07:10 | |
*** feiyuliu has joined #openstack-dev | 07:10 | |
*** Mandell has quit IRC | 07:15 | |
openstackgerrit | ZhongYue Luo proposed a change to openstack/nova: missing packages in nova's pip-requires https://review.openstack.org/2646 | 07:19 |
openstackgerrit | sleepsonthefloor proposed a change to openstack-dev/devstack: Port work on SERVICE_HOST from stable/diablo. Allows user to specify service endpoint host separately from HOST_IP https://review.openstack.org/2533 | 07:23 |
*** mdragon has joined #openstack-dev | 08:07 | |
*** sleepsonthefloo has quit IRC | 08:30 | |
openstackgerrit | Tres Henry proposed a change to openstack/horizon: Refactoring the JS to clean things up and pull stuff out into separate files. https://review.openstack.org/2648 | 08:44 |
openstackgerrit | Tres Henry proposed a change to openstack/horizon: Refactoring the JS to clean things up and pull stuff out into separate files. https://review.openstack.org/2649 | 08:48 |
*** mdragon has quit IRC | 08:51 | |
*** zigo has joined #openstack-dev | 09:31 | |
openstackgerrit | Ziad Sawalha proposed a change to openstack/keystone: Move URL Normalizer to Frontends https://review.openstack.org/2650 | 09:38 |
*** mnour has joined #openstack-dev | 09:51 | |
*** GheRivero has quit IRC | 10:29 | |
*** LinuxJedi has joined #openstack-dev | 10:39 | |
*** zns has quit IRC | 11:26 | |
*** pixelbeat has joined #openstack-dev | 11:30 | |
*** yamahata_ has joined #openstack-dev | 12:01 | |
*** zykes- has quit IRC | 12:48 | |
openstackgerrit | unmesh-gurjar proposed a change to openstack/nova: Added app/server name to log format(lp:904307). https://review.openstack.org/2353 | 12:51 |
*** zykes- has joined #openstack-dev | 13:00 | |
*** hugo_kuo has joined #openstack-dev | 13:15 | |
*** ben_duyujie has joined #openstack-dev | 13:19 | |
*** RobertLaptop has joined #openstack-dev | 13:22 | |
openstackgerrit | heut2008 proposed a change to openstack/nova: add a flag to fix mouse cursor problem. https://review.openstack.org/2582 | 13:28 |
openstackgerrit | A change to openstack/tempest has been rejected: Test name modified according to the comment https://review.openstack.org/2627 | 13:47 |
*** mjfork has joined #openstack-dev | 13:58 | |
*** zykes- has quit IRC | 14:03 | |
*** bepernoot has joined #openstack-dev | 14:07 | |
*** zykes- has joined #openstack-dev | 14:13 | |
*** dprince has joined #openstack-dev | 14:21 | |
*** bcwaldon has joined #openstack-dev | 14:27 | |
*** kbringard has joined #openstack-dev | 14:31 | |
openstackgerrit | A change to openstack/nova has been rejected: missing packages in nova's pip-requires https://review.openstack.org/2646 | 14:35 |
*** Oneiroi has joined #openstack-dev | 14:36 | |
*** Oneiroi has joined #openstack-dev | 14:36 | |
*** kbringard has quit IRC | 14:40 | |
*** kbringard has joined #openstack-dev | 14:40 | |
*** zigo has quit IRC | 14:44 | |
*** GheRivero has joined #openstack-dev | 15:01 | |
*** tryggvil__ has quit IRC | 15:07 | |
*** dtroyer has joined #openstack-dev | 15:08 | |
*** zns has joined #openstack-dev | 15:17 | |
*** dtroyer has quit IRC | 15:19 | |
*** kbringard has quit IRC | 15:21 | |
*** kbringard has joined #openstack-dev | 15:21 | |
*** dillon-w has joined #openstack-dev | 15:22 | |
*** hub_cap has joined #openstack-dev | 15:25 | |
*** MrHeat has joined #openstack-dev | 15:32 | |
*** dtroyer has joined #openstack-dev | 15:38 | |
*** zzed has joined #openstack-dev | 15:45 | |
*** tryggvil_ has joined #openstack-dev | 15:48 | |
*** cp16net has joined #openstack-dev | 15:55 | |
*** Oneiroi has quit IRC | 15:58 | |
openstackgerrit | Johannes Erdfelt proposed a change to openstack/glance: Synchronize notification queue setup between nova and glance https://review.openstack.org/2651 | 16:07 |
*** Mandell has joined #openstack-dev | 16:15 | |
*** cp16net has quit IRC | 16:15 | |
*** cp16net has joined #openstack-dev | 16:15 | |
*** hub_cap has quit IRC | 16:17 | |
*** hub_cap has joined #openstack-dev | 16:17 | |
openstackgerrit | Major Hayden proposed a change to openstack/glance: Fixes keystone auth test failures in python 2.6. https://review.openstack.org/2652 | 16:19 |
*** spiffxp has joined #openstack-dev | 16:22 | |
openstackgerrit | Major Hayden proposed a change to openstack/glance: Fixes keystone auth test failures in python 2.6. https://review.openstack.org/2652 | 16:26 |
*** tryggvil__ has joined #openstack-dev | 16:27 | |
*** tryggvil_ has quit IRC | 16:28 | |
*** bepernoot has quit IRC | 16:29 | |
*** jdg has joined #openstack-dev | 16:34 | |
*** troytoman-away is now known as troytoman | 16:35 | |
*** jog0 has quit IRC | 16:35 | |
*** gyee has joined #openstack-dev | 16:39 | |
*** dillon-w has quit IRC | 16:39 | |
*** ben_duyujie has quit IRC | 16:42 | |
*** spiffxp has quit IRC | 16:46 | |
*** mtaylor has quit IRC | 16:51 | |
*** mtaylor has joined #openstack-dev | 16:51 | |
*** mtaylor has quit IRC | 16:51 | |
*** mtaylor has joined #openstack-dev | 16:51 | |
*** ChanServ sets mode: +v mtaylor | 16:51 | |
*** spiffxp has joined #openstack-dev | 17:02 | |
*** bcwaldon has quit IRC | 17:02 | |
*** sleepsonthefloo has joined #openstack-dev | 17:05 | |
*** misheska has joined #openstack-dev | 17:06 | |
openstackgerrit | Dean Troyer proposed a change to openstack-dev/devstack: Clean up log files older than 7 (default) days https://review.openstack.org/2653 | 17:08 |
*** rnirmal has joined #openstack-dev | 17:14 | |
*** misheska has quit IRC | 17:14 | |
*** n0ano has joined #openstack-dev | 17:16 | |
*** johnpur has quit IRC | 17:16 | |
*** zigo has joined #openstack-dev | 17:20 | |
*** Mandell has quit IRC | 17:26 | |
openstackgerrit | Johannes Erdfelt proposed a change to openstack/glance: Synchronize notification queue setup between nova and glance https://review.openstack.org/2651 | 17:30 |
*** bepernoot has joined #openstack-dev | 17:32 | |
openstackgerrit | guang-yee proposed a change to openstack/keystone: Add HP-IDM extension to fix Bug 890411 https://review.openstack.org/2654 | 17:32 |
uvirtbot | Launchpad bug 890411 in keystone "Tenant role conflicts/overlaps can be a security issue" [Medium,In progress] https://launchpad.net/bugs/890411 | 17:32 |
openstackgerrit | Ziad Sawalha proposed a change to openstack/keystone: Move Global Role variables out of backendutils https://review.openstack.org/2655 | 17:36 |
openstackgerrit | Ziad Sawalha proposed a change to openstack/keystone: Move Global Role variables out of backendutils https://review.openstack.org/2655 | 17:38 |
*** bcwaldon has joined #openstack-dev | 17:39 | |
*** jdurgin has joined #openstack-dev | 17:42 | |
*** bcwaldon has joined #openstack-dev | 17:43 | |
openstackgerrit | Dean Troyer proposed a change to openstack/horizon: Fix ram quota display to be in MB https://review.openstack.org/2656 | 17:45 |
*** hugo_kuo has quit IRC | 17:46 | |
openstackgerrit | Dean Troyer proposed a change to openstack/horizon: Fix ram quota display to be in MB https://review.openstack.org/2656 | 17:48 |
ironcamel | any devstack people around? | 17:48 |
*** Ryan_Lane has joined #openstack-dev | 17:50 | |
*** MrHeat has quit IRC | 17:53 | |
*** mnour has quit IRC | 17:54 | |
openstackgerrit | Dan Prince proposed a change to openstack/openstack-chef: Add config options for firewall_driver. https://review.openstack.org/2657 | 17:55 |
*** Mandell has joined #openstack-dev | 17:55 | |
dtroyer | ironcamel: how can I help? | 17:58 |
ironcamel | dtroyer: i'm having trouble with keystone on a new devstack stack | 18:01 |
ironcamel | it doesn't return a service catalog | 18:01 |
ironcamel | so how is one to determine the nova endpoint? | 18:02 |
dtroyer | ironcamel: how old is your devstack checkout? | 18:03 |
ironcamel | 1 day old | 18:03 |
*** lloydde has joined #openstack-dev | 18:06 | |
dtroyer | ironcamel: are you checking out trunk on all projects? | 18:11 |
ironcamel | dtroyer: devstack checks out the projects itself, and it checks out the latest ones | 18:11 |
*** andrewbogott has joined #openstack-dev | 18:12 | |
dtroyer | ironcamel: right, jost making sure you are on trunk and not diablo or something else | 18:12 |
ironcamel | all i did was run stack.sh on a new ubuntu 11.10 cloud box | 18:13 |
ironcamel | dtroyer: this is what keystone returns, see, no catalog http://paste.openstack.org/show/3996/ | 18:15 |
openstackgerrit | A change to openstack/nova has been rejected: Adding index to instances project_id column https://review.openstack.org/2633 | 18:18 |
*** vishy has joined #openstack-dev | 18:19 | |
bcwaldon | jeblair: I'm not sure I understand why we still have a +2 in the code review category | 18:23 |
vishy | did i miss anything important? | 18:24 |
vishy | bcwaldon: only core members can +2 | 18:24 |
vishy | bcwaldon: so you can look at the review numbers to know if a person is core | 18:25 |
openstackgerrit | Verification of a change to openstack/nova failed: Add an API for associating floating IPs with DNS entries. https://review.openstack.org/2401 | 18:25 |
bcwaldon | vishy: and it says 'core review' next to the +2 | 18:25 |
bcwaldon | vishy: kk | 18:25 |
johan_-_ | do we need to approve now as well? | 18:25 |
vishy | the approve is what +2 used to do | 18:25 |
*** yamahata_ has quit IRC | 18:26 | |
jeblair | right, so only do approve when you're ready for the change to go in; there's no automatic "two core people voted so it can merge" function, that's still determined by a human via the "Approve" vote | 18:27 |
johan_-_ | but we have to manually do an approve right? | 18:27 |
johan_-_ | jeblair: perfect. thanks | 18:27 |
dtroyer | ironcamel: I'm seeing the same thing, I don't know when keystone changed, I really don't keep up with it much. are you doing this with curl or is nova client broken? | 18:27 |
*** yamahata_ has joined #openstack-dev | 18:27 | |
*** kbringard has quit IRC | 18:27 | |
ironcamel | dtroyer: i'm doing this with curl | 18:28 |
ironcamel | dtroyer: nova client handles this fine | 18:28 |
ironcamel | but the ruby bindings for example dont work | 18:28 |
ironcamel | looking at the novaclient source code i can't figure out what the hell is going on | 18:29 |
*** misheska has joined #openstack-dev | 18:29 | |
dtroyer | ironcamel: my notes are from diablo timeframe, it returned the service catalog then… | 18:29 |
ironcamel | maybe novaclient makes some assumptions when it cant find a service catalog? | 18:29 |
*** nati2 has joined #openstack-dev | 18:30 | |
ironcamel | this whole auth thing seems more complicated than it needs to be | 18:31 |
*** jog0 has joined #openstack-dev | 18:33 | |
*** tryggvil__ has quit IRC | 18:35 | |
dtroyer | ironcamel: try something like this: curl -i http://127.0.0.1:5000/v2.0/tokens -X POST -H "Content-Type: application/json" -d '{"auth": {"tenantName": "demo", "passwordCredentials": {"username": "demo", "password": "0penstack"}}}' | 18:37 |
ironcamel | so you want me to change the password to 0penstack? | 18:38 |
dtroyer | no, use whatever you set it to | 18:38 |
ironcamel | i mean that is basically what i am doing | 18:38 |
ironcamel | i showed you the response | 18:38 |
ironcamel | http://paste.openstack.org/show/3996/ | 18:38 |
dtroyer | what curl command produced that? | 18:39 |
ironcamel | i was using perl bindings which in effect do that exact curl call | 18:39 |
openstackgerrit | Verification of a change to openstack/nova failed: Update glance Xen plugin w/ purge props header. https://review.openstack.org/2625 | 18:40 |
ironcamel | dtroyer: http://paste.openstack.org/show/3997/ | 18:40 |
*** n0ano has quit IRC | 18:40 | |
dtroyer | the difference is you don't have tenant in your hash, add that, it is what made the difference for me | 18:41 |
ironcamel | ok, i will try that | 18:41 |
*** zigo has quit IRC | 18:42 | |
*** bepernoot has quit IRC | 18:42 | |
ironcamel | dtroyer: where did you get the tenantName? | 18:44 |
openstackgerrit | Verification of a change to openstack/nova failed: Update glance Xen plugin w/ purge props header. https://review.openstack.org/2625 | 18:44 |
ironcamel | dtroyer: i am looking at the openrc file that devstack creates. did you use NOVA_PROJECT_ID? | 18:45 |
dtroyer | ironcamel: yes, project is the old internal nova name for tenant | 18:45 |
ironcamel | dtroyer: is project_id now required for auth? | 18:46 |
ironcamel | dtroyer: adding tenantName fixed it! thanks | 18:47 |
dtroyer | ironcamel: I'm not certain, it may be in the keystone 2.0 api. | 18:47 |
ironcamel | i get the feeling the authors of keystone were java guys | 18:48 |
ironcamel | with all the camelCasing going on | 18:48 |
*** dtroyer has left #openstack-dev | 18:48 | |
*** dtroyer has joined #openstack-dev | 18:48 | |
dprince | jeblair/monty: any idea why this is failing to merge: https://jenkins.openstack.org/job/gate-nova-pep8/2139/console | 18:52 |
dprince | jeblair/monty: mv: cannot stat `jenkins-artifacts/branch=master,label=build/venv.tgz': No such file or directory | 18:52 |
dprince | jeblair/monty: the branch has no pep8 errors and unit tests pass, etc. | 18:52 |
andrewbogott | Looks like Jenkins is just rejecting everything on nova today. | 18:53 |
*** sdake has quit IRC | 18:53 | |
mtaylor | dprince: ooh! fun. | 18:53 |
*** Ryan_Lane has quit IRC | 18:54 | |
*** n0ano has joined #openstack-dev | 18:56 | |
dprince | mtaylor: Yeah. Its like the upstream job wasn't putting the tarball in the right place? | 18:57 |
mtaylor | dprince: fascinatingly odd. fixing. | 18:58 |
mtaylor | dprince: well, a portion of the upstream job just flat-out didn't run | 18:58 |
dprince | mtaylor: Did someone slide an exit statement in there? | 18:59 |
*** bepernoot has joined #openstack-dev | 19:01 | |
*** jdg has quit IRC | 19:01 | |
*** misheska has quit IRC | 19:03 | |
*** jdg has joined #openstack-dev | 19:05 | |
*** zns has quit IRC | 19:06 | |
*** zns has joined #openstack-dev | 19:06 | |
jeblair | i also re-triggered https://review.openstack.org/#change,2401 which failed for the same reason | 19:09 |
jeblair | bcwaldon: ^ it's in the queue after 2625 | 19:10 |
dprince | jeblair: any idea what the issue was? | 19:10 |
bcwaldon | jeblair: thanks, bud | 19:11 |
jeblair | mtaylor described the proximal cause -- the venv build job didn't run for the master branch. as for why, my best guess is minor configuration difference during the upgrade that's been ironed out now. | 19:13 |
dprince | jeblair: cool. Thanks for the info. | 19:13 |
jeblair | (nova is the only project with a multi-branch venv build at the moment) | 19:14 |
jeblair | dprince: 2525 is merged | 19:14 |
*** Ryan_Lane has joined #openstack-dev | 19:15 | |
*** Ryan_Lane1 has joined #openstack-dev | 19:16 | |
openstackjenkins | Project nova-docs build #1849: SUCCESS in 2 min 31 sec: https://jenkins.openstack.org/job/nova-docs/1849/ | 19:18 |
openstackjenkins | dan.prince: Update glance Xen plugin w/ purge props header. | 19:18 |
openstackgerrit | James E. Blair proposed a change to openstack/openstack-ci: Add script to sync CLA wiki and LP team. https://review.openstack.org/2658 | 19:18 |
*** Ryan_Lane has quit IRC | 19:19 | |
openstackgerrit | Johannes Erdfelt proposed a change to openstack/nova: Create notification queues as durable https://review.openstack.org/2659 | 19:20 |
*** Oneiroi has joined #openstack-dev | 19:25 | |
*** Ryan_Lane1 has quit IRC | 19:26 | |
*** Ryan_Lane1 has joined #openstack-dev | 19:26 | |
*** Ryan_Lane1 is now known as Ryan_Lane | 19:26 | |
vishy | soren: ping | 19:38 |
*** pixelbeat has quit IRC | 19:39 | |
*** misheska has joined #openstack-dev | 19:46 | |
openstackgerrit | Jake Dahn proposed a change to openstack/horizon: Changing the title of the user dashboard sidebar tab to 'user'. https://review.openstack.org/2660 | 19:49 |
*** nati2 has quit IRC | 19:59 | |
*** nati2 has joined #openstack-dev | 19:59 | |
*** Oneiroi has quit IRC | 19:59 | |
*** nati2_ has joined #openstack-dev | 20:03 | |
*** nati2 has quit IRC | 20:05 | |
*** bepernoot has quit IRC | 20:09 | |
openstackgerrit | Dean Troyer proposed a change to openstack-dev/devstack: Use DEFAULT_INSTANCE_TYPE in exercises https://review.openstack.org/2661 | 20:21 |
*** nati2_ has quit IRC | 20:22 | |
*** nati2 has joined #openstack-dev | 20:22 | |
openstackgerrit | Jake Dahn proposed a change to openstack/horizon: Changing the title of the user dashboard sidebar tab to 'user'. https://review.openstack.org/2660 | 20:28 |
*** Mandell has quit IRC | 20:29 | |
*** nati2_ has joined #openstack-dev | 20:39 | |
*** nati2 has quit IRC | 20:41 | |
*** bcwaldon has quit IRC | 20:45 | |
*** bcwaldon has joined #openstack-dev | 20:50 | |
*** dspano has joined #openstack-dev | 20:57 | |
openstackgerrit | Jake Dahn proposed a change to openstack/horizon: Changing the title of the user dashboard sidebar tab to 'user'. https://review.openstack.org/2660 | 20:57 |
openstackgerrit | Jake Dahn proposed a change to openstack/horizon: Changing the title of the user dashboard sidebar tab to 'user'. https://review.openstack.org/2660 | 20:59 |
openstackgerrit | Jake Dahn proposed a change to openstack/python-novaclient: Adding return statement to get_console_output https://review.openstack.org/2662 | 21:05 |
*** zzed has quit IRC | 21:05 | |
jdg | bcwaldon: got a second to talk about tests for my driver? | 21:06 |
bcwaldon | jdg: sure | 21:06 |
jdg | Cool... so I've been thinking of ideas and looking at the existing code and there are some challenges. :) | 21:06 |
bcwaldon | jdg: not surprised... | 21:07 |
jdg | Since Hudson or other users will not have access to a SolidFire device I'm not sure how I'd implement tests for folks? | 21:07 |
jdg | The only option that comes to mind is implement a "Fake" | 21:07 |
jdg | But to do that and really test the methods in the driver I'd have to come up with some way of directing san.py to use that Fake and I don't see a good way to do that. | 21:08 |
bcwaldon | jdg: yep, that's the common method of doing this | 21:08 |
bcwaldon | lemme take a look | 21:08 |
jdg | Ok... cool. | 21:08 |
bcwaldon | jdg: so one approach would be to limit a method like _issue_api_request to JUST the code you need to talk to the underlying service | 21:09 |
bcwaldon | jdg: then stub out that one method | 21:10 |
bcwaldon | so you could stub it out and have fixtures that are returned for specific method names, such as GetAccountByName | 21:10 |
jdg | bcwaldon: Ok, that would be the easiest approach as it's the only call that actually interacts with the devcie. | 21:10 |
bcwaldon | jdg: yep, I'd vote for that | 21:10 |
bcwaldon | jdg: its a common way of doing this in the project | 21:10 |
jdg | bcwaldon: Excellent!! Thanks, I didn't know if it was valid to stub out the one piece like that but it seems to be the only good option. | 21:11 |
jeblair | jdg: agree with what bcwaldon said, and i think that's defininitely the first order of business, but also... | 21:11 |
bcwaldon | jdg: I wouldn't say it's the best approach, but it has been accepted as such in the context of nova | 21:12 |
jeblair | jdg: if you have a lab setup and would like to have tests run on your hardware triggered by the openstack jenkins, we can set that up with you | 21:12 |
bcwaldon | yep, so that would be functional level testing | 21:12 |
bcwaldon | in addition to the unit testing i'm asking you to write | 21:12 |
jeblair | yep, to reiterate, i definitely think bcwaldon's unit test stuff comes first. :) | 21:12 |
jdg | jeblair: I have a simluated version of the device on my workstation, but the firewall issues and all of that cause concern for folks. | 21:13 |
jdg | I'll start with the unit test and we can go from there. | 21:13 |
bcwaldon | jdg: great, thanks! | 21:13 |
jdg | I appreciate the info, thank you!! | 21:13 |
jeblair | jdg: we have solutions to the firewall issue (everyone has that concern). you can talk to myself or mtaylor when you're ready to look into it further. | 21:14 |
mtaylor | jdg: ++ to what jeblair said | 21:15 |
*** dprince has quit IRC | 21:15 | |
jdg | jeblair: I'll link up with you next week or so and see what we can figure out. As I go along and "hopefully" start contributing more this will sure be handy | 21:15 |
jeblair | jdg: i look forward to it! | 21:16 |
jdg | jeblair: it's on my calendar :) | 21:17 |
*** Mandell has joined #openstack-dev | 21:17 | |
openstackjenkins | Project nova-docs build #1850: SUCCESS in 2 min 30 sec: https://jenkins.openstack.org/job/nova-docs/1850/ | 21:18 |
openstackjenkins | Johannes Erdfelt: Create notification queues as durable | 21:18 |
*** bengrue has joined #openstack-dev | 21:21 | |
andrewbogott | bcwaldon, is 2401 stuck in limbo because of the Jenkins misfire, or does it still need another reviewer? | 21:24 |
bcwaldon | andrewbogott: I'll look in a sec | 21:24 |
openstackgerrit | Verification of a change to openstack/horizon failed: Changing sidebar tab titles. https://review.openstack.org/2660 | 21:29 |
openstackgerrit | e0ne proposed a change to openstack/nova: Floating address range fixed https://review.openstack.org/2663 | 21:30 |
*** pixelbeat has joined #openstack-dev | 21:34 | |
*** bepernoot has joined #openstack-dev | 21:37 | |
*** lts has joined #openstack-dev | 21:39 | |
openstackgerrit | A change to openstack/nova has been rejected: Fating address range fixed https://review.openstack.org/1866 | 21:39 |
*** lts has quit IRC | 21:39 | |
*** nati2 has joined #openstack-dev | 21:40 | |
bcwaldon | andrewbogott: kicked it off again to get it in | 21:41 |
andrewbogott | thanks | 21:41 |
*** rnirmal has quit IRC | 21:41 | |
*** nati2_ has quit IRC | 21:42 | |
openstackgerrit | Jake Dahn proposed a change to openstack/python-novaclient: Adding return statement to get_console_output https://review.openstack.org/2662 | 21:46 |
*** yamahata_ has quit IRC | 21:55 | |
*** yamahata_ has joined #openstack-dev | 21:55 | |
*** GheRivero has quit IRC | 21:56 | |
andrewbogott | bcwaldon: Now I have a question about unit tests in python-novaclient. The test framework wants me to have stub functions with impossible names like 'get_os_floating_ip_dns_zone1?ip=1()'. Is there some system in place to alias such things, or is there another approach I should take? | 21:58 |
bcwaldon | andrewbogott: hmm, I haven't played with that much | 21:59 |
andrewbogott | ok. | 21:59 |
bcwaldon | andrewbogott: it may just be an issue with that mechanism | 21:59 |
bcwaldon | andrewbogott: I'd suggest you find a way to ignore the question mark and everything after it | 22:00 |
andrewbogott | Yeah, it automatically maps urls to function names in a way which clearly can't work for interesting URLs. | 22:00 |
bcwaldon | and pass the query params in as kwargs to the stub | 22:00 |
andrewbogott | Well, that may require modifying the testing framework. | 22:00 |
bcwaldon | andrewbogott: it most likely will | 22:00 |
andrewbogott | fair enough! | 22:00 |
andrewbogott | Oh, actually, maybe this is easy. | 22:02 |
openstackjenkins | Project nova-docs build #1851: SUCCESS in 2 min 43 sec: https://jenkins.openstack.org/job/nova-docs/1851/ | 22:03 |
openstackjenkins | abogott: Add an API for associating floating IPs with DNS entries. | 22:03 |
*** bepernoot has quit IRC | 22:08 | |
openstackgerrit | Jake Dahn proposed a change to openstack/horizon: Changing sidebar tab titles. https://review.openstack.org/2660 | 22:10 |
openstackgerrit | Brian Waldon proposed a change to openstack/python-novaclient: Removing cache-busting query param (fresh) https://review.openstack.org/2664 | 22:12 |
*** hub_cap has quit IRC | 22:16 | |
*** bcwaldon has quit IRC | 22:19 | |
openstackgerrit | e0ne proposed a change to openstack/nova: Floating address range fixed https://review.openstack.org/2663 | 22:21 |
*** cp16net has quit IRC | 22:25 | |
openstackgerrit | Johannes Erdfelt proposed a change to openstack/nova: Be explicit with kombu to avoid redeclaring incompatible exchanges https://review.openstack.org/2665 | 22:42 |
*** dspano has quit IRC | 22:42 | |
*** spiffxp has quit IRC | 22:50 | |
*** mdragon has joined #openstack-dev | 22:50 | |
*** Gordonz has joined #openstack-dev | 23:02 | |
*** davlap has quit IRC | 23:03 | |
*** hub_cap has joined #openstack-dev | 23:06 | |
openstackgerrit | Jake Dahn proposed a change to openstack/python-novaclient: Simplifying get_console_output client interface. https://review.openstack.org/2666 | 23:14 |
*** nati2 has quit IRC | 23:40 | |
*** nati2 has joined #openstack-dev | 23:41 | |
openstackgerrit | andrewbogott proposed a change to openstack/python-novaclient: Improve the test framework to handle urls with args. https://review.openstack.org/2667 | 23:43 |
openstackgerrit | andrewbogott proposed a change to openstack/python-novaclient: Add the python api for floating IP DNS. https://review.openstack.org/2668 | 23:43 |
*** dtroyer has quit IRC | 23:45 | |
*** Gordonz_ has joined #openstack-dev | 23:46 | |
*** spiffxp has joined #openstack-dev | 23:47 | |
*** Ryan_Lane has quit IRC | 23:48 | |
*** Gordonz has quit IRC | 23:49 | |
openstackgerrit | sleepsonthefloor proposed a change to openstack-dev/devstack: make python-keystoneclient configurable via devstack, remove pip requirement for horizon. https://review.openstack.org/2642 | 23:56 |
*** RobertLaptop has left #openstack-dev | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!