NobodyCam | max_lobur romcheg lucasagomes lets keep our eyes on https://review.openstack.org/#/c/70348 | 00:01 |
---|---|---|
max_lobur | ack | 00:02 |
max_lobur | https://review.openstack.org/#/c/73440/3 devananda, NobodyCam | 00:15 |
NobodyCam | max_lobur: passed | 00:15 |
max_lobur | tests passed | 00:15 |
max_lobur | :) | 00:15 |
NobodyCam | :) already +2'd | 00:15 |
NobodyCam | is romcheg still here? | 00:16 |
NobodyCam | devananda: afk atm | 00:16 |
max_lobur | the context lib approach was much prettier, but we unfortunately we won't be able to use only it, we need a public method to release resources manually. that's where complexity comes from | 00:17 |
max_lobur | now i'm going sleep :) | 00:18 |
max_lobur | night NobodyCam | 00:18 |
NobodyCam | night max_lobur :) | 00:18 |
NobodyCam | brb | 00:19 |
*** max_lobur has quit IRC | 00:19 | |
*** anniec has quit IRC | 00:29 | |
*** anniec has joined #openstack-ironic | 00:33 | |
*** anniec has quit IRC | 01:06 | |
*** epim has quit IRC | 01:10 | |
openstackgerrit | Josh Gachnang proposed a change to openstack/ironic: Removing unused get_node_power_state https://review.openstack.org/73467 | 01:18 |
*** anniec has joined #openstack-ironic | 01:20 | |
*** nosnos has joined #openstack-ironic | 01:39 | |
*** wusWWW has quit IRC | 01:46 | |
*** derekh has quit IRC | 02:06 | |
openstackgerrit | Shuangtai Tian proposed a change to openstack/ironic: Remove dependent module py3kcompat from ironic https://review.openstack.org/73479 | 02:22 |
*** rongze has joined #openstack-ironic | 02:28 | |
*** anniec has quit IRC | 02:32 | |
*** anniec has joined #openstack-ironic | 02:37 | |
*** rongze has quit IRC | 02:52 | |
*** anniec has quit IRC | 02:57 | |
*** anniec has joined #openstack-ironic | 03:11 | |
*** jcooley_ has joined #openstack-ironic | 03:26 | |
openstackgerrit | Ruby Loo proposed a change to openstack/ironic: Add common.service config options to sample https://review.openstack.org/73488 | 03:27 |
*** rloo has quit IRC | 03:37 | |
*** anniec has quit IRC | 03:50 | |
*** jcooley_ has quit IRC | 04:24 | |
*** harlowja_ is now known as harlowja_away | 04:31 | |
*** saju_m has joined #openstack-ironic | 04:39 | |
*** epim has joined #openstack-ironic | 04:55 | |
*** jcooley_ has joined #openstack-ironic | 05:13 | |
*** loki_ has joined #openstack-ironic | 05:16 | |
*** anniec has joined #openstack-ironic | 05:31 | |
*** anniec_ has joined #openstack-ironic | 05:36 | |
*** anniec has quit IRC | 05:38 | |
*** anniec_ is now known as anniec | 05:38 | |
*** epim_ has joined #openstack-ironic | 05:53 | |
*** epim has quit IRC | 05:54 | |
*** epim_ is now known as epim | 05:54 | |
*** saju_m has quit IRC | 05:58 | |
openstackgerrit | Jenkins proposed a change to openstack/ironic: Imported Translations from Transifex https://review.openstack.org/71192 | 06:06 |
*** vkozhukalov has joined #openstack-ironic | 06:21 | |
*** jcooley_ has quit IRC | 06:23 | |
*** lazy_prince has joined #openstack-ironic | 07:00 | |
*** saju_m has joined #openstack-ironic | 07:25 | |
*** vkozhukalov has quit IRC | 07:33 | |
*** Guest19634 has quit IRC | 07:46 | |
openstackgerrit | Shuangtai Tian proposed a change to openstack/ironic: emove dependent module py3kcompat from ironic https://review.openstack.org/73479 | 07:48 |
*** epim has quit IRC | 07:51 | |
openstackgerrit | Shuangtai Tian proposed a change to openstack/ironic: Remove module py3kcompat from ironic https://review.openstack.org/73479 | 08:05 |
*** jistr has joined #openstack-ironic | 08:06 | |
*** romcheg1 has joined #openstack-ironic | 08:18 | |
*** ndipanov has joined #openstack-ironic | 08:25 | |
*** mdurnosvistov_ has joined #openstack-ironic | 08:29 | |
*** aignatov_ is now known as aignatov | 08:35 | |
*** Haomeng|2 has joined #openstack-ironic | 08:36 | |
*** yuriyz has joined #openstack-ironic | 08:40 | |
*** Haomeng has quit IRC | 08:42 | |
*** ndipanov has quit IRC | 08:49 | |
*** dshulyak has joined #openstack-ironic | 08:49 | |
*** hemna has quit IRC | 08:59 | |
*** ifarkas has joined #openstack-ironic | 09:01 | |
*** vkozhukalov has joined #openstack-ironic | 09:05 | |
*** derekh has joined #openstack-ironic | 09:13 | |
*** hemna has joined #openstack-ironic | 09:14 | |
*** mdurnosvistov_ has quit IRC | 09:14 | |
*** lucasagomes has joined #openstack-ironic | 09:28 | |
*** athomas has joined #openstack-ironic | 09:35 | |
*** saju_m has quit IRC | 09:51 | |
*** vkozhukalov has quit IRC | 09:53 | |
*** tatyana has joined #openstack-ironic | 09:58 | |
*** vkozhukalov has joined #openstack-ironic | 10:05 | |
*** hemna has quit IRC | 10:08 | |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Make the PXE driver understand ephemeral disks https://review.openstack.org/65750 | 10:15 |
*** max_lobur_afk is now known as max_lobur | 10:17 | |
*** hemna has joined #openstack-ironic | 10:21 | |
*** nosnos has quit IRC | 10:21 | |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Make the PXE driver understand ephemeral disks https://review.openstack.org/65750 | 10:21 |
mdurnosvistov | Morning folks! =) | 10:23 |
*** romcheg1 has quit IRC | 10:23 | |
Haomeng|2 | mdurnosvistov: morning:) | 10:24 |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Make the PXE driver understand ephemeral disks https://review.openstack.org/65750 | 10:24 |
lucasagomes | I should have slept more :P | 10:24 |
lucasagomes | morning folks | 10:24 |
Haomeng|2 | :) | 10:25 |
max_lobur | morning Ironic | 10:26 |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Make the PXE driver understand ephemeral disks https://review.openstack.org/65750 | 10:26 |
lucasagomes | morning max_lobur | 10:26 |
Haomeng|2 | max_lobur: morning:) | 10:27 |
max_lobur | morning Lucas, Haomeng :) | 10:27 |
Haomeng|2 | max_lobur: :) | 10:27 |
*** aignatov is now known as aignatov_ | 10:33 | |
*** dshulyak has quit IRC | 10:34 | |
*** dshulyak has joined #openstack-ironic | 10:45 | |
yuriyz | morning all | 10:48 |
Haomeng|2 | yuriyz: morning:) | 10:51 |
*** rsacharya has joined #openstack-ironic | 10:55 | |
*** rsacharya has quit IRC | 10:56 | |
*** romcheg1 has joined #openstack-ironic | 10:56 | |
*** rsacharya has joined #openstack-ironic | 10:57 | |
*** rsacharya has quit IRC | 10:57 | |
*** rsacharya has joined #openstack-ironic | 10:58 | |
*** rsacharya has quit IRC | 10:59 | |
*** rsacharya has joined #openstack-ironic | 11:00 | |
yuriyz | lucasagomes, question about node POST | 11:01 |
yuriyz | I think predefined uuid can be posted in some cases | 11:01 |
yuriyz | what you think? | 11:04 |
*** rsacharya has quit IRC | 11:04 | |
max_lobur | hi yuriyz, I remember we already support this | 11:06 |
max_lobur | what you guys discussing? | 11:06 |
Haomeng|2 | yuriyz: you mean we pass the uuid for posting? | 11:07 |
yuriyz | we pass uuid in tests POST at least | 11:10 |
lucasagomes | yuriyz, hi yuirty yea as max said I think it's already supported | 11:14 |
lucasagomes | if uuid is passed it uses it | 11:14 |
lucasagomes | if not it generate a random one | 11:15 |
lucasagomes | although the client doesn't support passing an UUID | 11:15 |
lucasagomes | but it's kinda a hidden func and makes me wonder whether we should do that or not | 11:15 |
lucasagomes | http://paste.openstack.org/show/65392/ | 11:17 |
max_lobur | lucasagomes, I think it's not really needed for CLI. My thoughts was that it can be usefull for someone who builds system around ironic, use ironic lib, and do thier own modelling first (already have uuid) | 11:19 |
max_lobur | but for cli - I'd not use it :) | 11:19 |
max_lobur | to support it by client lib we need just update CREATION ATTRIBUTES https://github.com/openstack/python-ironicclient/blob/master/ironicclient/v1/node.py#L20-L21 | 11:22 |
*** pbrooko has joined #openstack-ironic | 11:22 | |
max_lobur | we can add it to wish list if needed | 11:22 |
lucasagomes | max_lobur, ack, yea | 11:26 |
lucasagomes | that would be useful in the lib indeed | 11:26 |
lucasagomes | so +1 :) | 11:26 |
*** lazy_prince has left #openstack-ironic | 11:36 | |
*** mgoddard has quit IRC | 11:38 | |
*** mgoddard_ has joined #openstack-ironic | 11:38 | |
romcheg1 | Morning folks | 11:39 |
romcheg1 | I need some opinions | 11:40 |
romcheg1 | I was thinking about migrating things from Nova in terms of testing them automatically | 11:41 |
*** romcheg1 is now known as romcheg_ltp | 11:41 | |
romcheg_ltp | To perform the migration we need several parameters that would allow access to nova's databases | 11:42 |
openstackgerrit | Yuriy Zveryanskyy proposed a change to openstack/ironic: Do not allow POST node's internal attributes https://review.openstack.org/73281 | 11:42 |
*** aignatov_ is now known as aignatov | 11:42 | |
romcheg_ltp | In order to be able to test that automatically I think about adding a new configuration section to Ironic config. That section should contain all required parameters to connect to Nova's DBs | 11:43 |
romcheg_ltp | Using nova's config would be better but it's a singleton, so we cannot spawn two instances | 11:44 |
romcheg_ltp | lucasagomes, max_lobur what do you think about adding a new section to Ironic's config? ^ | 11:45 |
romcheg_ltp | yuriyz and everyone else, are also welcome to share your opinion | 11:45 |
max_lobur | romcheg_ltp, is it possible to use those singletone? or you mean it will live in another process | 11:46 |
max_lobur | and it's cross-process singleton | 11:46 |
yuriyz | I think this not very good because this section intended only for mirgation, not for production config | 11:47 |
max_lobur | As we need these options only for one-time migration, I don't think it's good to add this to conf | 11:47 |
max_lobur | maybe to have some temp conf for migration purposes? | 11:47 |
max_lobur | if it's impossible to reuse nova's one | 11:47 |
openstackgerrit | Haomeng,Wang proposed a change to openstack/ironic: Supports OPERATOR priv level for ipmitool driver https://review.openstack.org/73578 | 11:47 |
yuriyz | +1 for separate config | 11:47 |
yuriyz | this is one-shot task | 11:48 |
max_lobur | yep | 11:48 |
max_lobur | so what about nova's. Is it really cross-process singleton? | 11:49 |
* max_lobur still not very familiar with oslo config :-/ | 11:49 | |
max_lobur | bbiaw | 11:53 |
*** loki_ has quit IRC | 11:54 | |
lucasagomes | romcheg_ltp, sorry for the delay, yea I agree with yuriyz and max_lobur | 11:59 |
openstackgerrit | Haomeng,Wang proposed a change to openstack/ironic: Supports OPERATOR priv level for ipmitool driver https://review.openstack.org/73578 | 12:06 |
romcheg_ltp | lucasagomes: Then I have to create a separate configuration file and copy Ironic's and Nova's options there | 12:15 |
*** ifarkas has quit IRC | 12:18 | |
*** ifarkas has joined #openstack-ironic | 12:20 | |
openstackgerrit | Yuriy Zveryanskyy proposed a change to openstack/ironic: Do not allow POST node's internal attributes https://review.openstack.org/73281 | 12:20 |
agordeev | Morning Ironic. What you think of https://bugs.launchpad.net/ironic/+bug/1280267 ? | 12:23 |
Haomeng|2 | agordeev: morning:) | 12:30 |
Haomeng|2 | agordeev: yes, agree with you, make sense | 12:30 |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Support preserve_ephemeral https://review.openstack.org/68236 | 12:40 |
*** jbjohnso has joined #openstack-ironic | 12:46 | |
max_lobur | romcheg, can we use both Ironic and temp nova's in one process? | 12:53 |
max_lobur | romcheg_ltp, ^^ | 12:53 |
*** lucasagomes is now known as lucas-hungry | 13:06 | |
romcheg_ltp | max_lobur: No I need options that will conflict | 13:08 |
romcheg_ltp | Easier to copy them to a separate config | 13:08 |
openstackgerrit | Yuriy Zveryanskyy proposed a change to openstack/ironic: Do not allow POST ports and chassis internal attributes https://review.openstack.org/73593 | 13:21 |
*** jdob has joined #openstack-ironic | 13:21 | |
*** mgoddard_ has quit IRC | 13:29 | |
*** vkozhukalov has quit IRC | 13:34 | |
openstackgerrit | Mikhail Durnosvistov proposed a change to openstack/ironic: Get rid object model `dict` methods part 3 https://review.openstack.org/64108 | 13:36 |
openstackgerrit | Mikhail Durnosvistov proposed a change to openstack/ironic: Get rid object model `dict` methods part 1 https://review.openstack.org/60025 | 13:36 |
openstackgerrit | Mikhail Durnosvistov proposed a change to openstack/ironic: Get rid object model `dict` methods part 2 https://review.openstack.org/62331 | 13:36 |
openstackgerrit | Mikhail Durnosvistov proposed a change to openstack/ironic: Get rid object model `dict` methods part 5 https://review.openstack.org/64278 | 13:36 |
openstackgerrit | Mikhail Durnosvistov proposed a change to openstack/ironic: Get rid object model `dict` methods part 4 https://review.openstack.org/64251 | 13:36 |
openstackgerrit | Jarrod Johnson proposed a change to stackforge/pyghmi: Fix large paste to SOL session https://review.openstack.org/73598 | 13:42 |
*** vkozhukalov has joined #openstack-ironic | 13:47 | |
*** Shrews has joined #openstack-ironic | 13:49 | |
openstackgerrit | Jarrod Johnson proposed a change to stackforge/pyghmi: Fix large paste to SOL session https://review.openstack.org/73598 | 13:59 |
openstackgerrit | A change was merged to stackforge/pyghmi: Fix large paste to SOL session https://review.openstack.org/73598 | 14:01 |
*** lucas-hungry is now known as lucasagomes | 14:10 | |
*** Haomeng|2 has quit IRC | 14:22 | |
*** Haomeng has joined #openstack-ironic | 14:23 | |
openstackgerrit | Sirushti Murugesan proposed a change to openstack/ironic: Adds support for deploying Disk Images https://review.openstack.org/73054 | 14:28 |
*** matty_dubs|gone is now known as matty_dubs | 14:31 | |
*** romcheg_ltp has left #openstack-ironic | 14:40 | |
openstackgerrit | Jarrod Johnson proposed a change to stackforge/pyghmi: Fix race condition on sending data https://review.openstack.org/73610 | 14:42 |
NobodyCam | good morning Ironic | 14:43 |
NobodyCam | TGIF | 14:44 |
lucasagomes | morning NobodyCam | 14:45 |
lucasagomes | oh yeahhhh | 14:46 |
openstackgerrit | Jarrod Johnson proposed a change to stackforge/pyghmi: Fix race condition on sending data https://review.openstack.org/73610 | 14:48 |
NobodyCam | morning lucasagomes :) | 14:49 |
openstackgerrit | A change was merged to stackforge/pyghmi: Fix race condition on sending data https://review.openstack.org/73610 | 14:50 |
NobodyCam | hey hey lucasagomes you see https://review.openstack.org/#/c/70348/ | 14:52 |
NobodyCam | romcheg: ^^ | 14:52 |
lucasagomes | NobodyCam, nice haven't seem that no | 14:52 |
NobodyCam | we should keep our eyes on it. | 14:53 |
lucasagomes | +1 def I will do that | 14:54 |
*** yuriyz has quit IRC | 14:57 | |
max_lobur | morning NobodyCam :) | 14:58 |
NobodyCam | morning max_lobur | 15:00 |
*** Haomeng has quit IRC | 15:01 | |
NobodyCam | I see you already saw devananda and lucasagomes' comment in the review | 15:01 |
lucasagomes | NobodyCam, which review? | 15:01 |
lucasagomes | devstack? | 15:01 |
NobodyCam | 73440 | 15:02 |
lucasagomes | ah yea | 15:02 |
lucasagomes | it's good to add a small test to stress that code | 15:03 |
max_lobur | NobodyCam, yea, working on that now | 15:03 |
max_lobur | actually spotted one more old problem | 15:03 |
NobodyCam | :) | 15:03 |
max_lobur | will bring it to the team with the new patch | 15:03 |
NobodyCam | w00t | 15:04 |
max_lobur | *with new rev | 15:04 |
*** romcheg_ltp has joined #openstack-ironic | 15:14 | |
romcheg_ltp | Gah! | 15:15 |
*** ifarkas has quit IRC | 15:17 | |
max_lobur | ? | 15:17 |
romcheg_ltp | We have to sync oslo in order to have a separate config file for the migration tool | 15:17 |
romcheg_ltp | But we cannot do that until https://review.openstack.org/#/c/72984/ lands | 15:17 |
romcheg_ltp | It returns lazy loading which is required for Ironic to start :) | 15:18 |
*** ifarkas has joined #openstack-ironic | 15:19 | |
*** aignatov is now known as aignatov_ | 15:19 | |
*** aignatov_ is now known as aignatov | 15:22 | |
*** jdob has quit IRC | 15:23 | |
*** jdob has joined #openstack-ironic | 15:23 | |
*** pbrooko has quit IRC | 15:36 | |
*** jdob has quit IRC | 15:36 | |
*** jdob has joined #openstack-ironic | 15:36 | |
max_lobur | brb | 15:38 |
max_lobur | back | 15:46 |
NobodyCam | Wb back | 15:48 |
*** comstud is now known as bearhands | 15:56 | |
*** thedodd has joined #openstack-ironic | 15:56 | |
openstackgerrit | Max Lobur proposed a change to openstack/ironic: Fix task_manager acquire post landing for c4f2f26ed https://review.openstack.org/73440 | 16:10 |
* NobodyCam looks | 16:10 | |
max_lobur | finally I came up with version equal to what we had with context lib | 16:12 |
max_lobur | I didn't realize how smart it was | 16:12 |
max_lobur | folks, when you reviewing that ^^ patch, please note that it should work exactly like https://github.com/openstack/ironic/blob/de30d60aaf4cff72b1ea21c3d389c33a6da2cc45/ironic/conductor/task_manager.py#L121-L132 | 16:13 |
NobodyCam | mr on his toes today! | 16:16 |
*** russellb is now known as rustlebee | 16:16 | |
NobodyCam | Mr J is on his toes :-p | 16:17 |
*** romcheg_ltp has quit IRC | 16:25 | |
*** vkozhukalov has quit IRC | 16:26 | |
NobodyCam | max_lobur: tempest failure | 16:29 |
NobodyCam | haven't lookd in to the log | 16:29 |
NobodyCam | brb | 16:30 |
max_lobur | hmm | 16:31 |
max_lobur | looking | 16:31 |
max_lobur | BuildErrorException: Server 964b1a14-2459-4f2d-93d4-abb643c6fe89 failed to build and is in ERROR status | 16:32 |
max_lobur | and so on | 16:32 |
max_lobur | We don't have any tests for locking in tempest yet. as I recall | 16:32 |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Fix dev-quick-start.rst post landing for 9d81333fd0 https://review.openstack.org/73642 | 16:32 |
max_lobur | :) | 16:35 |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add wsme custom BooleanType type https://review.openstack.org/73279 | 16:35 |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Use the custom wsme BooleanType on the nodes api https://review.openstack.org/73284 | 16:37 |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: API: Expose a way to start/stop the console https://review.openstack.org/72998 | 16:38 |
*** loki_ has joined #openstack-ironic | 16:42 | |
*** loki_ has quit IRC | 16:44 | |
*** saju_m has joined #openstack-ironic | 16:54 | |
*** aignatov is now known as aignatov_ | 16:56 | |
*** vkozhukalov has joined #openstack-ironic | 16:56 | |
*** ifarkas has quit IRC | 16:59 | |
*** anniec has quit IRC | 17:00 | |
lucasagomes | devananda, http://logs.openstack.org/84/73284/2/check/gate-ironic-python26/ffac58e/console.html | 17:03 |
lucasagomes | devananda, happened in my local tests as well, do you think it's something with the new stevedore (0.14)? | 17:03 |
lucasagomes | I haven't looked deep into it yet | 17:03 |
*** matty_dubs is now known as matty_dubs|lunch | 17:16 | |
*** vkozhukalov has quit IRC | 17:17 | |
*** athomas has quit IRC | 17:25 | |
*** tatyana has quit IRC | 17:25 | |
*** dshulyak has quit IRC | 17:26 | |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Add wsme custom BooleanType type https://review.openstack.org/73279 | 17:29 |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: Use the custom wsme BooleanType on the nodes api https://review.openstack.org/73284 | 17:31 |
NobodyCam | max_lobur: I don't 73440 in the queue but I also dont see a reply from Mr j | 17:38 |
max_lobur | https://review.openstack.org/#/c/73440/ tempest failed | 17:43 |
max_lobur | or what do you mean | 17:43 |
max_lobur | BuildErrorException: Server 964b1a14-2459-4f2d-93d4-abb643c6fe89 failed to build and is in ERROR status | 17:43 |
max_lobur | that's tempest error | 17:43 |
max_lobur | any reason why we have Integer as primary key in conductor table? | 17:46 |
NobodyCam | max_lobur: do you see -1 on the revew? | 17:49 |
NobodyCam | is it auto inc? | 17:49 |
max_lobur | I see -1 | 17:51 |
max_lobur | Patch Set 4: Doesn't seem to work | 17:51 |
max_lobur | Build failed. For information on how to proceed, see https://wiki.openstack.org/wiki/GerritJenkinsGit#Test_Failures | 17:51 |
max_lobur | gate-ironic-pep8 SUCCESS in 1m 27s | 17:51 |
max_lobur | gate-ironic-docs SUCCESS in 7m 50s | 17:51 |
max_lobur | gate-ironic-python26 SUCCESS in 5m 33s | 17:51 |
max_lobur | gate-ironic-python27 SUCCESS in 2m 18s | 17:51 |
*** jistr has quit IRC | 17:51 | |
max_lobur | check-tempest-dsvm-ironic FAILURE in 18m 58s | 17:51 |
max_lobur | check-tempest-dsvm-ironic-postgres FAILURE in 16m 19s | 17:51 |
*** bearhands has quit IRC | 17:51 | |
max_lobur | auto what ? :) | 17:51 |
*** bearhands has joined #openstack-ironic | 17:52 | |
NobodyCam | ahh now i see | 17:53 |
max_lobur | forgot cmd+R? :) | 17:53 |
*** hstimer has joined #openstack-ironic | 17:53 | |
NobodyCam | i refreshed several times and did not see anything | 17:53 |
NobodyCam | :-p | 17:54 |
max_lobur | strange | 17:54 |
*** epim has joined #openstack-ironic | 17:55 | |
NobodyCam | started recheck for Bug #1252618 | 18:02 |
NobodyCam | on that | 18:02 |
*** derekh has quit IRC | 18:04 | |
devananda | g'morning | 18:05 |
*** bearhands is now known as comstud | 18:07 | |
max_lobur | morning devananda | 18:07 |
NobodyCam | good mornig devananda | 18:08 |
*** comstud is now known as bearhands | 18:08 | |
*** jistr has joined #openstack-ironic | 18:09 | |
*** jistr has quit IRC | 18:09 | |
* devananda edits ironic wiki | 18:13 | |
NobodyCam | brb | 18:15 |
*** epim has quit IRC | 18:15 | |
*** matty_dubs|lunch is now known as matty_dubs | 18:16 | |
openstackgerrit | Max Lobur proposed a change to openstack/ironic: Fix JSONEncodedDict default values https://review.openstack.org/68413 | 18:17 |
max_lobur | done 4 today | 18:19 |
max_lobur | going to be in channel some time on weekend | 18:19 |
max_lobur | have a nice rest folks! | 18:19 |
*** max_lobur is now known as max_lobur_afk | 18:20 | |
openstackgerrit | Lucas Alvares Gomes proposed a change to openstack/ironic: API: Expose a way to start/stop the console https://review.openstack.org/72998 | 18:26 |
lucasagomes | max_lobur_afk, enjoy the weekend | 18:26 |
lucasagomes | I'm also done for the day | 18:27 |
lucasagomes | have a good night everybody | 18:27 |
lucasagomes | enjoy the weekend | 18:27 |
*** lucasagomes has quit IRC | 18:28 | |
devananda | ok, https://wiki.openstack.org/wiki/Ironic is now much simpler | 18:30 |
devananda | NobodyCam: https://review.openstack.org/#/c/73440/ looks good to me now | 18:45 |
devananda | max_lobur_afk: comment posted to 73440 explaining why this issue happened. your unit tests LGTM, thanks! | 18:45 |
*** thedodd has quit IRC | 18:46 | |
*** hstimer has quit IRC | 18:47 | |
devananda | still dont know why it seemed to only happen on py26, though | 18:48 |
*** bearhands is now known as comstud | 18:49 | |
*** comstud is now known as bearhands | 18:49 | |
*** wusWWW has joined #openstack-ironic | 18:51 | |
*** Maha has joined #openstack-ironic | 19:00 | |
*** saju_m has quit IRC | 19:11 | |
openstackgerrit | A change was merged to openstack/ironic: Fix task_manager acquire post landing for c4f2f26ed https://review.openstack.org/73440 | 19:30 |
*** thedodd has joined #openstack-ironic | 19:32 | |
*** Maha has quit IRC | 19:42 | |
NobodyCam | ahh Ty devananda for starting rechecks :) | 19:46 |
devananda | np | 19:56 |
devananda | looks like swift broke the gate | 19:56 |
devananda | which is why we got all those tempest failures | 19:56 |
devananda | NobodyCam: https://review.openstack.org/#/c/72067/ could use a +A | 19:56 |
devananda | and let's see if https://review.openstack.org/#/c/67415/ passes (if so, let's try to lan dit again) | 19:57 |
* devananda gets lunch | 19:57 | |
NobodyCam | :) | 19:58 |
*** vkozhukalov has joined #openstack-ironic | 20:02 | |
*** hstimer has joined #openstack-ironic | 20:17 | |
NobodyCam | devananda: https://jenkins05.openstack.org/job/gate-ironic-python26/11/console py26 error still | 20:32 |
NobodyCam | on 67415 | 20:32 |
*** max_lobur has joined #openstack-ironic | 20:32 | |
NobodyCam | Failed to load driver fake_pxe. still | 20:32 |
NobodyCam | brb | 20:32 |
max_lobur | unexpectedly back :-P | 20:33 |
max_lobur | because we still have 26 failure | 20:34 |
max_lobur | :-/ | 20:35 |
max_lobur | do we need to rebase those hanging patches to let fix actually land on them? | 20:37 |
NobodyCam | lol | 20:39 |
NobodyCam | WUB max_lobur | 20:40 |
max_lobur | :) | 20:40 |
devananda | urg | 20:40 |
devananda | https://jenkins05.openstack.org/job/gate-ironic-python26/11/parameters/ | 20:40 |
devananda | contains the fix | 20:40 |
devananda | that is a py26 fail from https://review.openstack.org/#/c/67415/ | 20:41 |
devananda | just now | 20:41 |
NobodyCam | ya | 20:41 |
devananda | urgcommit aa6dcc9f3f1c60c11f8ff1bbd57cb5bbceba0740 | 20:41 |
devananda | after zuul rebased it on top of commit aa6dcc9f3f1c60c11f8ff1bbd57cb5bbceba0740 | 20:41 |
*** romcheg1 has joined #openstack-ironic | 20:41 | |
openstackgerrit | A change was merged to openstack/ironic: Make ironic-api not single threaded https://review.openstack.org/72851 | 20:48 |
max_lobur | looks like my threading patch is the only candidate to broke things. looked through what we merged and did not found anything else | 20:51 |
NobodyCam | but why to I get this | 20:54 |
NobodyCam | py26: commands succeeded | 20:54 |
NobodyCam | http://paste.openstack.org/show/4GQQIPSQ6EzwqhsubYwY/ | 20:56 |
NobodyCam | quick walkies brb | 20:57 |
max_lobur | when did you run tox-r last time? | 20:57 |
max_lobur | maybe it's because of some package | 20:58 |
max_lobur | wait, don't start tox -r | 20:59 |
max_lobur | try .tox/py26/usr/bin/pip freeze - and write that to some file | 21:02 |
max_lobur | after we can try with tox -r | 21:02 |
max_lobur | if it will fail then again pip freeze | 21:02 |
max_lobur | and diff | 21:02 |
max_lobur | ah, there's also pip freeze in 26 logs | 21:05 |
max_lobur | so we can compare now | 21:05 |
max_lobur | .tox/py26/bin/pip freeze | 21:05 |
max_lobur | is correct | 21:05 |
NobodyCam | that was fresh env from 5 minutes before rm'd /tox and ran the same cmd | 21:16 |
NobodyCam | s/\//\./ | 21:17 |
max_lobur | k | 21:18 |
NobodyCam | http://paste.openstack.org/show/zqsBmJghGYQlITa1YtFM/ | 21:19 |
max_lobur | meanwhile some patches passing jenkinks | 21:46 |
max_lobur | some no | 21:46 |
NobodyCam | ya | 21:46 |
max_lobur | any Ideas? I just checked driver factory and din't found any reason why it may fail because of unreleased resource | 21:47 |
max_lobur | also checked patch that added those test | 21:47 |
max_lobur | update neutron | 21:47 |
max_lobur | did not found anything as well | 21:47 |
NobodyCam | ya | 21:48 |
max_lobur | the only diff bitween NobodyCam 's env and jenkins is docutils==0.11 | 21:48 |
NobodyCam | and I can get it to fail fo me :( | 21:48 |
max_lobur | which I think is not what we looking 4 | 21:49 |
max_lobur | so it failed at your's too? | 21:49 |
NobodyCam | s/can/cann't/ | 21:49 |
max_lobur | ah | 21:49 |
max_lobur | hmmm | 21:49 |
max_lobur | wth | 21:50 |
max_lobur | as for my today's 'fix' | 21:50 |
max_lobur | according to traceback it releases resources | 21:50 |
max_lobur | so this is not the problem | 21:51 |
NobodyCam | are you able to run tests localy? | 21:51 |
max_lobur | seems so | 21:51 |
max_lobur | nope :( not now | 21:51 |
NobodyCam | :( | 21:51 |
max_lobur | it will take a long time to build env | 21:51 |
max_lobur | was going to do this on weekend | 21:51 |
NobodyCam | your on a air? | 21:52 |
max_lobur | yep | 21:54 |
max_lobur | I have 2 | 21:54 |
max_lobur | desktop in the office | 21:54 |
NobodyCam | thats what I use. :) | 21:54 |
max_lobur | which is fine I think | 21:54 |
NobodyCam | heheeh | 21:54 |
max_lobur | and air's is stale | 21:54 |
NobodyCam | like a f'n rock | 21:54 |
max_lobur | :) | 21:55 |
NobodyCam | better then my *cough* work laptop runnig ubuntu | 21:55 |
NobodyCam | I always have one to two vbox vms running | 21:55 |
NobodyCam | for work windows image | 21:56 |
NobodyCam | and run all my tox test directly from the code dir on the air | 21:56 |
max_lobur | why do you need windows | 21:57 |
NobodyCam | i5-1.8ghz with 8 gb ram | 21:57 |
NobodyCam | outlook with usb auth token | 21:57 |
max_lobur | i'm using ubuntu server + sshfs to mount work dir | 21:57 |
max_lobur | ahh | 21:57 |
NobodyCam | ahh | 21:58 |
NobodyCam | real cloud fs : http://en.wikipedia.org/wiki/GmailFS | 21:58 |
max_lobur | :) | 21:59 |
NobodyCam | lol | 21:59 |
NobodyCam | J/k | 21:59 |
NobodyCam | max_lobur: have you seen https://wiki.openstack.org/wiki/HackingNovaMacOSX | 22:00 |
max_lobur | yep | 22:01 |
NobodyCam | :) | 22:01 |
max_lobur | but didn't try | 22:01 |
max_lobur | seems easier to spin ubuntu server | 22:01 |
NobodyCam | the only thing I'd add would be the apple dev tools for command line (xcode i think) | 22:02 |
NobodyCam | lol | 22:02 |
NobodyCam | :) | 22:02 |
*** jdob has quit IRC | 22:03 | |
*** matty_dubs is now known as matty_dubs|gone | 22:09 | |
max_lobur | k, still can't find anything | 22:18 |
max_lobur | will have a rest | 22:18 |
max_lobur | and try tomorrow | 22:19 |
max_lobur | night NobodyCam | 22:20 |
*** max_lobur has quit IRC | 22:21 | |
openstackgerrit | A change was merged to openstack/ironic: Move eventlent monkeypatch out of cmd/ https://review.openstack.org/72067 | 22:24 |
*** rwsu has quit IRC | 22:34 | |
NobodyCam | night max_lobur_afk | 22:40 |
NobodyCam | brb | 22:41 |
*** mdurnosvistov_ has joined #openstack-ironic | 22:49 | |
*** rwsu has joined #openstack-ironic | 23:04 | |
*** vkozhukalov has quit IRC | 23:04 | |
*** thedodd has quit IRC | 23:12 | |
*** romcheg1 has quit IRC | 23:18 | |
*** romcheg1 has joined #openstack-ironic | 23:20 | |
*** rwsu has quit IRC | 23:22 | |
*** lazy_prince has joined #openstack-ironic | 23:51 | |
*** killer_prince has quit IRC | 23:52 | |
*** lazy_prince is now known as killer_prince | 23:52 | |
NobodyCam | oh darn Dyslexia | 23:54 |
*** killer_prince has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!