*** purp has quit IRC | 00:05 | |
*** purp has joined #openstack-ironic | 00:06 | |
*** ParsectiX has quit IRC | 00:06 | |
*** ParsectiX has joined #openstack-ironic | 00:07 | |
*** spandhe has quit IRC | 00:22 | |
*** naohirot has joined #openstack-ironic | 00:26 | |
*** achanda has joined #openstack-ironic | 00:46 | |
*** achanda has quit IRC | 00:51 | |
*** wuhg has joined #openstack-ironic | 01:34 | |
*** jerryz_ has joined #openstack-ironic | 01:37 | |
*** chenglch has joined #openstack-ironic | 02:00 | |
*** chenglch has quit IRC | 02:10 | |
*** gabrielbezerra has quit IRC | 02:21 | |
*** purp has quit IRC | 02:21 | |
*** purp has joined #openstack-ironic | 02:21 | |
*** gabrielbezerra has joined #openstack-ironic | 02:24 | |
*** yuanying has quit IRC | 02:46 | |
openstackgerrit | Yuiko Takada proposed stackforge/ironic-discoverd: Add scripts to manage translations https://review.openstack.org/158981 | 02:50 |
---|---|---|
* mrda wonders whether a Intel NUC5i5MYHE (which has AMT) would be a good Ironic test network node... | 03:06 | |
*** yuanying has joined #openstack-ironic | 03:07 | |
*** chenglch has joined #openstack-ironic | 03:07 | |
*** chenglch has quit IRC | 03:13 | |
*** yuanying has quit IRC | 03:19 | |
*** yuanying has joined #openstack-ironic | 03:25 | |
*** anderbubble has joined #openstack-ironic | 03:28 | |
*** yuanying has quit IRC | 03:35 | |
*** yuanying has joined #openstack-ironic | 03:37 | |
openstackgerrit | Yuiko Takada proposed stackforge/ironic-discoverd: Verify input uuid https://review.openstack.org/159732 | 03:44 |
openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/158750 | 03:45 |
*** achanda has joined #openstack-ironic | 03:49 | |
*** achanda has quit IRC | 03:54 | |
*** david-lyle_afk has quit IRC | 04:02 | |
*** Marga_ has quit IRC | 04:02 | |
*** Marga_ has joined #openstack-ironic | 04:04 | |
openstackgerrit | Yuiko Takada proposed stackforge/ironic-discoverd: Consider dropping check on power state https://review.openstack.org/160223 | 04:10 |
openstackgerrit | Yuiko Takada proposed stackforge/ironic-discoverd: Verify input uuid https://review.openstack.org/159732 | 04:17 |
*** chenglch has joined #openstack-ironic | 04:37 | |
*** coolsvap_ is now known as coolsvap | 04:43 | |
*** ParsectiX has quit IRC | 04:44 | |
*** ParsectiX has joined #openstack-ironic | 04:45 | |
*** achanda has joined #openstack-ironic | 04:49 | |
openstackgerrit | Syed Ismail Faizan Barmawer proposed openstack/ironic-python-agent: Add support for partition images in agent driver https://review.openstack.org/160224 | 04:54 |
*** pensu has joined #openstack-ironic | 05:04 | |
*** pensu has left #openstack-ironic | 05:05 | |
*** chenglch has quit IRC | 05:06 | |
*** yuanying_ has joined #openstack-ironic | 05:12 | |
*** yuanying has quit IRC | 05:15 | |
*** chenglch|2 has joined #openstack-ironic | 05:31 | |
*** rameshg87 has joined #openstack-ironic | 05:31 | |
rameshg87 | good morning ironic | 05:33 |
openstackgerrit | Shivanand Tendulker proposed stackforge/proliantutils: Support CDROM in get/set persistent boot methods https://review.openstack.org/159799 | 05:34 |
*** spandhe has joined #openstack-ironic | 05:34 | |
*** achanda has quit IRC | 06:03 | |
*** syrus has joined #openstack-ironic | 06:11 | |
naohirot | rameshg87: good morning | 06:15 |
naohirot | rameshg87: Can I ask a question regarding " pass this information as a kernel command-line argument"? | 06:16 |
naohirot | rameshg87: I can put some environment variables to pre-process init-func, but I'm not sure about the kernel command line argument. | 06:19 |
*** achanda has joined #openstack-ironic | 06:27 | |
rameshg87 | naohirot, hey | 06:28 |
rameshg87 | naohirot, 1 min let me check | 06:28 |
naohirot | rameshg87: Hi, yes please. | 06:29 |
*** ukalifon1 has joined #openstack-ironic | 06:31 | |
rameshg87 | naohirot, i meant you can first check the kernel command line arguments for the device name | 06:32 |
rameshg87 | naohirot, sorry | 06:32 |
*** lintan has quit IRC | 06:32 | |
rameshg87 | naohirot, brb | 06:32 |
naohirot | rameshg87: I'll submit patch #2, and review it again if you got some time. | 06:33 |
rameshg87 | naohirot, back again | 06:33 |
*** chenglch has joined #openstack-ironic | 06:33 | |
rameshg87 | naohirot, i meant something like this: https://github.com/openstack/diskimage-builder/blob/master/elements/ramdisk-base/init.d/20-init-variables#L7 | 06:33 |
*** chenglch|2 has quit IRC | 06:35 | |
naohirot | rameshg87: First of all, I'd like to know which way are you talking about either pre-processing init or post-processing init-func | 06:36 |
rameshg87 | naohirot, something like this i meant http://paste.openstack.org/show/184710/ | 06:37 |
rameshg87 | naohirot, sorry L3 s/DEVICE_MODEL/SYSFS_VMEDIA_DEVICE_MODEL | 06:37 |
rameshg87 | naohirot, can something like this be done ? | 06:37 |
rameshg87 | naohirot, but the problem is while deploying you will have to pass this inside the iso | 06:38 |
naohirot | rameshg87: Is it okay to replace when ramdisk-image-create is invoked? This way I called pre-processing. | 06:39 |
rameshg87 | naohirot, replace what ? | 06:39 |
naohirot | rameshg87: I've already created patch #2 | 06:40 |
rameshg87 | naohirot, let me check | 06:40 |
naohirot | rameshg87: let me submit it, and the discuss, Okay? | 06:40 |
rameshg87 | naohirot, yeah sure .. | 06:40 |
naohirot | rameshg87: just moment :) | 06:40 |
rameshg87 | naohirot, sure .. | 06:40 |
*** rameshg87 is now known as rameshg87-brb | 06:41 | |
naohirot | rameshg87-brb: I submitted. | 06:42 |
*** rameshg87-brb is now known as rameshg87 | 06:52 | |
rameshg87 | naohirot, ack, will take a look | 06:52 |
naohirot | rameshg87: thanks | 06:52 |
*** jay-lau-513 has joined #openstack-ironic | 06:52 | |
*** jerryz_ has quit IRC | 07:10 | |
*** ParsectiX has quit IRC | 07:12 | |
*** ParsectiX has joined #openstack-ironic | 07:13 | |
*** achanda has quit IRC | 07:13 | |
*** ParsectiX has quit IRC | 07:13 | |
*** jerryz_ has joined #openstack-ironic | 07:13 | |
*** ParsectiX has joined #openstack-ironic | 07:14 | |
*** achanda has joined #openstack-ironic | 07:15 | |
openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add localboot support for uefi boot mode https://review.openstack.org/159855 | 07:17 |
openstackgerrit | Naohiro Tamura proposed openstack/ironic-specs: iRMC Virtual Media Deploy Driver for Ironic https://review.openstack.org/134865 | 07:19 |
*** ifarkas has joined #openstack-ironic | 07:23 | |
*** achanda has quit IRC | 07:23 | |
*** achanda has joined #openstack-ironic | 07:29 | |
*** jcoufal has joined #openstack-ironic | 07:29 | |
*** rameshg87 is now known as rameshg87-lunch | 07:34 | |
*** anderbubble has quit IRC | 07:36 | |
openstackgerrit | Shivanand Tendulker proposed stackforge/proliantutils: Support CDROM in get/set persistent boot methods https://review.openstack.org/159799 | 07:42 |
*** yuriyz has quit IRC | 07:49 | |
*** achanda has quit IRC | 07:52 | |
*** achanda has joined #openstack-ironic | 07:57 | |
*** yuriyz has joined #openstack-ironic | 08:01 | |
*** chlong has quit IRC | 08:02 | |
*** athomas has joined #openstack-ironic | 08:06 | |
*** rameshg87-lunch is now known as rameshg87 | 08:09 | |
*** achanda has quit IRC | 08:16 | |
*** Marga_ has quit IRC | 08:16 | |
*** Guest78669 is now known as d0ugal | 08:20 | |
*** d0ugal has joined #openstack-ironic | 08:21 | |
*** dtantsur|afk is now known as dtantsur | 08:26 | |
dtantsur | Good Monday, Ironic | 08:26 |
dtantsur | :) | 08:26 |
*** habuka036 has joined #openstack-ironic | 08:45 | |
*** dlpartain has joined #openstack-ironic | 08:47 | |
*** dlpartain has left #openstack-ironic | 08:48 | |
*** viktors|afk has quit IRC | 08:50 | |
rameshg87 | dtantsur, o/ | 08:53 |
dtantsur | o/ | 08:53 |
*** chenglch has quit IRC | 08:56 | |
*** chenglch has joined #openstack-ironic | 08:56 | |
openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add localboot support for uefi boot mode https://review.openstack.org/159855 | 08:57 |
*** romcheg has joined #openstack-ironic | 08:59 | |
rameshg87 | dtantsur, why did we remove the check for ironic.conf.sample from our gate ? | 09:03 |
rameshg87 | dtantsur, any idea ? | 09:04 |
dtantsur | yep | 09:04 |
dtantsur | rameshg87, because we have options from other projects | 09:04 |
*** mgoddard has joined #openstack-ironic | 09:04 | |
rameshg87 | oh okay | 09:04 |
dtantsur | rameshg87, and every time they changed something, it broke our gate | 09:04 |
rameshg87 | dtantsur, but now there is no way than human inspection to find it out :( | 09:05 |
dtantsur | also true. there was a heated discussion on this, but in the end, not breaking gate pretty often was a good reason | 09:05 |
openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add localboot support for uefi boot mode https://review.openstack.org/159855 | 09:05 |
rameshg87 | okay | 09:06 |
openstackgerrit | Merged stackforge/ironic-discoverd: Verify input uuid https://review.openstack.org/159732 | 09:09 |
*** MattMan has joined #openstack-ironic | 09:10 | |
*** mgoddard has quit IRC | 09:10 | |
*** mgoddard has joined #openstack-ironic | 09:11 | |
*** jistr has joined #openstack-ironic | 09:12 | |
*** mgoddard has quit IRC | 09:18 | |
*** mgoddard has joined #openstack-ironic | 09:21 | |
*** pshige has joined #openstack-ironic | 09:29 | |
*** chenglch has quit IRC | 09:30 | |
*** chenglch has joined #openstack-ironic | 09:30 | |
*** jcoufal_ has joined #openstack-ironic | 09:33 | |
*** jcoufal has quit IRC | 09:36 | |
*** lazy_prince has quit IRC | 09:47 | |
*** killer_prince has joined #openstack-ironic | 09:52 | |
*** killer_prince is now known as lazy_prince | 09:52 | |
*** igordcard has joined #openstack-ironic | 09:57 | |
rameshg87 | dtantsur, can you please have a look at raid changes when you have some time : https://review.openstack.org/#/c/155230/ | 09:58 |
dtantsur | yeah sure (not now though) | 09:58 |
rameshg87 | okay | 09:59 |
openstackgerrit | Shivanand Tendulker proposed stackforge/proliantutils: Support CDROM in get/set persistent boot methods https://review.openstack.org/159799 | 10:01 |
*** romcheg has quit IRC | 10:12 | |
*** jay-lau-513 has quit IRC | 10:13 | |
*** romcheg has joined #openstack-ironic | 10:19 | |
*** pas-ha has joined #openstack-ironic | 10:23 | |
*** dtantsur is now known as dtantsur|bbl | 10:29 | |
openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: HPSSA: Add functional tests for create https://review.openstack.org/158122 | 10:29 |
*** romcheg1 has joined #openstack-ironic | 10:31 | |
*** romcheg has quit IRC | 10:31 | |
*** saripurigopi has joined #openstack-ironic | 10:34 | |
*** bauwser is now known as bauzas | 10:34 | |
saripurigopi | rameshg87, Thanks for the review comments. | 10:38 |
rameshg87 | saripurigopi, wc :) | 10:38 |
saripurigopi | its makes more sense to have python-ucsclient wrapper. | 10:38 |
saripurigopi | rameshg87, Is the review submission and approval process is same ? | 10:39 |
rameshg87 | saripurigopi, which review submission do you mean ? | 10:40 |
openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: HPSSA: Add support for nested RAID levels https://review.openstack.org/158994 | 10:40 |
saripurigopi | https://review.openstack.org/#/c/159734/ | 10:40 |
rameshg87 | saripurigopi, i just thought code dealing with cisco hardware in helper.py can be moved to a separate module python-ucsclient | 10:41 |
rameshg87 | saripurigopi, that can be maintained in stackforge | 10:42 |
rameshg87 | saripurigopi, http://ci.openstack.org/stackforge.html | 10:42 |
rameshg87 | saripurigopi, for stackforge you will need to have python-ucsclient ready hosted in your own github | 10:42 |
rameshg87 | saripurigopi, that will somewhere http://github.com/xxxx/python-ucsclient | 10:43 |
rameshg87 | saripurigopi, with this and the stuffs mentioned in http://docs.openstack.org/infra/manual/creators.html | 10:43 |
rameshg87 | saripurigopi, you can raise a request to get it added to stackforge | 10:43 |
rameshg87 | saripurigopi, after adding to stackforge it will be in http://github.com/stackforge/python-ucsclient | 10:43 |
saripurigopi | rameshg87, okay, will go through the wiki page and get back on that. | 10:44 |
rameshg87 | saripurigopi, no direct pushing into git repository will be disabled and you can push the code only through gerrit | 10:44 |
*** pelix has joined #openstack-ironic | 10:44 | |
saripurigopi | rameshg87, okay. same set of ironic-dev people review this as well? | 10:45 |
rameshg87 | saripurigopi, no this module will be under your team's control | 10:45 |
rameshg87 | saripurigopi, python-ucsclient will be owned by you | 10:45 |
saripurigopi | rameshg87, okay. Let me do this ASAP. | 10:46 |
rameshg87 | saripurigopi, great | 10:46 |
*** erwan_taf has joined #openstack-ironic | 10:48 | |
*** EmilienM is now known as EmilienM|afk | 10:54 | |
*** achanda has joined #openstack-ironic | 11:17 | |
openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: HPSSA: create and get should return physical disks info https://review.openstack.org/160307 | 11:21 |
*** achanda has quit IRC | 11:21 | |
*** jerryz_ has quit IRC | 11:22 | |
*** rameshg87 has quit IRC | 11:22 | |
*** yuanying_ has quit IRC | 11:22 | |
*** romcheg has joined #openstack-ironic | 11:32 | |
*** romcheg1 has quit IRC | 11:34 | |
*** jerryz_ has joined #openstack-ironic | 11:38 | |
*** erwan_taf has quit IRC | 11:38 | |
*** Haomeng has joined #openstack-ironic | 11:38 | |
*** Shrews has quit IRC | 11:49 | |
*** Shrews has joined #openstack-ironic | 11:50 | |
*** erwan_taf has joined #openstack-ironic | 11:53 | |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Do not save auth token on TFTP serfer in PXE driver https://review.openstack.org/159819 | 12:00 |
*** igordcard has quit IRC | 12:02 | |
*** coolsvap is now known as coolsvap_ | 12:06 | |
*** coolsvap_ is now known as coolsvap | 12:15 | |
*** derekh has joined #openstack-ironic | 12:18 | |
openstackgerrit | Sirushti Murugesan proposed openstack/ironic: Adds support for deploying whole disk images https://review.openstack.org/150142 | 12:27 |
*** igordcard has joined #openstack-ironic | 12:29 | |
*** lintan has joined #openstack-ironic | 12:42 | |
*** ndipanov has joined #openstack-ironic | 12:49 | |
*** BadCub has joined #openstack-ironic | 13:06 | |
BadCub | Morning Ironic | 13:08 |
*** rameshg87 has joined #openstack-ironic | 13:24 | |
openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic: Do not save auth token on TFTP serfer in PXE driver https://review.openstack.org/159819 | 13:32 |
*** saripurigopi has quit IRC | 13:34 | |
jroll | morning all :) | 13:44 |
*** chenglch has quit IRC | 13:44 | |
*** jcoufal_ has quit IRC | 13:48 | |
*** jcoufal has joined #openstack-ironic | 13:49 | |
openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: HPSSA: create and get should return physical disks info https://review.openstack.org/160307 | 13:52 |
*** thelexx is now known as lxsli | 13:53 | |
*** erwan_taf has quit IRC | 13:54 | |
*** dtantsur|bbl is now known as dtantsur | 13:59 | |
dtantsur | jroll, morning | 13:59 |
jroll | \o :) | 13:59 |
*** ParsectiX has quit IRC | 14:02 | |
*** ParsectiX has joined #openstack-ironic | 14:02 | |
*** saripurigopi has joined #openstack-ironic | 14:06 | |
*** jjohnson2 has joined #openstack-ironic | 14:07 | |
*** erwan_taf has joined #openstack-ironic | 14:12 | |
pshige | 2315 JST :) | 14:15 |
*** saripurigopi has quit IRC | 14:19 | |
*** mjturek1 has joined #openstack-ironic | 14:20 | |
*** ParsectiX has quit IRC | 14:22 | |
naohirot | jroll: good morning | 14:38 |
jroll | hi naohirot :) | 14:38 |
naohirot | jroll: when I created ipa-coreos.iso, I got error at here https://github.com/openstack/ironic-python-agent/blob/master/imagebuild/coreos/coreos-oem-inject.py#L137 | 14:39 |
naohirot | jroll: is this really necessary check? | 14:39 |
naohirot | jroll: I just commented out and created the iso. | 14:39 |
*** rloo has joined #openstack-ironic | 14:39 | |
jroll | naohirot: it's necessary if you want to make sure someone malicious isn't serving you their own special version of coreos :) | 14:40 |
jroll | naohirot: what's the error? | 14:40 |
naohirot | jroll: I know it's necessary to detect tampering or alteration. | 14:41 |
naohirot | jroll: my question is that if I'm sure there is no tampering, is it necessary? let me check what error I got. | 14:42 |
jroll | naohirot: how do you know there is no tampering without doing that check? | 14:42 |
jroll | the hash digests for coreos releases are signed, you need to validate that signature to know that the hashes haven't been tampered with | 14:43 |
naohirot | jroll: Actually I am behind http proxy, coreos-oem-inject.py doesn't work due to http proxy error. | 14:44 |
naohirot | jroll: I read source code, and typed commands by hand. | 14:45 |
jroll | naohirot: without the error, I can't help much... I have no idea why verifying the signature would fail behind a proxy | 14:45 |
naohirot | jroll: here is the error I got http://paste.openstack.org/show/185126/ | 14:47 |
jroll | naohirot: and if you run the gpg command manually? | 14:48 |
jroll | I guess that directory is gone by now | 14:48 |
jroll | I have a feeling this is related: gpg: WARNING: unsafe ownership on configuration file `/home/naohirot/.gnupg/gpg.conf' | 14:49 |
naohirot | jroll: no, I typed "sudo make coreos" | 14:49 |
jroll | shouldn't need sudo | 14:49 |
jroll | that may also be related | 14:49 |
naohirot | jroll: okay, without sudo, the result is same. | 14:51 |
jroll | naohirot: I have a feeling this is related: gpg: WARNING: unsafe ownership on configuration file `/home/naohirot/.gnupg/gpg.conf' | 14:51 |
jroll | naohirot: permissions on that file should be 600 and owned by naohirot:naohirot | 14:51 |
GheRivero | morning ironic | 14:53 |
jroll | morning GheRivero :) | 14:53 |
NobodyCam | good morning Ironic | 14:53 |
naohirot | jroll -rw------- 1 naohirot naohirot 9188 Jan 27 16:47 gpg.conf | 14:53 |
openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Properly implement authentication via keystonemiddleware https://review.openstack.org/160367 | 14:54 |
naohirot | jroll: http proxy cause error in cache_file() https://github.com/openstack/ironic-python-agent/blob/master/imagebuild/coreos/coreos-oem-inject.py#L137-L141 | 14:55 |
naohirot | jroll: so I downloaded those file by "wget" | 14:56 |
naohirot | jroll: wget http://storage.core-os.net/coreos/amd64-usr/367.1.0/coreos_production_pxe_image.cpio.gz.DIGESTS.asc | 14:58 |
naohirot | jroll: wget http://storage.core-os.net/coreos/amd64-usr/367.1.0/coreos_production_pxe.vmlinuz | 14:58 |
jroll | naohirot: ok, so if you manually do the gpg_verify_file stuff, what error do you get? | 14:58 |
jroll | naohirot: gpg is failing, we need to figure out why | 14:58 |
naohirot | jroll: wget http://storage.core-os.net/coreos/amd64-usr/367.1.0/coreos_production_pxe_image.cpio.gz | 14:58 |
jroll | naohirot: put this in an empty directory as coreos.key https://github.com/openstack/ironic-python-agent/blob/master/imagebuild/coreos/coreos-oem-inject.py#L178 | 14:59 |
jroll | naohirot: and run: gpg --batch --no-default-keyring --keyring /tmp/oem-gpg-validateTJrJdI/tmp.gpg --import /tmp/oem-gpg-validateTJrJdI/coreos.key | 14:59 |
jroll | naohirot: assuming you put the key in /tmp/oem-gpg-validateTJrJdI/ | 15:00 |
naohirot | jroll: okay, just a minite | 15:00 |
*** mgoddard1 has joined #openstack-ironic | 15:00 | |
*** mgoddard has quit IRC | 15:01 | |
naohirot | jroll: http://paste.openstack.org/show/185133/ | 15:04 |
naohirot | jroll: It seems succeeded | 15:04 |
jroll | weird | 15:05 |
jroll | naohirot: now gpg --batch --no-default-keyring --keyring /tmp/oem-gpg-validateTJrJdI/tmp.gpg --verify coreos_production_pxe_image.cpio.gz.DIGESTS.asc | 15:05 |
jroll | naohirot: and then you can skip the gpg_verify_file call | 15:06 |
naohirot | jroll: that means wget download worked without any tampering. | 15:06 |
jroll | the command with --verify means that downloading the *digests* file worked without tampering | 15:06 |
jroll | and then you should be able to compare those hashes to the hashes of the actual files | 15:07 |
naohirot | jroll: I see | 15:07 |
naohirot | jroll: I'll check why this happened later, but what I'd like to know is that generated ipa-coreos.iso should be okay to use my testing, right? | 15:10 |
jroll | naohirot: yes | 15:11 |
jroll | naohirot: by the way, we build a ramdisk every time we merge a commit | 15:11 |
jroll | if you don't have custom modifications, you could just use that :) | 15:11 |
jroll | naohirot: http://tarballs.openstack.org/ironic-python-agent/coreos/files/ | 15:11 |
naohirot | jroll: And I'm wondering why there is no code to detect virtual floppy like DIB does. | 15:12 |
naohirot | rameshg87: are you around? | 15:13 |
jroll | naohirot: because nobody has written any? | 15:13 |
jroll | I thought there was a thing | 15:13 |
rameshg87 | naohirot, hey | 15:13 |
* rameshg87 reads back | 15:13 | |
jroll | oh wait | 15:14 |
jroll | naohirot: https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/cmd/agent.py#L90-124 | 15:14 |
naohirot | rameshg87: why isn't there some code in IPA to detect virtual floppy? | 15:14 |
*** mtanino has joined #openstack-ironic | 15:14 | |
rameshg87 | naohirot, i believe it's there: https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/cmd/agent.py#L78-L85 | 15:15 |
naohirot | rameshg87: Aha, there is. :) | 15:15 |
rameshg87 | naohirot, get all files in "/sys/class/block/*/device/model" | 15:15 |
rameshg87 | naohirot, and find out | 15:15 |
rameshg87 | naohirot, same logic | 15:15 |
naohirot | rameshg87: Yap | 15:15 |
naohirot | rameshg87: python code | 15:16 |
rameshg87 | naohirot, yep :) | 15:16 |
naohirot | rameshg87: I was looking for shell code :) | 15:16 |
derekh | Anybody hitting this but with current neutron ? https://bugs.launchpad.net/tripleo/+bug/1421835 | 15:17 |
openstack | Launchpad bug 1421835 in neutron "Timeout reached while waiting for callback for node" [Undecided,In progress] | 15:17 |
derekh | *bug | 15:17 |
jroll | I just linked that vmedia code... | 15:17 |
naohirot | rameshg87: Have you ever tried "make iso" behind http proxy? | 15:18 |
rameshg87 | naohirot, yeah | 15:18 |
rameshg87 | naohirot, it's trouble some | 15:18 |
naohirot | rameshg87: succeeded? | 15:18 |
rameshg87 | naohirot, first you need Dockerfile change | 15:18 |
naohirot | rameshg87: Yes, I added ENV http_proxy | 15:19 |
rameshg87 | naohirot, my Dockerfile looks like this: http://paste.openstack.org/show/185147/ | 15:20 |
rameshg87 | naohirot, over that you need to start docker.io with proxy | 15:20 |
rameshg87 | naohirot, i couldn't find the conf option for this | 15:20 |
rameshg87 | naohirot, so what i did was "sudo service docker.io stop" | 15:21 |
rameshg87 | naohirot, start a screen session | 15:21 |
naohirot | rameshg87: I succeeded "make docker", but not "make coreos". | 15:21 |
naohirot | rameshg87: ./coreos-oem-inject.py doesn't work with proxy settings | 15:22 |
rameshg87 | naohirot, then export HTTP_PROXY=http://proxy:port | 15:22 |
jroll | derekh: I don't believe we've seen that | 15:22 |
rameshg87 | naohirot, then sudo -E docker.io | 15:22 |
rameshg87 | naohirot, iirc | 15:22 |
rameshg87 | naohirot, inshort you have to start docker.io with proxy settings as well | 15:22 |
rameshg87 | naohirot, i did build with proxy settings | 15:23 |
naohirot | rameshg87: I did, but ./coreos-oem-inject.py doesn't work, I don't know "request.get" complained. | 15:23 |
rameshg87 | naohirot, did you start docker.io with proxy settings ? | 15:24 |
derekh | jroll: weird, in tripleo ci we've been carrying a neutron revert for it, maybe its no longer needed or something, let me find out... | 15:24 |
rameshg87 | naohirot, no conf file worked for me for docker.io | 15:24 |
rameshg87 | naohirot, finally i had start the daemon manually with proxy settings in the environment variable | 15:24 |
rameshg87 | naohirot, that's what worked for me | 15:24 |
naohirot | rameshg87: Yes, I edited /etc/default/docker.io | 15:24 |
rameshg87 | naohirot, it didn't work for me :D | 15:24 |
jroll | derekh: yeah, our CI has been green, idk | 15:25 |
naohirot | rameshg87: and did "sudo service docker.io restart" | 15:25 |
rameshg87 | naohirot, nope | 15:25 |
rameshg87 | naohirot, i started docker.io daemon manually | 15:25 |
derekh | jroll: ok, will try a ci run without it | 15:25 |
*** anderbubble has joined #openstack-ironic | 15:25 | |
derekh | jroll: thanks | 15:25 |
rameshg87 | naohirot, just run docker.io after export environment variables | 15:25 |
jroll | np derekh | 15:25 |
naohirot | rameshg87: so finally have you succeeded or not? | 15:26 |
openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Properly implement authentication via keystonemiddleware https://review.openstack.org/160367 | 15:26 |
rameshg87 | naohirot, yeah it worked | 15:26 |
naohirot | rameshg87: You succeeded, amazing :) | 15:27 |
naohirot | rameshg87: So I should be able to create the iso. | 15:27 |
naohirot | rameshg87: I succeeded "make docker" | 15:28 |
rameshg87 | naohirot, yeah because it worked for me :) | 15:28 |
rameshg87 | naohirot, yeah i did ./imagebuild/coreos/full_trusty_build.sh iirc | 15:29 |
naohirot | rameshg87: I believe "make coreos" has nothing to do with docker, Am I right? | 15:29 |
rameshg87 | hope so | 15:29 |
rameshg87 | but i am not sure ... | 15:29 |
naohirot | rameshg87: what is "iirc"? | 15:30 |
rameshg87 | if i remember correctly | 15:30 |
rameshg87 | naohirot, i did sometime back .. | 15:30 |
*** beekneemech is now known as bnemec | 15:32 | |
*** achanda has joined #openstack-ironic | 15:33 | |
*** zz_jgrimm is now known as jgrimm | 15:35 | |
naohirot | rameshg87: I just tried "./imagebuild/coreos/full_trusty_build.sh iirc" | 15:36 |
naohirot | rameshg87: but I got same error at ./coreos-oem-inject.py oem UPLOAD | 15:36 |
*** anderbubble has quit IRC | 15:37 | |
*** hj-hp has joined #openstack-ironic | 15:37 | |
*** jlvillal|afk has quit IRC | 15:37 | |
*** anderbubble has joined #openstack-ironic | 15:38 | |
*** hj-hp has quit IRC | 15:39 | |
*** jlvillal|afk has joined #openstack-ironic | 15:39 | |
*** jlvillal|afk is now known as jlvillal | 15:39 | |
*** achanda has quit IRC | 15:44 | |
*** saripurigopi has joined #openstack-ironic | 15:44 | |
naohirot | jroll: rameshg87: Thank you for your help, I continue to investigate IPA error tomorrow. Good night! | 15:45 |
*** naohirot has quit IRC | 15:45 | |
jroll | naohirot: night :) | 15:45 |
jlvillal | Good morning Ironic | 15:45 |
NobodyCam | mornign jlvillal :) | 15:45 |
NobodyCam | niught naotokl | 15:45 |
NobodyCam | doh tab fail | 15:46 |
jlvillal | NobodyCam: I think he already left... | 15:46 |
*** hj-hp has joined #openstack-ironic | 15:46 | |
NobodyCam | yep :-p | 15:46 |
dtantsur | NobodyCam, jlvillal, morning | 15:48 |
NobodyCam | morning dtantsur | 15:48 |
openstackgerrit | Chris Krelle proposed openstack/ironic: Check temp dir is writable https://review.openstack.org/160383 | 15:49 |
* rameshg87 signs off for the day | 15:49 | |
rameshg87 | good night ironic :) | 15:49 |
NobodyCam | night rameshg87 | 15:49 |
jroll | morning NobodyCam jlvillal :) | 15:49 |
jroll | night rameshg87 ! | 15:49 |
NobodyCam | mornign jroll :) | 15:49 |
*** rameshg87 has quit IRC | 15:49 | |
jlvillal | jroll: Morning | 15:51 |
openstackgerrit | Chris Krelle proposed openstack/ironic: Check temp dir is writable https://review.openstack.org/160383 | 15:51 |
openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: HPSSA: Add support for sharing physical disks https://review.openstack.org/160385 | 15:52 |
*** Marga_ has joined #openstack-ironic | 15:55 | |
*** Marga_ has quit IRC | 15:55 | |
*** Marga_ has joined #openstack-ironic | 15:56 | |
trown | dtantsur: for 160367, are we using the "delegated" keystone middleware flow in order to raise discoverd Errors? | 15:57 |
dtantsur | trown, yes, because we have one non-authenticated endpoint | 15:58 |
trown | ah, cool | 15:58 |
*** saripurigopi has quit IRC | 16:01 | |
*** mgoddard1 has quit IRC | 16:02 | |
*** coolsvap is now known as coolsvap_ | 16:05 | |
openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Properly implement authentication via keystonemiddleware https://review.openstack.org/160367 | 16:13 |
trown | dtantsur: I am betting you just fixed all the things I was commenting on :p | 16:16 |
dtantsur | trown, maybe, sorry :) | 16:16 |
dtantsur | folks, NobodyCam, could you give some reviewing love to the inspecting thingy https://review.openstack.org/#/c/149823/ ? It's blocking my work and deadlines are dangerously close :) thanks | 16:21 |
NobodyCam | dtantsur: just +2 it | 16:22 |
dtantsur | ok, let's have more folks :) rloo, jroll, hey, mind having a look too? ^^^ | 16:23 |
NobodyCam | I didn't +a just so other reviewers can ensure their comments have been addressed... with no serious -'s on it I'll +A toward my eod | 16:23 |
jroll | such a large patch :| | 16:23 |
rloo | dtantsur: hmm. ok, let me see. | 16:24 |
NobodyCam | its 1/2 the other one :-p | 16:24 |
NobodyCam | well almost 1/2 | 16:24 |
jroll | NobodyCam: still recovering from that one :P | 16:24 |
NobodyCam | lol ya | 16:24 |
jroll | I may be able to look later on, no need to wait for me though | 16:24 |
rloo | dtantsur: oh, that patch. not a quick one but I guess I had a bunch of comments there. If you're happy then I'm happy but I'll look if you want. | 16:25 |
dtantsur | rloo, please look if your comments were addressed, at least the most serious ones :) | 16:26 |
dtantsur | and yeah, a hard patch for real :( | 16:26 |
rloo | dtantsur: darn, i was hoping to get out of doing that ;) | 16:26 |
dtantsur | I'm not really against just pressing +A, but I don't want your concerns to go unnoticed :) | 16:27 |
rloo | dtantsur: ok, looking. i already found one problem but minor one. | 16:28 |
NobodyCam | dtantsur: just a question on one of the your comments. have you seen issue with gettempdir? "I don't think gettempdir() will return successfully without creating a directory" | 16:30 |
NobodyCam | that should return the system temp dir no? | 16:31 |
dtantsur | NobodyCam, judging by https://docs.python.org/2/library/tempfile.html#tempfile.tempdir this check does not make sense at all: gettempdir should return something existing and writable | 16:32 |
NobodyCam | humm... deva reported this on friday: http://paste.openstack.org/show/Wg1aaQGqjF1PmfkTQSmL/ | 16:34 |
dtantsur | strange... | 16:35 |
dtantsur | ifarkas, hi! could you review https://review.openstack.org/#/c/160367/ ? | 16:35 |
ifarkas | dtantsur, sure | 16:35 |
NobodyCam | brb | 16:38 |
*** derekh has quit IRC | 16:39 | |
*** wuhg has quit IRC | 16:39 | |
*** Nisha has joined #openstack-ironic | 16:43 | |
*** igordcard has quit IRC | 16:50 | |
*** jjohnson2 has quit IRC | 16:50 | |
*** jjohnson2 has joined #openstack-ironic | 16:59 | |
*** mgoddard has joined #openstack-ironic | 17:00 | |
*** rwsu has joined #openstack-ironic | 17:09 | |
openstackgerrit | Merged stackforge/ironic-discoverd: Properly implement authentication via keystonemiddleware https://review.openstack.org/160367 | 17:10 |
openstackgerrit | Sirushti Murugesan proposed openstack/ironic: Adds support for deploying whole disk images https://review.openstack.org/150142 | 17:10 |
*** jmccrory has joined #openstack-ironic | 17:12 | |
*** spandhe has quit IRC | 17:15 | |
*** igordcard has joined #openstack-ironic | 17:16 | |
*** jistr has quit IRC | 17:21 | |
*** jcoufal has quit IRC | 17:24 | |
*** adam_g_out is now known as adam_g | 17:27 | |
*** anderbubble has quit IRC | 17:31 | |
*** anderbubble has joined #openstack-ironic | 17:37 | |
*** alexpilotti has joined #openstack-ironic | 17:39 | |
*** ukalifon1 has quit IRC | 17:41 | |
NobodyCam | brb | 17:43 |
*** ChuckC has quit IRC | 17:45 | |
openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Switch to Python 3.4 as default Python 3 env https://review.openstack.org/160434 | 17:53 |
*** pas-ha has quit IRC | 17:56 | |
*** spandhe has joined #openstack-ironic | 18:05 | |
*** jerryz_ has quit IRC | 18:05 | |
*** ijw has joined #openstack-ironic | 18:10 | |
openstackgerrit | Chris Krelle proposed openstack/ironic: Check temp dir is writable https://review.openstack.org/160383 | 18:12 |
*** MattMan has quit IRC | 18:17 | |
*** jjohnson2 has quit IRC | 18:19 | |
openstackgerrit | Chris Krelle proposed openstack/ironic: Check temp dir is writable https://review.openstack.org/160383 | 18:20 |
*** harlowja_away is now known as harlowja_ | 18:21 | |
NobodyCam | more coffee needed ... brb | 18:23 |
dtantsur | pretty late already, see you tomorrow | 18:28 |
*** dtantsur is now known as dtantsur|afk | 18:28 | |
openstackgerrit | Josh Gachnang proposed openstack/ironic: Implement execute clean steps https://review.openstack.org/155561 | 18:30 |
NobodyCam | night dtantsur|afk | 18:30 |
*** achanda has joined #openstack-ironic | 18:31 | |
*** igordcard has quit IRC | 18:33 | |
*** mgoddard has quit IRC | 18:35 | |
*** achanda has quit IRC | 18:46 | |
*** achanda has joined #openstack-ironic | 18:46 | |
*** pelix has quit IRC | 18:50 | |
NobodyCam | quick walkies .... brb | 18:52 |
*** kkoski has joined #openstack-ironic | 18:57 | |
* NobodyCam is back | 18:57 | |
*** devlaps has joined #openstack-ironic | 19:01 | |
*** ijw has quit IRC | 19:02 | |
*** jiangfei|3 has quit IRC | 19:03 | |
*** jiangfei|3 has joined #openstack-ironic | 19:03 | |
*** ijw has joined #openstack-ironic | 19:05 | |
*** jjohnson2 has joined #openstack-ironic | 19:05 | |
openstackgerrit | Merged stackforge/ironic-discoverd: Switch to Python 3.4 as default Python 3 env https://review.openstack.org/160434 | 19:07 |
*** ChuckC has joined #openstack-ironic | 19:11 | |
*** hj-hp has quit IRC | 19:16 | |
*** hj-hp has joined #openstack-ironic | 19:17 | |
rloo | NobodyCam, devananda, other core reviewers: how are decisions made? http://lists.openstack.org/pipermail/openstack-dev/2015-February/057794.html | 19:18 |
*** anderbubble has quit IRC | 19:19 | |
NobodyCam | rloo: humm | 19:19 |
rloo | I would like some closure on that ^^ eg review all such patches, single +2/+A if they don't touch code. | 19:19 |
*** anderbubble has joined #openstack-ironic | 19:19 | |
NobodyCam | give me a few to wrap up what I'm on and think about it for a sec | 19:19 |
rloo | NobodyCam: thx. it is the process of making a decision, that I'm not sure of. | 19:20 |
devananda | rloo: single +2/+A for nits and follow-ups by cores | 19:20 |
NobodyCam | but off the top of my head. I do tent to agree, | 19:20 |
NobodyCam | single +2/a is ok for that type of patch | 19:20 |
*** shakamunyi has joined #openstack-ironic | 19:20 | |
NobodyCam | :) | 19:20 |
rloo | NobodyCam, devananda. yup. but do we (core reviewers) vote on that? or devananda, can you say 'do it' ? | 19:21 |
JayF | sounds like a good thing to be announced at the meeting :) | 19:21 |
rloo | JayF: well, I'd also like to end the thread with some conclusion ;) | 19:21 |
devananda | rloo: I can say somethjing on the ML if that's necessary | 19:21 |
rloo | devananda: thx. I don't know if it is 'necessary', but I don't like discussions that are left hanging if you know what I mean. | 19:22 |
devananda | rloo: sure | 19:22 |
rloo | devananda: so it'd be great if you would say something :-) thanks in advance! | 19:22 |
rloo | btw devananda, separate topic. it seemed to me that you didn't like unit tests that had asserts on strings being in some exception or msg. I forgot why, or maybe I am wrong about that? | 19:23 |
devananda | rloo: i18n | 19:24 |
rloo | devananda: do our unit tests use the xlated strings if they are avail? | 19:24 |
rloo | devananda: or some other i18n reason that I'm not aware of? | 19:25 |
devananda | rloo: i'm not 100% sure, but I believe they did at one time | 19:25 |
devananda | iow, it is possible | 19:25 |
rloo | devananda: hmm. ok. I think we've approved such tests in the past week or so. | 19:25 |
devananda | a unit test that relies on the LOCALE setting is not reliable | 19:26 |
JayF | I think there are tests like that in IPA right now | 19:26 |
devananda | humm | 19:26 |
devananda | :( | 19:26 |
JayF | the error tests that you added, rloo | 19:26 |
JayF | but nothing in IPA gets translated today either | 19:26 |
rloo | JayF: yeah, IPA doesn't translate anything, so I didn't worry about it. (I say after the fact, I probably forgot about it.) | 19:27 |
devananda | need a break from this server closet .... bbi10 | 19:27 |
* rloo wonders what devananda is doing in a server closet | 19:28 | |
*** ifarkas has quit IRC | 19:29 | |
*** devlaps has quit IRC | 19:29 | |
*** Marga_ has quit IRC | 19:30 | |
Shrews | rloo: i forced him there to gimme some h/w to break. but he has to unbreak it, first :) | 19:30 |
*** Marga_ has joined #openstack-ironic | 19:31 | |
*** Marga_ has quit IRC | 19:31 | |
rloo | Shrews: good for you! :D | 19:31 |
*** Marga_ has joined #openstack-ironic | 19:32 | |
*** Marga_ has quit IRC | 19:37 | |
openstackgerrit | Merged openstack/ironic: Update install-guide for Ubuntu 14.10 package changes https://review.openstack.org/159597 | 19:37 |
openstackgerrit | Merged openstack/ironic: Generic changes for Node Inspection https://review.openstack.org/149823 | 19:38 |
NobodyCam | brb | 19:42 |
*** athomas has quit IRC | 19:46 | |
openstackgerrit | Sirushti Murugesan proposed openstack/ironic: Adds support for deploying whole disk images https://review.openstack.org/150142 | 19:48 |
*** ijw has quit IRC | 20:03 | |
*** achanda has quit IRC | 20:07 | |
*** achanda has joined #openstack-ironic | 20:08 | |
*** EmilienM|afk is now known as EmilienM | 20:10 | |
*** anderbubble has quit IRC | 20:34 | |
*** rloo has quit IRC | 20:36 | |
*** hj-hp has quit IRC | 20:37 | |
openstackgerrit | Chris Krelle proposed openstack/ironic: Check temp dir is writable https://review.openstack.org/160383 | 20:39 |
NobodyCam | brb | 20:40 |
*** anderbubble has joined #openstack-ironic | 20:40 | |
*** ndipanov is now known as ndipanov_gone | 20:40 | |
openstackgerrit | Merged openstack/ironic: Implement Cleaning in DriverInterfaces https://review.openstack.org/157208 | 20:41 |
*** hj-hp has joined #openstack-ironic | 20:44 | |
mrda | Morning Ironic | 20:45 |
NobodyCam | mornign mrda | 20:45 |
mrda | \o NobodyCam | 20:46 |
*** ijw has joined #openstack-ironic | 20:52 | |
*** achanda has quit IRC | 20:52 | |
NobodyCam | :) | 21:01 |
NobodyCam | devananda: forgot to ask... did you file a bug for the temp dir I ssue you hit on friday? | 21:04 |
*** shakamunyi has quit IRC | 21:07 | |
*** shakamunyi has joined #openstack-ironic | 21:08 | |
devananda | NobodyCam: nope | 21:18 |
NobodyCam | :) | 21:19 |
openstackgerrit | Josh Gachnang proposed openstack/ironic-specs: Change clean_steps to a dict https://review.openstack.org/160515 | 21:21 |
*** aignatov has quit IRC | 21:23 | |
*** chlong has joined #openstack-ironic | 21:23 | |
*** Marga_ has joined #openstack-ironic | 21:24 | |
*** BadCub is now known as BadCub_Away | 21:27 | |
*** rloo has joined #openstack-ironic | 21:33 | |
Nisha | devananda, NobodyCam hi | 21:34 |
NobodyCam | Hi Nisha :) | 21:34 |
Nisha | #link https://review.openstack.org/#/c/151596/23/ironic/drivers/modules/ilo/inspect.py | 21:35 |
Nisha | There is a comment to not to import "from ironic.db.sqlalchemy import api as dbapi" directly instead import it as its instance which is actually true | 21:36 |
Nisha | but i directly imported sqlalchemy because destroy_port always fails for me because of following check introduced https://github.com/openstack/ironic/blob/master/ironic/db/sqlalchemy/api.py#L469 | 21:37 |
Nisha | This was introduced as part of https://review.openstack.org/#/c/40219 | 21:38 |
Nisha | NobodyCam, devananda ^^^^ | 21:38 |
NobodyCam | h because reservation is cleared in inspecting | 21:40 |
NobodyCam | s/h/oh/ | 21:40 |
Nisha | it is not cleared.. | 21:40 |
Nisha | inspection actually doesnt deal with reservation | 21:40 |
Nisha | #link http://paste.openstack.org/show/185494/ following always fail with NodeLock error | 21:41 |
Nisha | NobodyCam, ^^^ | 21:42 |
NobodyCam | hummm: https://github.com/openstack/ironic-specs/commit/39598f5289a54ad3bc4149bdad410bf6fe667c41#diff-399db3a10ad90763f95700f6939a3932R89 | 21:43 |
*** ChuckC has quit IRC | 21:44 | |
NobodyCam | I may not have full or even correct context here | 21:44 |
Nisha | NobodyCam, but in case of in-band there is a seperate client getting invoked so it has to release the lock | 21:45 |
Nisha | and in this case it is happening within same function | 21:45 |
*** achanda has joined #openstack-ironic | 21:46 | |
Nisha | NobodyCam, yeah actually i am doubtful how to proceed for my case now | 21:46 |
NobodyCam | ack... let me think for a minute | 21:47 |
*** PaulCzar has quit IRC | 21:47 | |
NobodyCam | (run outside to smoke and think) :) | 21:47 |
Nisha | if i mask the check spoken above, then some of the unit tests for db fails | 21:47 |
Nisha | :) | 21:47 |
jlvillal | Nisha: I feel like you are either up very late or very early ;) | 21:50 |
Nisha | up very late | 21:50 |
Nisha | jlvillal, :) | 21:50 |
jlvillal | I hope you get to sleep late then! | 21:51 |
*** chlong has quit IRC | 21:51 | |
Nisha | not slept till now...wasnt well from few days so couldnt address the comments.. | 21:51 |
Nisha | so was trying to address them.... | 21:51 |
Nisha | jlvillal, yeah...was just thinking how to resolve this issue | 21:51 |
*** achanda has quit IRC | 21:52 | |
Nisha | NobodyCam, devananda should i post the patch with the comment that the piece of code doesnt work due to db issue and post the patch for now? | 21:52 |
*** absubram has joined #openstack-ironic | 21:53 | |
*** shakamunyi_ has joined #openstack-ironic | 21:53 | |
*** shakamunyi has quit IRC | 21:54 | |
NobodyCam | Nisha: No .. I would be happy with a # NOTE or # TODO above that import. that explaines the issue you've hit. will need to think of the Right way to fix this | 21:58 |
Nisha | NobodyCam, then what shall i import? | 21:58 |
NobodyCam | I can think of a couple of ulgy hacks that can get around it. but I would - any of my own ideas | 21:58 |
Nisha | is it fine to import sqlalchemy directly | 21:59 |
Nisha | ? | 21:59 |
NobodyCam | for now the direct import you are doing | 21:59 |
NobodyCam | ya other wise you're going to the reservation issue | 21:59 |
NobodyCam | but please Call out WHY you are doing it | 22:00 |
Nisha | yeah, because others have strong objection to import sqlalchemy directly | 22:00 |
NobodyCam | (sorry about the caps :)) | 22:00 |
jroll | wait | 22:00 |
jroll | isn't there a thing to get a dbapi instance? | 22:00 |
NobodyCam | hey jroll | 22:00 |
jroll | (whether it's sqlalchemy or some other driver) | 22:00 |
Nisha | jroll, its there | 22:00 |
Nisha | and thats where i am hitting this issue | 22:01 |
jroll | one sec | 22:01 |
Nisha | because the check is only for destroy_port | 22:01 |
NobodyCam | jroll: that hits this: https://github.com/openstack/ironic/blob/master/ironic/db/sqlalchemy/api.py#L142 | 22:01 |
jroll | besides the point | 22:02 |
jroll | Nisha: I think you want to use https://github.com/openstack/ironic/blob/master/ironic/db/api.py#L32 | 22:02 |
jroll | right? | 22:02 |
*** trown is now known as trown|outttypeww | 22:02 | |
jroll | not get a connection directly | 22:02 |
Nisha | jroll, yes | 22:02 |
Nisha | and i hit the issue given above | 22:03 |
Nisha | by NobodyCam | 22:03 |
jroll | whaaaaaat | 22:03 |
*** crow23 has joined #openstack-ironic | 22:03 | |
Nisha | jroll, so to get the functionality i had to import the sqlalchemy directly | 22:03 |
NobodyCam | reservation is set so you cann't delete a port | 22:03 |
*** crow23 has quit IRC | 22:04 | |
jroll | couldn't we add a force parameter to delete_port() or whatever that skips that check? | 22:04 |
jroll | (if we decide that check should be skipped) | 22:05 |
jroll | we should fix dbapi, not skip it | 22:05 |
Nisha | jroll, yes correct | 22:05 |
Nisha | but is it fine to do in inspection patch? | 22:05 |
Nisha | IMO it should be a seperate patch | 22:05 |
NobodyCam | add a new patch and make inspection depend on it | 22:06 |
jroll | then do it in a separate patch ahead of the inspection patch | 22:06 |
Nisha | NobodyCam, should i go ahead and file a bug? | 22:06 |
jroll | I don't care either way, personally, but we should fix it, not hack around it | 22:06 |
NobodyCam | not sure a bug is needed here, this is a change required to implament the spec | 22:07 |
Nisha | NobodyCam, ok, thats more helpful :) | 22:08 |
NobodyCam | implement even | 22:08 |
NobodyCam | :-p | 22:08 |
Nisha | I will do the change in the ilo patch then | 22:08 |
Nisha | :) | 22:08 |
NobodyCam | ok :) please call out in the commit message the 'why' you are makeing that change | 22:09 |
Nisha | NobodyCam, yes sure | 22:09 |
Nisha | I will add comment in the code as well | 22:09 |
NobodyCam | thank you Nisha, | 22:10 |
*** kkoski has quit IRC | 22:10 | |
NobodyCam | and jroll of the idea :) | 22:10 |
Nisha | :) | 22:10 |
NobodyCam | s/of/for | 22:10 |
jroll | :) | 22:13 |
jroll | thanks Nisha! | 22:13 |
Nisha | jroll NobodyCam thanks... | 22:13 |
* jlvillal is learning a lot about Mock by reviewing https://review.openstack.org/#/c/156562/9/ironic/tests/drivers/test_discoverd.py :) | 22:16 | |
*** jjohnson2 has quit IRC | 22:17 | |
rfchapman | NobodyCam: Hey, hows it going... I'm o-so close to getting a system to boot. right now I'm having an issue with the deploy kernel not detecting disks. | 22:17 |
*** andreykurilin_ has joined #openstack-ironic | 22:18 | |
NobodyCam | hey hey rfchapman | 22:20 |
NobodyCam | awesome | 22:20 |
NobodyCam | what does the os see the local disk as? | 22:20 |
rfchapman | NobodyCam: Its interesting that I installed a ubuntu image and it came up with the message that it could not find any disks. I think installed fedora image and it comes up with the same message. On fedora I would expect that the message would be different. the boot sequence looks identical to ubuntu. I'm thinking that the fedora image is some how the same as ubuntu image. Is there a way to tell which image is being u | 22:27 |
Nisha | rfchapman, you may face this issue if raid is not configured | 22:28 |
NobodyCam | Nisha: oh like no sda seen by the deploy ram disk | 22:29 |
Nisha | NobodyCam, i have faced this issue if raid is not configured on the system | 22:30 |
Nisha | atleast hp boxes | 22:30 |
rfchapman | Nisha, Ah I'll check that... These systems are vary old as well HP dl365g1 | 22:31 |
Nisha | rfchapman, :) | 22:31 |
Nisha | rfchapman, the way to interact these servers are ilo? | 22:32 |
rfchapman | Nisha, Yes, I'm lookin at them now | 22:33 |
*** andreykurilin_ has quit IRC | 22:33 | |
*** andreykurilin_ has joined #openstack-ironic | 22:34 | |
rfchapman | Is the best way to keep ironic from temporarly controlling the nodes, to put them into maintenance mode? | 22:34 |
jroll | yes | 22:35 |
rfchapman | thanks | 22:35 |
Nisha | NobodyCam, just adding the check for state helps to use the destroy_port function | 22:35 |
*** aignatov has joined #openstack-ironic | 22:36 | |
NobodyCam | Nisha: oh cool | 22:37 |
Nisha | NobodyCam, yeah but i will need to modify the test cases appropriately | 22:37 |
Nisha | so will push the patch tomorrow in office now..i was thinking to push it now | 22:38 |
Nisha | but test cases needs to be handled properly | 22:38 |
rloo | jroll: qq wrt https://review.openstack.org/#/c/157723/8/ironic/drivers/modules/agent_client.py | 22:38 |
*** mtanino has quit IRC | 22:39 | |
rloo | jroll: is image.install_bootloader supposed to be in ipa? | 22:39 |
JayF | makes sense that one such call would exist | 22:39 |
JayF | but afaik it does /not/ today | 22:39 |
JayF | although anything like that should be in the hardware manager as well | 22:40 |
rloo | JayF: ok, so I'm not blind. | 22:40 |
jroll | rloo: https://review.openstack.org/#/c/157469/ | 22:40 |
rloo | JayF: maybe there is a patch for it? | 22:40 |
*** alexpilotti has quit IRC | 22:41 | |
rloo | jroll: so we shouldn't approve 157723 until that patch ^^ is approved? | 22:41 |
jroll | rloo: I guess not | 22:41 |
rloo | jroll: ok | 22:41 |
jroll | I didn't think of it | 22:41 |
jroll | feel free to -2 :) | 22:41 |
*** achanda has joined #openstack-ironic | 22:42 | |
rloo | jroll: lucas' commit mentions a 'depends-on'. guess i should have looked there first. if there is a depends-on, it won't get merged until the other patch is approved anyway, right? | 22:43 |
jroll | oh, right | 22:44 |
jroll | rloo: you would think it won't get merged... but I haven't personally tested it yet :) | 22:44 |
rloo | jroll: well, there's only one way to test... | 22:44 |
mrda | Depends-on: has been implemented, at least that's what infra tells me. | 22:45 |
jroll | mrda: even in terms of merging thigns? | 22:46 |
jroll | I know it works for gate tests etc | 22:46 |
rloo | thx mrda. I'm testing that now ;) https://review.openstack.org/#/c/157723/ | 22:47 |
*** mtanino has joined #openstack-ironic | 22:47 | |
openstackgerrit | Nisha Agarwal proposed openstack/ironic: iLO implementation for hardware inspection https://review.openstack.org/151596 | 22:49 |
mrda | jroll: yes, for merging things | 22:49 |
mrda | Allows for cross-project depoendencies | 22:49 |
jroll | sweet | 22:50 |
jroll | thanks | 22:50 |
mrda | (yay for python-ironicclient <-> ironic dependencies being met before merge) | 22:50 |
mrda | Of course one hiccup is that I haven't tested this personally. But my infra friends are insistent that it's all fine and dandy | 22:51 |
*** ijw has quit IRC | 22:51 | |
*** wuhg has joined #openstack-ironic | 22:52 | |
*** Nisha has quit IRC | 22:54 | |
mrda | And double checked that this is correct | 22:55 |
*** PaulCzar has joined #openstack-ironic | 22:55 | |
* mrda reads this with interest: http://blog.nemebean.com/content/ipmi-controller-openstack-instances | 22:58 | |
NobodyCam | oh neat-oh | 23:00 |
NobodyCam | bnemec: very cool! | 23:00 |
*** andreykurilin_ has quit IRC | 23:00 | |
NobodyCam | going to have to ping that to jjohnson | 23:01 |
bnemec | Yeah, it was actually super easy with pyghmi. | 23:01 |
NobodyCam | :) sweet | 23:01 |
JayF | bnemec: I know martini (not in channel atm) did somethign similar | 23:01 |
mrda | bnemec: thanks for writing this up | 23:01 |
bnemec | I spent as much time figuring out how to use the Python Nova client as anything else. :-) | 23:01 |
JayF | except he wired it up with virtualbox | 23:01 |
JayF | at the mid-cycle | 23:01 |
JayF | well, at the rogue-mid-cycle (SF) | 23:02 |
NobodyCam | lol | 23:02 |
*** shakamunyi_ has quit IRC | 23:06 | |
*** absubram has quit IRC | 23:07 | |
*** ijw has joined #openstack-ironic | 23:16 | |
*** romcheg has quit IRC | 23:18 | |
*** romcheg has joined #openstack-ironic | 23:19 | |
*** hj-hp has quit IRC | 23:19 | |
*** romcheg has quit IRC | 23:19 | |
* NobodyCam thinks he needs food | 23:21 | |
*** hj-hp has joined #openstack-ironic | 23:22 | |
*** jgrimm is now known as zz_jgrimm | 23:26 | |
rfchapman | NobodyCam: Well that didn't seem to do the trick. Both Fedora and Ubuntu come up with the same message "No disks found, press t to troubleshoot." | 23:27 |
*** hj-hp has quit IRC | 23:28 | |
NobodyCam | rfchapman: how should the os see the disk sda? | 23:28 |
*** hj-hp has joined #openstack-ironic | 23:29 | |
rfchapman | I think these systems should come up a /dev/cciss/c0d0 | 23:30 |
rfchapman | It does detect the controller I do some mention of cciss in the boot messages, but I still don't have console access so captureing that information is a bit tough. | 23:31 |
NobodyCam | check the pxe config in the tftpboot dir | 23:31 |
NobodyCam | make sure thats listed | 23:31 |
NobodyCam | you on juno right | 23:32 |
*** anderbubble has quit IRC | 23:32 | |
rfchapman | yup, on juno | 23:32 |
*** chlong has joined #openstack-ironic | 23:33 | |
rfchapman | NobodyCam: disk=cciss/c0d0,sda,hda,vda this is what pxe exposes | 23:34 |
NobodyCam | that looks right then | 23:35 |
jroll | I wonder if the deploy ramdisk doesn't support an extra dir like that | 23:36 |
jroll | and we just don't know it | 23:36 |
NobodyCam | thats what I was just thinking | 23:36 |
* jroll attempts to resist looking | 23:37 | |
NobodyCam | or needs a module or such | 23:37 |
rfchapman | Man this was all working before the last trouble that I had... It would get all the way to the partitioning of the disk. | 23:37 |
jroll | oh that could be too | 23:37 |
jroll | I did 5 lines of bash today, that's enough bash for me | 23:37 |
mrda | JayF: I described it as the rebel midcycle myself :) | 23:37 |
*** openstackgerrit has quit IRC | 23:38 | |
*** BadCub_Away is now known as BadCub | 23:38 | |
rfchapman | Jrol: doing 5 lines of bash can get quite expensive | 23:39 |
*** openstackgerrit has joined #openstack-ironic | 23:39 | |
rfchapman | may have to slow your roll | 23:39 |
rfchapman | :-) | 23:40 |
jroll | heh | 23:40 |
NobodyCam | rfchapman: are you on hp hardware? | 23:41 |
*** Haomeng|2 has joined #openstack-ironic | 23:51 | |
*** Haomeng has quit IRC | 23:52 | |
rfchapman | NobodyCam: yes, HP dl365g1 | 23:53 |
rfchapman | 15 of them | 23:53 |
NobodyCam | rfchapman: maybe http://support.hp.com/us-en/document/c03045397 | 23:53 |
rfchapman | Humm, where would I modify the pxe boot to blacklist that driver? | 23:56 |
rfchapman | Also is the deploy kernel the same for both ubuntu and fedora? | 23:57 |
*** EmilienM is now known as EmilienM|afk | 23:58 | |
NobodyCam | kernel for deploy ramdisk depends on how it was build. I don't think they are the same rev's | 23:58 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!