*** rcernin has joined #openstack-ironic | 00:03 | |
*** zhangfei has joined #openstack-ironic | 00:15 | |
*** zhangfei has quit IRC | 00:19 | |
*** livelace2 has quit IRC | 00:24 | |
*** livelace2 has joined #openstack-ironic | 00:24 | |
*** mjturek has joined #openstack-ironic | 00:34 | |
*** rpioso is now known as rpioso|afk | 00:37 | |
*** hamzy has joined #openstack-ironic | 00:38 | |
*** bdodd has joined #openstack-ironic | 00:56 | |
*** namnh has joined #openstack-ironic | 00:56 | |
*** namnh has quit IRC | 01:01 | |
*** phuongnh has joined #openstack-ironic | 01:02 | |
openstackgerrit | Michael Turek proposed openstack/ironic master: Pass prep_boot_part_uuid to install_bootloader for ppc64* partition images https://review.openstack.org/579159 | 01:03 |
---|---|---|
*** mjturek has quit IRC | 01:04 | |
*** gyee has quit IRC | 01:08 | |
*** ddurst has quit IRC | 01:09 | |
*** ddurst has joined #openstack-ironic | 01:09 | |
*** trungnv has quit IRC | 01:13 | |
*** trungnv has joined #openstack-ironic | 01:13 | |
*** tiendc has joined #openstack-ironic | 01:26 | |
*** rloo has quit IRC | 01:33 | |
*** zhangfei has joined #openstack-ironic | 01:45 | |
openstackgerrit | Merged openstack/ironic master: Add conductor_group to node notifications https://review.openstack.org/585371 | 01:46 |
*** namnh has joined #openstack-ironic | 01:47 | |
*** namnh has quit IRC | 01:50 | |
*** namnh has joined #openstack-ironic | 01:50 | |
openstackgerrit | Merged openstack/ironic master: Add deploy_step to NodePayload.SCHEMA https://review.openstack.org/585413 | 01:52 |
*** namnh has quit IRC | 01:52 | |
*** zhangfei has quit IRC | 01:57 | |
*** zhangfei has joined #openstack-ironic | 02:10 | |
*** openstack has joined #openstack-ironic | 02:49 | |
*** ChanServ sets mode: +o openstack | 02:49 | |
*** zhangfei has quit IRC | 03:01 | |
*** zhangfei has joined #openstack-ironic | 03:14 | |
*** edleafe has quit IRC | 03:14 | |
*** edleafe has joined #openstack-ironic | 03:15 | |
*** Sukhdev has joined #openstack-ironic | 03:27 | |
openstackgerrit | Nguyen Van Trung proposed openstack/ironic master: Fix for failure in cleaning in iRMC driver https://review.openstack.org/585936 | 03:29 |
openstackgerrit | vinay kumar muddu proposed openstack/ironic master: Documentation update of iLO BIOS settings https://review.openstack.org/585202 | 03:37 |
openstackgerrit | Nguyen Van Trung proposed openstack/ironic master: Follow-up to improve pep8 checking with hacking https://review.openstack.org/585939 | 03:39 |
*** gkadam has joined #openstack-ironic | 03:49 | |
*** mjura has joined #openstack-ironic | 04:40 | |
*** Sukhdev has quit IRC | 04:44 | |
*** Sukhdev has joined #openstack-ironic | 04:47 | |
*** jaganathan has joined #openstack-ironic | 04:48 | |
*** jaganathan has quit IRC | 04:49 | |
*** jaganathan has joined #openstack-ironic | 04:49 | |
*** e0ne has joined #openstack-ironic | 05:06 | |
*** mraineri has quit IRC | 05:06 | |
*** rwsu has quit IRC | 05:23 | |
*** Sukhdev has quit IRC | 05:31 | |
*** zhangfei has quit IRC | 05:33 | |
*** gkadam has quit IRC | 05:39 | |
*** zhangfei has joined #openstack-ironic | 05:47 | |
*** links has joined #openstack-ironic | 05:48 | |
*** e0ne has quit IRC | 06:00 | |
*** rbartal has joined #openstack-ironic | 06:02 | |
*** mraineri has joined #openstack-ironic | 06:12 | |
*** skazi has joined #openstack-ironic | 06:26 | |
*** serlex has joined #openstack-ironic | 06:41 | |
*** mgoddard has joined #openstack-ironic | 06:41 | |
*** exodusftw has quit IRC | 06:42 | |
*** exodusftw has joined #openstack-ironic | 06:42 | |
*** mgoddard has quit IRC | 06:50 | |
openstackgerrit | vinay kumar muddu proposed openstack/ironic master: Documentation update of iLO BIOS settings https://review.openstack.org/585202 | 06:52 |
openstackgerrit | vinay kumar muddu proposed openstack/ironic master: Documentation update of iLO BIOS settings https://review.openstack.org/585202 | 06:55 |
*** hoonetorg has quit IRC | 06:57 | |
openstackgerrit | yolanda.robla proposed openstack/ironic master: Add automated_clean field https://review.openstack.org/585795 | 07:03 |
*** gkadam has joined #openstack-ironic | 07:04 | |
*** liuzz has joined #openstack-ironic | 07:11 | |
*** tssurya has joined #openstack-ironic | 07:12 | |
*** hoonetorg has joined #openstack-ironic | 07:14 | |
*** racedo has joined #openstack-ironic | 07:21 | |
*** pcaruana has joined #openstack-ironic | 07:25 | |
*** ajya has joined #openstack-ironic | 07:27 | |
openstackgerrit | Nguyen Van Trung proposed openstack/ironic master: Refactor RAID configuration via iRMC driver https://review.openstack.org/585977 | 07:30 |
*** mgoddard has joined #openstack-ironic | 07:31 | |
*** AlexeyAbashkin has joined #openstack-ironic | 07:33 | |
*** zhangfei has quit IRC | 07:42 | |
*** dougsz has joined #openstack-ironic | 07:43 | |
*** dtantsur|afk is now known as dtantsur | 07:46 | |
dtantsur | morning ironic | 07:46 |
dtantsur | edleafe: likely the conductor did not start. tripleo has a lot of drivers by default. | 07:46 |
*** zhangfei has joined #openstack-ironic | 07:55 | |
*** mjura has quit IRC | 07:58 | |
*** mjura has joined #openstack-ironic | 07:58 | |
*** jiapei has joined #openstack-ironic | 08:13 | |
*** e0ne has joined #openstack-ironic | 08:17 | |
openstackgerrit | Nguyen Van Trung proposed openstack/ironic master: Update docs to portgroup with creating windows images https://review.openstack.org/585613 | 08:19 |
*** dougsz has quit IRC | 08:29 | |
openstackgerrit | yolanda.robla proposed openstack/ironic master: Add automated_clean field to the node object https://review.openstack.org/585991 | 08:30 |
*** ajya has quit IRC | 08:32 | |
*** ajya has joined #openstack-ironic | 08:32 | |
*** skazi has quit IRC | 08:33 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Follow-up patch for 7c5a04c1149f14900f504f32e000a7b4e69e661f https://review.openstack.org/585994 | 08:33 |
openstackgerrit | OpenStack Release Bot proposed openstack/python-ironic-inspector-client stable/rocky: Update .gitreview for stable/rocky https://review.openstack.org/585995 | 08:34 |
openstackgerrit | OpenStack Release Bot proposed openstack/python-ironic-inspector-client stable/rocky: Update UPPER_CONSTRAINTS_FILE for stable/rocky https://review.openstack.org/585996 | 08:34 |
openstackgerrit | OpenStack Release Bot proposed openstack/python-ironic-inspector-client master: Update reno for stable/rocky https://review.openstack.org/585997 | 08:34 |
*** tesseract has joined #openstack-ironic | 08:42 | |
openstackgerrit | yolanda.robla proposed openstack/ironic master: Add automated_clean field https://review.openstack.org/585795 | 08:42 |
openstackgerrit | Merged openstack/python-ironic-inspector-client master: Fix setup.cfg for release https://review.openstack.org/585857 | 08:42 |
openstackgerrit | Will Miller proposed openstack/ironic-inspector master: Allow nested action value formatting https://review.openstack.org/585779 | 08:43 |
*** derekh has joined #openstack-ironic | 08:43 | |
*** namnh has joined #openstack-ironic | 08:49 | |
*** sambetts|afk has quit IRC | 08:57 | |
*** sambetts_ has joined #openstack-ironic | 08:59 | |
*** stendulker has joined #openstack-ironic | 09:02 | |
openstackgerrit | Ankit Kumar proposed openstack/ironic-tempest-plugin master: Test cases for iLO drivers in standalone ironic https://review.openstack.org/535652 | 09:03 |
*** pmannidi has quit IRC | 09:08 | |
openstackgerrit | yolanda.robla proposed openstack/ironic master: Add automated_clean field to the node object https://review.openstack.org/585991 | 09:10 |
*** namnh has quit IRC | 09:16 | |
*** zhangfei has quit IRC | 09:19 | |
*** d0ugal has joined #openstack-ironic | 09:21 | |
openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Reorder unittest assertions https://review.openstack.org/586017 | 09:21 |
*** dougsz has joined #openstack-ironic | 09:25 | |
openstackgerrit | Hamdy Khader proposed openstack/ironic master: Add support for smartNICs https://review.openstack.org/583573 | 09:27 |
*** zhangfei has joined #openstack-ironic | 09:31 | |
openstackgerrit | Derek Higgins proposed openstack/ironic master: Switch ironic-tempest-dsvm-pxe_ipmitool-postgres https://review.openstack.org/583185 | 09:37 |
openstackgerrit | Derek Higgins proposed openstack/ironic master: Switch ironic-tempest-dsvm-ipa-wholedisk-bios-agent_ipmitool-tinyipa https://review.openstack.org/583337 | 09:37 |
openstackgerrit | Derek Higgins proposed openstack/ironic master: Switch ironic-tempest-dsvm-ipa-wholedisk-bios-pxe_snmp-tinyipa https://review.openstack.org/583546 | 09:37 |
openstackgerrit | Derek Higgins proposed openstack/ironic master: Switch ironic-tempest-dsvm-ipa-partition-uefi-pxe_ipmitool-tinyipa https://review.openstack.org/583551 | 09:37 |
openstackgerrit | Derek Higgins proposed openstack/ironic master: Switch ironic-tempest-dsvm-ipa-partition-pxe_ipmitool-tinyipa-python3 https://review.openstack.org/583555 | 09:37 |
*** stendulker_ has joined #openstack-ironic | 09:38 | |
openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Clean up libvirt unit tests https://review.openstack.org/586029 | 09:38 |
*** stendulker has quit IRC | 09:41 | |
openstackgerrit | Aija Jaunteva proposed openstack/sushy master: Add DictionaryField class to resource base https://review.openstack.org/586031 | 09:41 |
etingof | this job seems to occasionally fail elsewhere as well, but what's cause? -- http://logs.openstack.org/25/449425/8/check/ironic-tempest-dsvm-ipa-wholedisk-agent_ipmitool-tinyipa-multinode/bb55c31/job-output.txt.gz#_2018-07-24_17_08_04_499657 | 09:43 |
dtantsur | etingof: it's pretty unstable overall, but I haven't looked into why it fails now | 09:47 |
etingof | well, it's not just "now" ;) | 09:48 |
*** d0ugal has quit IRC | 09:52 | |
dtantsur | well, for example some of test nodes are slower than the others. and this test is particularly complex and resource-sensitive | 09:53 |
etingof | so should I recheck or better wait while the situation improves? | 09:55 |
dtantsur | etingof: recheck | 09:55 |
dtantsur | the slow nodes will never improve. and other potential problems won't improve without someone looking into them (you can be this someone ;) | 09:56 |
openstackgerrit | Madhuri Kumari proposed openstack/ironic master: Implement basic interfaces for GraphicalConsole Interface https://review.openstack.org/547356 | 09:57 |
openstackgerrit | yolanda.robla proposed openstack/ironic master: Add documentation for BIOS settings https://review.openstack.org/585322 | 09:58 |
*** namnh has joined #openstack-ironic | 10:06 | |
*** namnh has quit IRC | 10:06 | |
*** namnh has joined #openstack-ironic | 10:07 | |
*** namnh_ has joined #openstack-ironic | 10:07 | |
openstackgerrit | Merged openstack/ironic-python-agent stable/queens: fix uname mock arguments for get-pip.py https://review.openstack.org/585811 | 10:09 |
openstackgerrit | Merged openstack/ironic-python-agent stable/pike: fix uname mock arguments for get-pip.py https://review.openstack.org/585810 | 10:09 |
openstackgerrit | Merged openstack/ironic-python-agent stable/ocata: fix uname mock arguments for get-pip.py https://review.openstack.org/585809 | 10:09 |
openstackgerrit | Madhuri Kumari proposed openstack/ironic master: Implement basic interfaces for GraphicalConsole Interface https://review.openstack.org/547356 | 10:10 |
*** namnh has quit IRC | 10:11 | |
*** stendulker has joined #openstack-ironic | 10:16 | |
*** stendulker_ has quit IRC | 10:19 | |
openstackgerrit | Madhuri Kumari proposed openstack/ironic master: Implement basic interfaces for GraphicalConsole Interface https://review.openstack.org/547356 | 10:22 |
*** racedo has quit IRC | 10:24 | |
*** namnh_ has quit IRC | 10:26 | |
*** namnh has joined #openstack-ironic | 10:29 | |
*** namnh has quit IRC | 10:30 | |
*** d0ugal has joined #openstack-ironic | 10:32 | |
*** d0ugal has quit IRC | 10:34 | |
*** phuongnh has quit IRC | 10:35 | |
*** namnh has joined #openstack-ironic | 10:37 | |
*** liuzz has quit IRC | 10:40 | |
*** stendulker_ has joined #openstack-ironic | 10:44 | |
*** stendulker has quit IRC | 10:47 | |
openstackgerrit | yolanda.robla proposed openstack/ironic master: WIP: Add automated_clean field to the node object and API https://review.openstack.org/585991 | 10:53 |
yolanda | dtantsur, i have a question on this error | 10:55 |
yolanda | http://logs.openstack.org/22/585322/7/check/build-openstack-api-ref/e4382e4/job-output.txt.gz#_2018-07-26_10_43_49_972283 | 10:55 |
yolanda | for the response of this call, we just return a dictionary, and the key is variable | 10:55 |
yolanda | i tried to reflect that, but i hit this error | 10:55 |
yolanda | how shall we reflect it? | 10:55 |
*** dougsz has quit IRC | 10:58 | |
*** linkmark has joined #openstack-ironic | 10:58 | |
*** dougsz has joined #openstack-ironic | 11:01 | |
*** stendulker has joined #openstack-ironic | 11:03 | |
*** rcernin has quit IRC | 11:04 | |
*** dougsz has quit IRC | 11:06 | |
*** stendulker_ has quit IRC | 11:06 | |
*** namnh has quit IRC | 11:07 | |
*** dougsz has joined #openstack-ironic | 11:08 | |
*** namnh has joined #openstack-ironic | 11:09 | |
*** namnh has quit IRC | 11:13 | |
dtantsur | yolanda: not sure. maybe just in textual form in the description? | 11:14 |
yolanda | so maybe i could just set <key>: d_bios_setting there? | 11:20 |
yolanda | i need to pass a name: value there | 11:20 |
dtantsur | yeah, maybe | 11:20 |
dtantsur | maybe we should not have done it this way.. | 11:21 |
yolanda | looks as it's not following the standards | 11:21 |
yolanda | that's the only output i could find with this format | 11:21 |
*** stendulker has quit IRC | 11:24 | |
openstackgerrit | yolanda.robla proposed openstack/ironic master: Add documentation for BIOS settings https://review.openstack.org/585322 | 11:26 |
*** liuzz has joined #openstack-ironic | 11:26 | |
openstackgerrit | Hamdy Khader proposed openstack/ironic master: Add support for smartNICs https://review.openstack.org/583573 | 11:26 |
TheJulia | Good morning | 11:35 |
*** zhangfei has quit IRC | 11:35 | |
openstackgerrit | yolanda.robla proposed openstack/ironic master: WIP: Add automated_clean field to the node object and API https://review.openstack.org/585991 | 11:36 |
*** skazi has joined #openstack-ironic | 11:43 | |
dtantsur | morning TheJulia | 11:51 |
TheJulia | good morning dtantsur | 11:55 |
dtantsur | so, I guess our client releases went fine, modulo inspector-client's setup.cfg? | 11:56 |
TheJulia | yup | 11:57 |
dtantsur | sweet! | 11:57 |
TheJulia | I need to take a look at https://review.openstack.org/#/c/578959/ again this morning | 11:58 |
patchbot | patch 578959 - ironic - Change PXE logic to always link macs with UEFI | 11:59 |
TheJulia | so it is WF-1 at the moment | 11:59 |
*** MattMan has quit IRC | 12:08 | |
*** MattMan has joined #openstack-ironic | 12:09 | |
*** liuzz_ has joined #openstack-ironic | 12:09 | |
*** liuzz has quit IRC | 12:10 | |
*** hamzy has quit IRC | 12:14 | |
*** rh-jelabarre has joined #openstack-ironic | 12:18 | |
*** hamzy has joined #openstack-ironic | 12:19 | |
*** tiendc has quit IRC | 12:21 | |
yolanda | hi, i'm getting this error when adding the automated_clean... http://logs.openstack.org/95/585795/3/check/openstack-tox-py27/f547ead/job-output.txt.gz#_2018-07-26_09_21_28_784963 | 12:21 |
yolanda | seems to be some problem between tinyint/boolean, not sure what's the problem, as i defined the type as boolean in the model and on migration... | 12:22 |
*** dprince has joined #openstack-ironic | 12:24 | |
*** mjturek has joined #openstack-ironic | 12:32 | |
*** jcoufal has joined #openstack-ironic | 12:32 | |
*** jiapei has quit IRC | 12:33 | |
*** openstackgerrit has quit IRC | 12:36 | |
*** links has quit IRC | 12:46 | |
TheJulia | yolanda: have you compard it to another boolean field? | 12:49 |
TheJulia | You really can't null a boolean either, although that should imply false I guess | 12:49 |
*** namnh has joined #openstack-ironic | 12:50 | |
yolanda | mm, then it will imply it will always have a value | 12:54 |
*** namnh has quit IRC | 12:54 | |
yolanda | i'm thinking on how that should work, if we have that value set to False , and the general automated_cleanup set to True... | 12:55 |
yolanda | TheJulia, i see it quite risky | 12:56 |
yolanda | maybe we should change the datatype then? | 12:56 |
*** Sukhdev has joined #openstack-ironic | 12:56 | |
*** Sukhdev has quit IRC | 13:01 | |
*** rloo has joined #openstack-ironic | 13:03 | |
*** bdodd has quit IRC | 13:05 | |
*** namnh has joined #openstack-ironic | 13:05 | |
yolanda | for what we need to do we need to have none, true, false. I'd try to change to tinyint | 13:05 |
TheJulia | dtantsur: yeah, I need to revise https://review.openstack.org/#/c/578959/5 :\ | 13:06 |
patchbot | patch 578959 - ironic - Change PXE logic to always link macs with UEFI | 13:06 |
TheJulia | comment inline | 13:06 |
TheJulia | yolanda: You almost need three options then | 13:07 |
TheJulia | What are we really trying to achieve with this option? | 13:08 |
yolanda | TheJulia, None will mean the node takes the option from general config. True/False means it takes the option from the node, and does automated cleanup/not | 13:09 |
yolanda | so integer with some constraint, or some enum | 13:09 |
TheJulia | So a boolean option would only be False, not none | 13:09 |
yolanda | yep, that doesn't fit | 13:10 |
TheJulia | may have been better in driver_info since it is all driver code | 13:10 |
yolanda | somehow i was thinking i could have a difference between null and false | 13:10 |
TheJulia | if it stored and treated as a string and then cast... | 13:17 |
TheJulia | there could be | 13:17 |
dtantsur | TheJulia: did not quite get it, to be honest | 13:22 |
TheJulia | dtantsur: regarding? | 13:23 |
dtantsur | TheJulia: well, both regarding the UEFI patch and the problem with boolean, actually :) | 13:24 |
TheJulia | dtantsur: no 01- for grub :\ | 13:24 |
*** bdodd has joined #openstack-ironic | 13:24 | |
TheJulia | and : instead of - | 13:24 |
dtantsur | ah, I see now | 13:24 |
TheJulia | yeah :\ | 13:24 |
dtantsur | TheJulia: also, what's the problem with NULL boolean? | 13:26 |
TheJulia | re the boolean, it sounds like we want an independent dual override for cleaning that just completely disregards the conductor default setting if set. So that an operator can choose to disable or enable for that specific node and disregard the conductor wide setting. Unless we want to default it to the present automated clean value and just migrate away from the global setting.... | 13:26 |
TheJulia | NULL should == None which should == False | 13:27 |
TheJulia | You can't have three options there :\ | 13:27 |
*** mjturek has quit IRC | 13:27 | |
dtantsur | no, automated_clean = None | True | false | 13:27 |
dtantsur | I don't see problems still | 13:27 |
*** mbeierl has joined #openstack-ironic | 13:27 | |
TheJulia | dtantsur: I agree there, but it can't be a boolean then, it has to be a string | 13:27 |
TheJulia | and then cast | 13:27 |
dtantsur | why? | 13:27 |
*** mjturek has joined #openstack-ironic | 13:27 | |
TheJulia | or just a string maybe if sqlalchemy will do it for us | 13:28 |
* dtantsur is lost | 13:28 | |
*** med_ has quit IRC | 13:28 | |
TheJulia | dtantsur: blame sqlalchemy | 13:28 |
dtantsur | sure, if it does not work with NULL, we have to fix it | 13:28 |
TheJulia | I don't think it will because the existing maintenance state is a boolean | 13:29 |
TheJulia | and I think it is always default false as a result of not being set | 13:29 |
dtantsur | I don't think so | 13:29 |
* TheJulia cracks open her local db | 13:29 | |
dtantsur | what you're saying is that NULL never works. I quite doubt it, to be honest.. | 13:29 |
*** jistr is now known as jistr|mtg | 13:29 | |
TheJulia | dtantsur: in the db, it is being set as a 0/1 | 13:30 |
TheJulia | yup, tinyint(1) | 13:30 |
TheJulia | so at the DB level, I can null it out | 13:31 |
dtantsur | TheJulia: well, at least sqlite works correctly itself | 13:31 |
*** mjturek has quit IRC | 13:31 | |
TheJulia | and..... apparently requests is gone from my systme :\ | 13:32 |
*** rbartal_ has joined #openstack-ironic | 13:32 | |
TheJulia | Field `maintenance' cannot be None (HTTP 500) | 13:33 |
dtantsur | sure, it's not nullable | 13:33 |
dtantsur | but you can create a nullable field, no? | 13:33 |
dtantsur | so, it's wsme problem, not sqlalchemy? | 13:33 |
dtantsur | I think yolanda's problem is simply because sqlite does not have proper types | 13:33 |
dtantsur | so boolean is emulated as tinyint (I can reproduce it with sqlite) | 13:34 |
TheJulia | not sure, you should always be able to actually | 13:34 |
*** rbartal has quit IRC | 13:34 | |
TheJulia | yeah, that i likely what is going on then | 13:34 |
dtantsur | still, it can be NULL on database level (just checked) | 13:34 |
dtantsur | it's just NULL | 0 | 1 internally | 13:34 |
TheJulia | yeah, that is what I did | 13:34 |
yolanda | so i'm trying now creating an enum type | 13:35 |
dtantsur | yolanda: please stick with boolean | 13:35 |
yolanda | dtantsur, but then how do we avoid this problem? | 13:35 |
TheJulia | enum won't work in sqllite either | 13:35 |
TheJulia | we tried to do that for deva a long time ago and it blew up on us | 13:36 |
*** mjturek has joined #openstack-ironic | 13:36 | |
dtantsur | TheJulia: with wsme you cannot have boolean = None, but you can use 'remove' PATCH action | 13:36 |
TheJulia | dtantsur: \o/ | 13:36 |
dtantsur | TheJulia: we do have enums in ironic-inspector. I guess they're also emulated | 13:36 |
dtantsur | yolanda: just check that the column is present, not assert its type? | 13:36 |
TheJulia | Yeah, I think that is what it was, we went to actually do somehting and it all went *boom* and had to backtrack | 13:36 |
dtantsur | I mean, as TheJulia said, the type check will probably fail on enum as well. in sqlite everything is a number or text IIRC | 13:37 |
yolanda | ok let me avoid that check of type... so for mysql, it will be safe to assume that null will exist? | 13:37 |
dtantsur | yolanda: null exists everywhere, it's a part of standard | 13:37 |
dtantsur | every column that is not declared as 'not null' can be null | 13:38 |
yolanda | i mean, will not be confused, converted, etc | 13:38 |
yolanda | None with False | 13:38 |
TheJulia | dtantsur: except pkey columns ;) | 13:38 |
dtantsur | TheJulia: yeah, they're explicitly 'not null' (I hope!) | 13:38 |
dtantsur | but sqlite is... funny.. | 13:38 |
yolanda | i also could not even generate the migration using sqlalchemy | 13:39 |
yolanda | it complains that cannot upgrade the db because it cannot alter a constraint | 13:39 |
yolanda | due to some of the migrations in the history | 13:39 |
yolanda | if had to do it on a devstack where i had mysql installed | 13:39 |
dtantsur | yolanda: yep, sqlite cannot change constraints | 13:39 |
TheJulia | dtantsur: I'm not going to call the in-brain select to find the rows (instances) due to painful database memories | 13:39 |
dtantsur | heh | 13:40 |
dtantsur | yolanda: there is a trick to handle it. we have a ironic-dbsync command to create a schema without going through migrations | 13:40 |
*** r-daneel has joined #openstack-ironic | 13:40 | |
dtantsur | lemme find docs for you | 13:40 |
dtantsur | meh, I cannot find docs. but it's something like ironic-dbsync create_schema | 13:42 |
dtantsur | then you can apply your change and then generate the migration | 13:42 |
dtantsur | yolanda: ^^^ | 13:42 |
yolanda | dtantsur, TheJulia , if we have all these issues with sqlite... why do we list that as an option? | 13:42 |
dtantsur | yolanda: it's only for local testing and unit tets | 13:42 |
dtantsur | otherwise we'd have to require mysql to run unit tests | 13:42 |
*** namnh has quit IRC | 13:43 | |
yolanda | i see | 13:44 |
* dtantsur remembers ironic-inspector only supporting sqlite.. fun times. | 13:44 | |
yolanda | doh | 13:44 |
dtantsur | it was kilo, time to prune it out of my memory :) | 13:44 |
*** rook-- is now known as rook | 13:45 | |
*** markguz has joined #openstack-ironic | 13:45 | |
*** hongbin has joined #openstack-ironic | 13:52 | |
TheJulia | so I searched for ironic on the talk voting and 55 results.... | 13:53 |
TheJulia | dtantsur: heh | 13:53 |
dtantsur | yep, quite a lot of things | 13:53 |
etingof | ...out of 746 talk proposals | 13:54 |
TheJulia | I had no idea it was that many | 13:54 |
TheJulia | That is really good numbers | 13:55 |
etingof | that's what it reports on empty search string | 13:55 |
TheJulia | Interesting to see the vendor technology integration pitches again | 13:56 |
TheJulia | "tragedy of ironic and nova" :\ | 14:00 |
TheJulia | It is well written | 14:00 |
* TheJulia finally gets the humor | 14:01 | |
etingof | with bonus: "one or two things about Shakespeare" | 14:03 |
TheJulia | ++ | 14:03 |
*** arnewiebalck_ has joined #openstack-ironic | 14:06 | |
*** rnoriega has quit IRC | 14:07 | |
*** weshay_PTO has quit IRC | 14:07 | |
*** rnoriega has joined #openstack-ironic | 14:07 | |
*** weshay has joined #openstack-ironic | 14:08 | |
*** arnewiebalck_ has quit IRC | 14:14 | |
dtantsur | yep | 14:16 |
*** diga has joined #openstack-ironic | 14:16 | |
*** etingof has quit IRC | 14:20 | |
*** tssurya has quit IRC | 14:23 | |
*** arnewiebalck_ has joined #openstack-ironic | 14:25 | |
*** rpioso|afk is now known as rpioso | 14:27 | |
rpioso | Good morning :) | 14:28 |
*** rbartal_ has quit IRC | 14:28 | |
*** arnewiebalck_ has quit IRC | 14:32 | |
*** etingof has joined #openstack-ironic | 14:33 | |
*** hongbin has quit IRC | 14:46 | |
*** Sukhdev has joined #openstack-ironic | 14:50 | |
NobodyCam | Good Morning Ironic'ers | 14:52 |
rpioso | NobodyCam: Mornin' | 14:55 |
NobodyCam | morning rpioso :) | 14:56 |
*** jistr|mtg is now known as jistr | 14:56 | |
*** vmud213 has joined #openstack-ironic | 15:04 | |
*** alexpilotti has quit IRC | 15:14 | |
*** Sukhdev has quit IRC | 15:17 | |
*** vmud213 has quit IRC | 15:22 | |
*** mjturek has quit IRC | 15:33 | |
*** pcaruana has quit IRC | 15:38 | |
*** skazi has quit IRC | 15:39 | |
*** hamzy has quit IRC | 15:41 | |
*** hamzy has joined #openstack-ironic | 15:41 | |
*** livelace2 has quit IRC | 15:44 | |
ajya | gerrit-irc hook seems to be down, is this discussed somewhere? | 15:44 |
ajya | also for a new code review Zuul was not added as a reviewer, I added manually. Should I add 'recheck' to trigger it? | 15:45 |
dtantsur | ajya: it's fine, it adds itself when it first votes | 15:46 |
*** hongbin has joined #openstack-ironic | 15:49 | |
*** mjturek has joined #openstack-ironic | 15:53 | |
mgoddard | anyone have time to look at https://review.openstack.org/#/c/530537/? It has a +2 already | 15:55 |
patchbot | patch 530537 - networking-generic-switch - Add a netmiko device driver for Juniper switches | 15:55 |
mgoddard | (and a +1 from someone from Juniper) | 15:55 |
TheJulia | ohh | 15:56 |
TheJulia | hmm looks like vdrok is idle and he would be ideal I think | 15:57 |
*** gyee has joined #openstack-ironic | 15:59 | |
clarkb | ajya: yes, in #openstack-infra. Long story short freenode spam killer bot decided it was spamming and killed it. My hunch is that we may just leave it in that state until the spam attack goes away and we can rerun it without getting caught by spam killers | 15:59 |
dtantsur | oh, that's funny. we're spamming with out patches :D | 16:02 |
dtantsur | s/out/our/ | 16:02 |
*** e0ne has quit IRC | 16:02 | |
*** alexpilotti has joined #openstack-ironic | 16:10 | |
*** AlexeyAbashkin has quit IRC | 16:10 | |
ajya | clarkb, thanks | 16:10 |
ajya | being too productive :) | 16:10 |
*** alexpilotti has quit IRC | 16:14 | |
*** skazi has joined #openstack-ironic | 16:16 | |
*** dprince has quit IRC | 16:16 | |
*** verdurin has quit IRC | 16:21 | |
*** verdurin has joined #openstack-ironic | 16:24 | |
* TheJulia bangs her head against the pxe interface | 16:27 | |
*** andymccr_ has joined #openstack-ironic | 16:29 | |
*** ajya has quit IRC | 16:29 | |
* etingof slides a beer stein full of cool sparkling beer to TheJulia | 16:30 | |
TheJulia | mmmmmm | 16:30 |
TheJulia | Beer sounds really good at the moment | 16:30 |
etingof | and it mixes with coffee \o/ | 16:30 |
TheJulia | not coffee beer? | 16:31 |
etingof | no, the mug I slide is just pure Czech beer | 16:32 |
* TheJulia feels sad suddenly | 16:32 | |
etingof | but next time you will have coffee stout \o/ | 16:32 |
*** dprince has joined #openstack-ironic | 16:32 | |
TheJulia | I do need to go to the post office.... | 16:32 |
TheJulia | Hmmmm | 16:32 |
* etingof almost typed `stdout`, time to wrap up | 16:32 | |
TheJulia | lol | 16:33 |
TheJulia | goodnight etingof | 16:33 |
*** baha has joined #openstack-ironic | 16:33 | |
*** andymccr_ has quit IRC | 16:33 | |
etingof | o/ | 16:33 |
*** andymccr_ has joined #openstack-ironic | 16:33 | |
*** andymccr_ has quit IRC | 16:34 | |
dtantsur | mmm, coffee stout | 16:34 |
dtantsur | we have good ones in brewdog | 16:34 |
*** andymccr_ has joined #openstack-ironic | 16:34 | |
* dtantsur is pondering spending tomorrow working from there | 16:34 | |
TheJulia | dtantsur: would you be up for looking at mgoddard's n-g-s patch? | 16:34 |
dtantsur | TheJulia: not today, sorry. I'm boiling :( | 16:37 |
*** diga has quit IRC | 16:37 | |
TheJulia | I think tomorrow is fine in all honesty | 16:37 |
dtantsur | actually I was about to say g'night | 16:37 |
dtantsur | yeah, tomorrow quite probably | 16:37 |
*** dtantsur is now known as dtantsur|afk | 16:38 | |
dtantsur|afk | c u | 16:38 |
TheJulia | I'm banging my head against this pxe fix righ tnow thinking it is time to take a beer break | 16:38 |
*** andymccr_ has quit IRC | 16:38 | |
dtantsur|afk | ++ go ahead | 16:38 |
* TheJulia senses she is going to be spending lots of time with this file open over the next few months | 16:38 | |
*** andymccr_ has joined #openstack-ironic | 16:38 | |
*** tesseract has quit IRC | 16:39 | |
*** andymccr_ has quit IRC | 16:39 | |
mgoddard | if you have time tomorrow dtantsur, would be appreciated | 16:41 |
mgoddard | also boiling here | 16:41 |
*** med_ has joined #openstack-ironic | 16:42 | |
*** andymccr_ has joined #openstack-ironic | 16:44 | |
*** andymccr_ has quit IRC | 16:53 | |
*** andymccr_ has joined #openstack-ironic | 16:54 | |
*** andymccr_ has quit IRC | 16:55 | |
*** andymccr_ has joined #openstack-ironic | 16:55 | |
*** d0ugal has joined #openstack-ironic | 16:55 | |
*** d0ugal has quit IRC | 16:55 | |
*** d0ugal has joined #openstack-ironic | 16:55 | |
TheJulia | and the moment you realize the code is right, it is the test that is wrong | 16:57 |
*** derekh has quit IRC | 17:00 | |
*** markguz has quit IRC | 17:03 | |
*** mjturek has quit IRC | 17:05 | |
*** openstackgerrit has joined #openstack-ironic | 17:05 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Change PXE logic to always link macs with UEFI https://review.openstack.org/578959 | 17:05 |
*** mjturek has joined #openstack-ironic | 17:07 | |
openstackgerrit | Merged openstack/sushy-tools master: Add unittests for OpenStack nova driver https://review.openstack.org/568636 | 17:08 |
openstackgerrit | Merged openstack/sushy-tools master: Reorder unittest assertions https://review.openstack.org/586017 | 17:08 |
*** r-daneel_ has joined #openstack-ironic | 17:10 | |
*** r-daneel has quit IRC | 17:10 | |
*** r-daneel_ is now known as r-daneel | 17:10 | |
*** Sukhdev has joined #openstack-ironic | 17:12 | |
*** r-daneel has quit IRC | 17:14 | |
*** dougsz has quit IRC | 17:16 | |
*** serlex has quit IRC | 17:16 | |
*** e0ne has joined #openstack-ironic | 17:20 | |
*** d0ugal has quit IRC | 17:21 | |
*** r-daneel has joined #openstack-ironic | 17:22 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Adds ramdisk deploy driver https://review.openstack.org/568940 | 17:22 |
*** mjturek has quit IRC | 17:24 | |
*** jtomasek has quit IRC | 17:29 | |
*** trown is now known as trown|lunch | 17:29 | |
*** med_ has quit IRC | 17:29 | |
*** baha has quit IRC | 17:33 | |
*** mjturek has joined #openstack-ironic | 17:33 | |
*** mjturek has quit IRC | 17:33 | |
*** e0ne has quit IRC | 17:37 | |
TheJulia | mgoddard: thanks for the +2 | 17:42 |
mgoddard | TheJulia: np, thanks for the driver | 17:43 |
TheJulia | mgoddard: also, https://review.openstack.org/#/c/585818/ is a quick review | 17:46 |
patchbot | patch 585818 - ironic - Deprecate xclarity hardware type | 17:46 |
* TheJulia runs into town to go to the post office | 17:47 | |
mgoddard | done | 17:48 |
TheJulia | thanks | 17:48 |
*** baoli has joined #openstack-ironic | 17:52 | |
*** jaganathan has quit IRC | 18:16 | |
*** e0ne has joined #openstack-ironic | 18:22 | |
*** baoli has quit IRC | 18:41 | |
*** skazi has quit IRC | 18:42 | |
*** mgoddard has quit IRC | 18:45 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Add decoratores to ramdisk deploy interface https://review.openstack.org/586327 | 18:55 |
*** hamzy has quit IRC | 19:02 | |
*** hamzy_ has joined #openstack-ironic | 19:02 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Ramdisk deploy driver doc https://review.openstack.org/585486 | 19:03 |
*** mjturek has joined #openstack-ironic | 19:07 | |
TheJulia | o/ | 19:09 |
*** med_ has joined #openstack-ironic | 19:10 | |
*** med_ has quit IRC | 19:10 | |
*** med_ has joined #openstack-ironic | 19:10 | |
*** baha has joined #openstack-ironic | 19:10 | |
*** ianychoi_ has joined #openstack-ironic | 19:15 | |
*** pcaruana has joined #openstack-ironic | 19:17 | |
*** ianychoi has quit IRC | 19:18 | |
*** r-daneel has quit IRC | 19:21 | |
*** r-daneel has joined #openstack-ironic | 19:22 | |
edleafe | dtantsur|afk: TheJulia: Figured out my issue. dtantsur|afk was correct; the Ironic conductor wasn't running. Digging into the logs I saw an exception which was the clue. I've summarized it here: http://paste.openstack.org/show/726710/ | 19:28 |
TheJulia | oh, I should have thought about that | 19:31 |
TheJulia | Good to know :) | 19:31 |
*** trown|lunch is now known as trown | 19:43 | |
*** pcaruana has quit IRC | 19:45 | |
openstackgerrit | yolanda.robla proposed openstack/ironic master: Add automated_clean field https://review.openstack.org/585795 | 19:46 |
openstackgerrit | yolanda.robla proposed openstack/ironic master: Add automated_clean field https://review.openstack.org/585795 | 19:51 |
openstackgerrit | yolanda.robla proposed openstack/ironic master: Add automated_clean field https://review.openstack.org/585795 | 19:52 |
*** dprince has quit IRC | 20:03 | |
*** alexpilotti has joined #openstack-ironic | 20:08 | |
*** alexpilo_ has joined #openstack-ironic | 20:12 | |
*** e0ne has quit IRC | 20:13 | |
openstackgerrit | Merged openstack/ironic master: Deprecate xclarity hardware type https://review.openstack.org/585818 | 20:14 |
openstackgerrit | Merged openstack/ironic master: Documentation update of iLO BIOS settings https://review.openstack.org/585202 | 20:14 |
*** alexpilotti has quit IRC | 20:17 | |
*** med_ has quit IRC | 20:18 | |
*** dougsz has joined #openstack-ironic | 20:20 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Prepare for Rocky release https://review.openstack.org/586342 | 20:22 |
*** med_ has joined #openstack-ironic | 20:22 | |
*** med_ has quit IRC | 20:22 | |
*** med_ has joined #openstack-ironic | 20:22 | |
*** alexpilo_ has quit IRC | 20:45 | |
*** alexpilotti has joined #openstack-ironic | 20:46 | |
*** dougsz has quit IRC | 20:47 | |
*** rh-jelabarre has quit IRC | 20:56 | |
*** r-daneel_ has joined #openstack-ironic | 20:59 | |
*** e0ne has joined #openstack-ironic | 20:59 | |
*** alexpilotti has quit IRC | 20:59 | |
*** e0ne has quit IRC | 21:00 | |
*** r-daneel has quit IRC | 21:00 | |
*** r-daneel_ is now known as r-daneel | 21:00 | |
*** trown is now known as trown|outtypewww | 21:03 | |
*** jcoufal has quit IRC | 21:15 | |
*** r-daneel has quit IRC | 21:17 | |
*** markguz has joined #openstack-ironic | 21:30 | |
*** skazi has joined #openstack-ironic | 21:31 | |
TheJulia | Speaker voting closes tonight! https://www.openstack.org/summit/berlin-2018/vote-for-speakers#/ | 21:31 |
*** mjturek has quit IRC | 21:33 | |
*** markguz has quit IRC | 21:34 | |
*** med_ has quit IRC | 21:48 | |
*** markguz has joined #openstack-ironic | 21:51 | |
*** skazi has quit IRC | 21:52 | |
*** baha has quit IRC | 21:54 | |
*** Sukhdev has quit IRC | 22:08 | |
*** rcernin has joined #openstack-ironic | 22:19 | |
*** gkadam has quit IRC | 22:30 | |
*** alexpilotti has joined #openstack-ironic | 22:36 | |
*** alexpilo_ has joined #openstack-ironic | 22:41 | |
*** alexpilotti has quit IRC | 22:44 | |
*** namnh has joined #openstack-ironic | 22:46 | |
*** rpioso is now known as rpioso|afk | 22:48 | |
*** bdodd has quit IRC | 22:56 | |
*** alexpilotti has joined #openstack-ironic | 22:57 | |
*** bdodd has joined #openstack-ironic | 22:57 | |
*** hongbin has quit IRC | 23:00 | |
*** lbragstad_ has joined #openstack-ironic | 23:01 | |
*** alexpilo_ has quit IRC | 23:01 | |
*** lbragstad has quit IRC | 23:01 | |
*** med_ has joined #openstack-ironic | 23:19 | |
*** med_ has quit IRC | 23:19 | |
*** med_ has joined #openstack-ironic | 23:19 | |
*** namnh has quit IRC | 23:41 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!