*** hwoarang has quit IRC | 00:12 | |
*** ijw_ has joined #openstack-ironic | 00:13 | |
*** hwoarang has joined #openstack-ironic | 00:14 | |
TheJulia | dhellmann: tl;dr == if priority == 0 then manual | 00:14 |
---|---|---|
*** ijw has quit IRC | 00:17 | |
*** ijw_ has quit IRC | 00:18 | |
*** rloo has quit IRC | 00:34 | |
*** gyee has quit IRC | 00:43 | |
*** bdodd_ has joined #openstack-ironic | 01:05 | |
*** bdodd has quit IRC | 01:05 | |
*** mkrai has joined #openstack-ironic | 01:07 | |
*** ricolin has joined #openstack-ironic | 02:25 | |
*** rloo has joined #openstack-ironic | 02:26 | |
openstackgerrit | Nguyen Van Trung proposed openstack/ironic master: Support firmware upgrade for irmc driver https://review.opendev.org/599992 | 02:28 |
*** rloo has quit IRC | 02:28 | |
openstackgerrit | Merged openstack/networking-baremetal master: Fix networking-baremetal CI https://review.opendev.org/675701 | 02:58 |
*** rh-jelabarre has quit IRC | 02:59 | |
*** rh-jelabarre has joined #openstack-ironic | 03:03 | |
*** rh-jelabarre has quit IRC | 03:15 | |
*** ianychoi has quit IRC | 03:29 | |
*** ianychoi has joined #openstack-ironic | 03:30 | |
*** gkadam has joined #openstack-ironic | 03:36 | |
*** gkadam has quit IRC | 03:36 | |
*** dsneddon has quit IRC | 03:51 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add deploy steps for iLO BIOS interface https://review.opendev.org/674270 | 03:55 |
*** dsneddon has joined #openstack-ironic | 04:24 | |
*** rachit7 has joined #openstack-ironic | 04:58 | |
*** stendulker has joined #openstack-ironic | 05:15 | |
*** jhesketh has quit IRC | 05:34 | |
rachit7 | Hi stendulker | 05:38 |
rachit7 | Good Morning | 05:38 |
stendulker | Hi rachit7 | 05:39 |
rachit7 | Can you please add this patch https://review.opendev.org/#/c/674021/ on your priority list ? | 05:39 |
patchbot | patch 674021 - ironic - DRAC : clear_job_queue clean step to fix pending b... - 8 patch sets | 05:39 |
*** yaawang has quit IRC | 05:39 | |
stendulker | ok | 05:40 |
*** yaawang has joined #openstack-ironic | 05:40 | |
stendulker | rachit7: I did not get how the exception is getting handled for client.delete_jobs(job_ids=[_CLEAR_JOB_IDS]) | 05:41 |
stendulker | Are you expecting it to get handled at conductor level? | 05:42 |
*** jhesketh has joined #openstack-ironic | 05:42 | |
rachit7 | I don't think it needs any exception to handled. I mean if there is any job pending or scheduled or ongoing job it will clear it after this call and other exception for this call(client.delete_jobs) is handled by iDRAC lib(python-dracclient) | 05:46 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Add iLO RAID deploy steps https://review.opendev.org/674271 | 05:47 |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: WIP: Add iDRAC RAID deploy steps https://review.opendev.org/641731 | 05:48 |
stendulker | rachit7: python-dracclient exception is not handled in Ironic | 05:56 |
stendulker | Probably you need to handle and translate it to DracOperationError https://github.com/openstack/ironic/blob/master/ironic/common/exception.py#L498 | 05:58 |
rachit7 | stendulker : Ohhh okay.. got it.. | 05:59 |
openstackgerrit | Ankit Kumar proposed openstack/ironic master: Add iso less virtual media deploy https://review.opendev.org/677996 | 06:00 |
rachit7 | Thank you stendulker | 06:00 |
stendulker | np | 06:00 |
*** dsneddon has quit IRC | 06:14 | |
openstackgerrit | Shivanand Tendulker proposed openstack/ironic master: Follow-up for deploy steps for Redfish BIOS interface https://review.opendev.org/677695 | 06:20 |
*** jtomasek has joined #openstack-ironic | 06:27 | |
*** pradip_k has joined #openstack-ironic | 06:28 | |
*** yaawang has quit IRC | 06:31 | |
*** yaawang has joined #openstack-ironic | 06:33 | |
*** kaifeng has quit IRC | 06:34 | |
*** kaifeng has joined #openstack-ironic | 06:35 | |
arne_wiebalck | Good morning, ironic! | 06:36 |
*** dsneddon has joined #openstack-ironic | 06:42 | |
*** dsneddon has quit IRC | 06:47 | |
*** dsneddon has joined #openstack-ironic | 06:48 | |
*** dsneddon has quit IRC | 06:54 | |
*** dsneddon has joined #openstack-ironic | 06:54 | |
*** stendulker has quit IRC | 06:59 | |
*** dsneddon has quit IRC | 07:00 | |
*** dsneddon has joined #openstack-ironic | 07:00 | |
*** dsneddon has quit IRC | 07:06 | |
*** dsneddon has joined #openstack-ironic | 07:07 | |
*** trident has quit IRC | 07:08 | |
*** pradip_k has quit IRC | 07:08 | |
*** rcernin has quit IRC | 07:12 | |
rachit7 | Hey arne_wiebalck I have a ironic upstream patch with conflicts. can you help me to resolve it ? | 07:14 |
*** trident has joined #openstack-ironic | 07:17 | |
arne_wiebalck | rachit7: Could you provide the link (also for others to have a look)? | 07:29 |
rachit7 | arne_wiebalck https://review.opendev.org/#/c/671038/ | 07:29 |
patchbot | patch 671038 - ironic - DRAC: Drives conversion from raid to jbod - 13 patch sets | 07:29 |
yoctozepto | off-topic: how can we invite patchbot to another channel (e.g. kolla) as well? so far only seen it here and it could come in handy in discussion (no need to copy the subject) | 07:32 |
*** mrda has joined #openstack-ironic | 07:33 | |
mrda | Hey ironic, just a question on extra-hardware collection with ipa... There's no privilege escalation here: https://opendev.org/openstack/ironic-python-agent/src/branch/master/ironic_python_agent/inspector.py#L264-L268 But `hardware-detect` really should be run as root or else it doesn't collect very much. Would we prefer ipa to escalate privileges, or expect hardware-detect to be setuid'd or | 07:36 |
mrda | similar? Thoughts anyone? | 07:36 |
mrda | Or have I missed the point? | 07:37 |
mrda | :-) | 07:37 |
rachit7 | arne_wiebalck Let me brief you what i did when i recieved conflicts. 1. I tried to push the changes in upstream topic branch but said it has conflicts. 2. Then i tried to fetch remote(master and gerrit) and did rebase gerrit/master. 3. solved conflicts and able to see my topic branch updated and my patch set commit was on top the latest commit. 4. | 07:39 |
rachit7 | But my master branch was not updated compare to latest commit on ironic branch . 5.updated new patch set(unfortunately i missed to run tox and it was having syntax error) . 6. Then i updated master branch by following command `git fetch gerrit` and it updated. 7 But after when i run tox command it gives this error : ` File "/home/devel/git/ironic/ | 07:39 |
rachit7 | ironic/tests/unit/common/test_nova.py", line 14, in <module> | 07:39 |
rachit7 | arne_wiebalck I tried to run tox on master branch after rebasing it but it shows me same error. | 07:46 |
rachit7 | arne_wiebalck I tried to run tox on master branch after rebasing it but it shows me same above error. | 07:47 |
arne_wiebalck | rachit7: I'm in a meeting now. Will try to have a look later. | 07:47 |
rachit7 | arne_wiebalck sure no prob | 07:47 |
kaifeng | mrda: probably we need to pass a run_as_root=True there, the execution uses rootwrap, so we also need proper configuration for that during ramdisk building. | 07:57 |
*** lucasagomes has joined #openstack-ironic | 08:04 | |
*** dougsz has joined #openstack-ironic | 08:08 | |
*** alexmcleod has joined #openstack-ironic | 08:22 | |
*** rachit7 has quit IRC | 08:22 | |
mrda | kaifeng: So I'll look atthis some more tomorrow. But we probably want privsep rather than rootwrap, right? | 08:23 |
* mrda notices that privsep hasn't got here yet, so rootwrap it is | 08:29 | |
*** rachit7 has joined #openstack-ironic | 08:29 | |
*** yolanda__ is now known as yolanda | 08:30 | |
openstackgerrit | raphael.glon proposed openstack/ironic-python-agent master: image extension, install_bootloader improvements https://review.opendev.org/674879 | 08:34 |
*** dsneddon has quit IRC | 08:38 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent-builder master: [WIP] Import ansible version of the image building job https://review.opendev.org/678884 | 08:41 |
*** derekh has joined #openstack-ironic | 08:43 | |
kaifeng | mrda: hmm, is privsep broadly adopted in other projects? maybe we need a story for tracking that. | 08:48 |
kaifeng | mrda: hmm, is privsep broadly adopted in other projects? maybe we need a story for tracking that. | 08:48 |
mrda | kaifeng: privsep is the future openstack thing to replace rootwrap. But we haven't moved here in Ironic yet. That's ok. We can do that in the future with a story and planning etc. For this, I'll just continue with rootwrap as that's what is in play right now. | 08:55 |
mrda | I'll get a patch up for i-p-a-builder tomorrow | 08:55 |
kaifeng | mrda: thanks! note that the ipa needs a patch too, otherwise it's not able to run as root. | 08:57 |
openstackgerrit | Merged openstack/ironic-python-agent master: Run a standalone job in the gate https://review.opendev.org/675937 | 08:59 |
arne_wiebalck | rachit7: I had a first look and left a comment on the review as to why tox is complaining. | 09:00 |
openstackgerrit | Merged openstack/bifrost master: Install mariadb instead of mysql on deb distros https://review.opendev.org/665902 | 09:01 |
openstackgerrit | Rachit Kapadia proposed openstack/ironic master: DRAC: Drives conversion from raid to jbod https://review.opendev.org/671038 | 09:04 |
rachit7 | Hi arne_wiebalck Thanks for the review i have already resolved that syntax error in local branch and just updated patchset too but still problem remains same | 09:05 |
arne_wiebalck | rachit7: One test failing you mean? | 09:07 |
rachit7 | arne_wiebalck : yess trying to run tox but giving this error `========================= | 09:08 |
rachit7 | ", line 456, in _find_test_path | 09:08 |
*** dsneddon has joined #openstack-ironic | 09:12 | |
arne_wiebalck | rachit7: for me, test_validate_configuration_custom_schema is the failing test | 09:15 |
*** dsneddon has quit IRC | 09:17 | |
rachit7 | arne_wiebalck okay | 09:20 |
arne_wiebalck | rachit7: without that test, tox passes py27 | 09:24 |
rachit7 | arne_wiebalck Ohhh.. do you know any solution for that? | 09:32 |
arne_wiebalck | rachit7: can you reproduce the failing test in your local tests? | 09:32 |
rachit7 | arne_wiebalck Is it possible to go two patchset behind and rebase branch again? | 09:33 |
*** dougsz has quit IRC | 09:35 | |
arne_wiebalck | rachit7: what is the problem you're trying to solve? | 09:35 |
arne_wiebalck | rachit7: you think your local repo is in a bad state? | 09:36 |
rachit7 | arne_wiebalck yes i think | 09:36 |
arne_wiebalck | rachit7: ok | 09:36 |
arne_wiebalck | rachit7: in case you didn't know, there is a download link on the top right corner in the gerrit dashboard | 09:37 |
*** hjensas has joined #openstack-ironic | 09:37 | |
rachit7 | arne_wiebalck i tried to run tox without my changes by checkout master branch and it gives me same error | 09:37 |
arne_wiebalck | rachit7: from there you can checkout the branch with your patch | 09:37 |
rachit7 | arne_wiebalck okay | 09:37 |
arne_wiebalck | rachit7: what I usually do in this case is sth like picking a commit ID, do a hard reset, followed by a pull :) | 09:38 |
arne_wiebalck | rachit7: try the checkout from gerrit, this should leave you on sth that is clean | 09:39 |
erolg | morning Ironic o/, Is there anyone available for review #677181. | 09:44 |
rachit7 | arne_wiebalck okay i will do that | 09:45 |
arne_wiebalck | rachit7: good luck :) | 09:46 |
rachit7 | Thanks for the help arne_wiebalck :) | 09:46 |
arne_wiebalck | rachit7: yw | 09:47 |
*** dougsz has joined #openstack-ironic | 09:51 | |
* mrda wonders whether anyone has used 'extra-hardware' in ipa in anger before.... | 10:06 | |
*** dsneddon has joined #openstack-ironic | 10:18 | |
*** ricolin has quit IRC | 10:25 | |
openstackgerrit | Rachit Kapadia proposed openstack/ironic master: DRAC: Drives conversion from raid to jbod https://review.opendev.org/671038 | 10:29 |
openstackgerrit | M V P Nitesh proposed openstack/ironic master: UEFI PXE Provisioning for wholedisk images https://review.opendev.org/664864 | 10:58 |
rachit7 | Hey arne_wiebalck it worked. i did copy git pull url of last patch set from gerrit dasboard and git checkout with last patch set and run tox and it passed | 11:03 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic stable/stein: Do not wait for console being started on timeout https://review.opendev.org/679025 | 11:07 |
arne_wiebalck | rachit7: nice | 11:10 |
*** tesseract has joined #openstack-ironic | 11:12 | |
*** dtantsur|afk is now known as dtantsur | 11:26 | |
dtantsur | hi ironic | 11:26 |
TheJulia | Good Morning! | 11:29 |
dtantsur | morning TheJulia | 11:31 |
arne_wiebalck | dtantsur: TheJulia: o/ | 11:32 |
*** Lucas_Gray has joined #openstack-ironic | 11:33 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent-builder master: Correct the relative path calculation https://review.opendev.org/679033 | 11:33 |
*** dsneddon has quit IRC | 11:36 | |
*** dsneddon has joined #openstack-ironic | 11:45 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent-builder master: Import ansible version of the image building job https://review.opendev.org/678884 | 11:45 |
* TheJulia pokes gerrit | 11:46 | |
etingof | rpioso, o/ I am refactoring sushy along the lines of your suggestions -- https://review.opendev.org/670579 | 11:56 |
patchbot | patch 670579 - sushy - Make message registries available to all resources (MERGED) - 3 patch sets | 11:56 |
etingof | rpioso, it seems this change is overwhelming | 11:56 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent-builder master: [WIP] Import ansible version of the image building job https://review.opendev.org/678884 | 11:58 |
openstackgerrit | Julia Kreger proposed openstack/bifrost stable/stein: Remove openSUSE Leap 42.3 jobs https://review.opendev.org/677181 | 12:00 |
*** rh-jelabarre has joined #openstack-ironic | 12:01 | |
TheJulia | dtantsur: Looking at https://zuul.opendev.org/t/openstack/build/f92849f0dfd744ba8b6c0dd7a38f1b42 I'm wondering if we're hitting a race in neutron | 12:03 |
dtantsur | sigh | 12:08 |
TheJulia | Actually looks like order of operations :\ | 12:09 |
TheJulia | yeah, race... except its in network booting https://a98906b115e1af31900d-686811860e75a35eabcecb5697905253.ssl.cf1.rackcdn.com/678729/1/gate/ironic-tempest-ipa-partition-redfish-tinyipa/f92849f/controller/logs/ironic-bm-logs/node-0_no_ansi_2019-08-28-10:43:22_log.txt.gz *sigh* | 12:16 |
TheJulia | Is there a reason to really run rescue testing so prolithicly in our CI? | 12:16 |
openstackgerrit | Rachit Kapadia proposed openstack/ironic master: DRAC: Drives conversion from raid to jbod https://review.opendev.org/671038 | 12:19 |
openstackgerrit | Julia Kreger proposed openstack/ironic-tempest-plugin master: Enable boot_option to be specified https://review.opendev.org/678633 | 12:22 |
rpioso | etingof: If memory serves, that change addresses a non-blocking bug. If you agree with my comments, it could be reverted and fixed anew later. | 12:23 |
etingof | rpioso, I think your concern was that we release a new API (registries) which we will have to pull back in the next release, no? | 12:24 |
TheJulia | arne_wiebalck: crazy off the wall question, but software raid implies localboot right? | 12:24 |
rachit7 | Good Morning rpioso :) | 12:25 |
rpioso | etingof: Correct. That would be an API change. | 12:25 |
rpioso | rachit7: Good morning :-) | 12:26 |
etingof | rpioso, that's why I am trying to change it just once, before the upcoming release... this patch will be huge :( | 12:26 |
arne_wiebalck | TheJulia: erm ... by implies you mean "wouldn't work without" ? | 12:26 |
arne_wiebalck | TheJulia: IIRC, the RAID code does not change anything related to how the instance is going to boot | 12:27 |
arne_wiebalck | TheJulia: if that is your question? | 12:28 |
rpioso | etingof: The alternative is revert and fix anew after the next release. And that could be back ported, because it’s a bug fix. | 12:28 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent-builder master: [WIP] Import ansible version of the image building job https://review.opendev.org/678884 | 12:29 |
etingof | rpioso, would reverting this patch undermine our Dell server relationship...? | 12:30 |
TheJulia | arne_wiebalck: what do you guys run with your default_boot_option? | 12:31 |
arne_wiebalck | TheJulia: local | 12:32 |
TheJulia | okay... hmm | 12:32 |
rpioso | etingof: I don’t believe so, but I’m not certain. The Dell EMC 3d Party CI would be a quick smoke test. | 12:32 |
TheJulia | arne_wiebalck: I'm trying to change the default to actually be local boot, and I noticed the raid CI jobs failed when local was explicitly set. I'm wondering if there is an edge case where the code gets confused with raid if it is set via capabilities | 12:33 |
arne_wiebalck | TheJulia: how are the jobs failing? | 12:35 |
TheJulia | https://dbe05845bd36655c0e9a-218375fc8749de6a1ac263f444b96da4.ssl.cf5.rackcdn.com/678633/4/check/ironic-standalone/899dbe5/testr_results.html.gz | 12:36 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent-builder master: [WIP] Import ansible version of the image building job https://review.opendev.org/678884 | 12:36 |
TheJulia | arne_wiebalck: "Failed to install a bootloader when deploying node 38350e1e-66b0-498a-8bba-606bec1f05b9. Error: {u'message': u\"Command execution failed: Installing GRUB2 boot loader to device /dev/md0 failed with Unexpected error while running command.\\nCommand: mount /dev/md0p1 /tmp/tmp4k691suz\\nExit code: 32\\nStdout: ''\\nStderr: 'mount: special device /dev/md0p1 does not exist\\\\n'.\" | 12:37 |
* TheJulia checks the job time because it may hav ebeen an old ipa | 12:37 | |
TheJulia | Looks like we have an even newer IPA build | 12:38 |
TheJulia | It might have run with an older build, so that may be it | 12:39 |
* TheJulia engages wait() | 12:39 | |
etingof | rpioso, let me go through that patch... on the other hand, the API change introduced by that patch is really negligible... | 12:41 |
*** rachit7 has quit IRC | 12:43 | |
*** rloo has joined #openstack-ironic | 12:45 | |
rpioso | etingof: The API change is most, if not all, objects now have a 'registries' property, right? | 12:46 |
arne_wiebalck | TheJulia: I'll need to have a look at the code, but the bootloader should not go onto the md device or its partition ... but maybe that's just a confusing error message? | 12:46 |
rpioso | etingof: s/objects/classes/ | 12:46 |
*** sthussey has joined #openstack-ironic | 12:47 | |
TheJulia | arne_wiebalck: possibly, I'm re-running my change of the default option patch as well which should give us an idea of if there is a bigger bug someplace under the hood | 12:47 |
etingof | rpioso, yes, but this is internal to sushy. these objects are not exposed through the package API | 12:48 |
TheJulia | arne_wiebalck: but always the possibility that the device patch didn't actually make it in to the prior IPA iamges because the builds were failing for a few days. | 12:48 |
dtantsur | TheJulia: last time this message was a sign of missing mdadm | 12:48 |
dtantsur | although we shouldn't have regressed in this aspect | 12:48 |
TheJulia | Agreed, time will tell | 12:48 |
rpioso | etingof: Isn't that property public, without an underscore prefix? | 12:49 |
etingof | rpioso, it is, but I doubt anyone would use it... | 12:49 |
etingof | rpioso, ...and we could preserve it going forward for backward compatibility if needed | 12:50 |
*** ricolin has joined #openstack-ironic | 12:50 | |
rpioso | etingof: Given that it's public, I believe it's considered part of the API. | 12:51 |
etingof | rpioso, formally, yes. but in reality, I doubt it's important. still, there should be no issue keeping it exposed | 12:52 |
etingof | rpioso, last minute reverting feels unsafe a bit, so I'd propose to release sushy as-is for now | 12:53 |
rpioso | etingof: Sure, it could be deprecated or preserved for backwards compatibility, but that seems like extra work, confusing, and untidy. If it's truly not essential, revert and revisit. I'm pretty certain it's not essential. | 12:54 |
rpioso | etingof: Do you agree it's a bug fix? | 12:55 |
rpioso | etingof: If so, revert, revisit, and back port is an alternative to leaving it as-is. | 12:56 |
openstackgerrit | Merged openstack/ironic master: Out-of-band `erase_devices` clean step for Proliant Servers https://review.opendev.org/641582 | 12:56 |
etingof | rpioso, it feels like something in-between a bug fix and a new feature | 12:58 |
* TheJulia is happy thigns actually pass CI now | 13:02 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent-builder master: Import ansible version of the image building job https://review.opendev.org/678884 | 13:03 |
dtantsur | TheJulia: ^^ | 13:03 |
TheJulia | dtantsur: ack | 13:04 |
TheJulia | I already had it on my list for later today | 13:04 |
dtantsur | I think I've finally finished cleaning it up :) | 13:04 |
TheJulia | :) | 13:05 |
etingof | rpioso, there seems to be a dependent patch related to message registries caching | 13:05 |
etingof | rpioso, we will have to revert it as well -- https://review.opendev.org/#/c/671091/3/sushy/main.py | 13:07 |
patchbot | patch 671091 - sushy - Cache message registries (MERGED) - 3 patch sets | 13:07 |
TheJulia | Hmmm... we don't mention software raid in the release notes | 13:09 |
*** jcoufal has joined #openstack-ironic | 13:17 | |
*** dsneddon has quit IRC | 13:20 | |
dtantsur | sudo: ovs-vsctl: command not found | 13:27 |
dtantsur | a silly error in the V6 job, I wonder what installs it usually | 13:27 |
*** dsneddon has joined #openstack-ironic | 13:28 | |
TheJulia | hmmm | 13:29 |
TheJulia | that is needed really early on for multinode job | 13:29 |
TheJulia | jobs | 13:29 |
TheJulia | because of the vxlan tunnel | 13:29 |
*** rachit7 has joined #openstack-ironic | 13:31 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: WIP ipv6 jobs https://review.opendev.org/613169 | 13:32 |
dtantsur | that's not a multinode job though | 13:32 |
dtantsur | trying setting Q_* stuff explicitly ^^ | 13:32 |
*** dsneddon has quit IRC | 13:33 | |
TheJulia | q_* _should_ only be for the legacy devstack plugin... | 13:34 |
dtantsur | TheJulia: yep, and I had to switch to it because the new one seemingly defaults neutron-api to 0.0.0.0 | 13:37 |
TheJulia | GAH | 13:42 |
* TheJulia hands dtantsur nikka coffey grain | 13:42 | |
*** Goneri has joined #openstack-ironic | 13:42 | |
openstackgerrit | Ilya Etingof proposed openstack/sushy master: Change OEM extensions architecture https://review.opendev.org/676889 | 13:45 |
openstackgerrit | Ilya Etingof proposed openstack/sushy master: Add conditional field matching https://review.opendev.org/675073 | 13:45 |
openstackgerrit | Dmitry Tantsur proposed openstack/bifrost master: Stop using deprecated OS_URL https://review.opendev.org/679058 | 13:47 |
dtantsur | TheJulia: also FYI ^^ | 13:47 |
TheJulia | dtantsur: thanks | 13:47 |
dtantsur | also Nikka Coffey Grain sounds amazing :) | 13:49 |
*** jtwill98 has joined #openstack-ironic | 13:49 | |
dtantsur | TheJulia: are we ready to remove coreos jobs from IPA? | 13:53 |
TheJulia | *sigh* | 13:53 |
TheJulia | I think so | 13:53 |
dtantsur | well, a replacement is coming together pretty quickly :) | 13:53 |
TheJulia | dib based? | 13:53 |
dtantsur | yeah | 13:54 |
dtantsur | all the stuff I've been doing the last couple of weeks | 13:54 |
TheJulia | yeah, CoreOS was a good alternative and I suspect we may revisit it, but the best thing for us to do at this point is to kill it with fire | 13:54 |
TheJulia | because for newer coreos stream we're going to have to start largely from scratch | 13:54 |
TheJulia | streams | 13:54 |
*** priteau has joined #openstack-ironic | 13:54 | |
rpioso | etingof: Agree | 14:04 |
*** dsneddon has joined #openstack-ironic | 14:07 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Remove support for CoreOS images https://review.opendev.org/679064 | 14:08 |
dtantsur | here we go | 14:08 |
TheJulia | https://review.opendev.org/679065 as well :) | 14:18 |
patchbot | patch 679065 - election - Add Julia Kreger for Ironic U cycle PTL - 1 patch set | 14:18 |
*** openstackgerrit has quit IRC | 14:22 | |
etingof | rpioso, do you think we should revert both patches or none? | 14:23 |
*** amotoki_ has quit IRC | 14:34 | |
*** amotoki has joined #openstack-ironic | 14:35 | |
dtantsur | revert the heck out of everything! | 14:42 |
* dtantsur has no idea, just kidding | 14:43 | |
*** hjensas has quit IRC | 14:44 | |
*** openstackgerrit has joined #openstack-ironic | 14:45 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Add software raid release note to ironic https://review.opendev.org/679075 | 14:45 |
dtantsur | hmm, suse jobs are actually broken, aren't they? | 14:45 |
TheJulia | not entirely sure, the bifrost change has a naming discrepency | 14:46 |
TheJulia | I've not cycled back to that yet this morning | 14:46 |
dtantsur | https://review.opendev.org/#/c/679058/ | 14:47 |
patchbot | patch 679058 - bifrost - Stop using deprecated OS_URL - 1 patch set | 14:47 |
TheJulia | *gasp* https://review.opendev.org/#/c/678633/ passed | 14:47 |
patchbot | patch 678633 - ironic-tempest-plugin - Enable boot_option to be specified - 5 patch sets | 14:47 |
TheJulia | ship it! | 14:47 |
TheJulia | And I feel like we might have a bug https://storage.gra1.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/logs_14/492114/18/check/ironic-standalone/d6aac91/testr_results.html.gz | 14:49 |
*** hjensas has joined #openstack-ironic | 14:51 | |
*** Lucas_Gray has quit IRC | 14:51 | |
dtantsur | sorry, I'm banging my head on the stupid tempest timeout | 14:53 |
* dtantsur wonders why we even use tempest | 14:54 | |
TheJulia | dtantsur: which one? | 14:54 |
dtantsur | https://c5b5c576705ed78be235-fc6b0fdbb44c8a933c3daf9bbf32644a.ssl.cf2.rackcdn.com/676239/7/check/ipa-tempest-wholedisk-bios-ipmi-direct-dib-src/c82cf60/job-output.txt | 14:55 |
TheJulia | lol, it timed out during cleaning | 14:56 |
dtantsur | yeah, the success was so close | 14:56 |
dtantsur | now I wonder why it timed out after 5 minutes if unprovision_timeout is 900 seconds | 14:57 |
TheJulia | I had to disable automated cleaning on the networking-baremetal job because the extra time just takes it a little bit longer and hits the failure point | 14:57 |
TheJulia | I did post a change to extend the base timeouts a little more | 14:57 |
TheJulia | because CI performance is always a variable | 14:57 |
*** igordc has joined #openstack-ironic | 14:57 | |
TheJulia | oh, hmmm | 14:58 |
TheJulia | the outer job timeout I would guess.... | 14:58 |
TheJulia | hmm | 14:58 |
TheJulia | in the actual wait | 14:58 |
*** mkrai_ has joined #openstack-ironic | 14:59 | |
dtantsur | and why do we poke it every second if interval=30 | 14:59 |
*** mkrai has quit IRC | 15:01 | |
TheJulia | so we don't flood the logs | 15:03 |
TheJulia | yeah, 900 seconds... hmm | 15:03 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: WIP ipv6 jobs https://review.opendev.org/613169 | 15:07 |
dtantsur | TheJulia: I suspect it may be OS_TEST_TIMEOUT which we do raise for coreos jobs | 15:07 |
dtantsur | asked nishagb to try it | 15:07 |
TheJulia | yeah, I think that gets read in the code internally and not logged as config, it could be the only real way | 15:08 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent-builder master: Install IPA from source in the CI https://review.opendev.org/679086 | 15:18 |
*** donnyd has quit IRC | 15:19 | |
*** donnyd has joined #openstack-ironic | 15:20 | |
*** donnyd has quit IRC | 15:20 | |
*** donnyd has joined #openstack-ironic | 15:21 | |
rpioso | etingof: Both | 15:22 |
etingof | rpioso, the second patch feels important - without it ironic log turns into message registries dump ;) | 15:23 |
*** gyee has joined #openstack-ironic | 15:26 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent-builder master: DIB: install IPA from source checked out by Zuul https://review.opendev.org/679086 | 15:26 |
*** dsneddon has quit IRC | 15:27 | |
*** ash2307 has joined #openstack-ironic | 15:28 | |
TheJulia | etingof: yeah, please don't let it dump registries into ironic's logs... | 15:28 |
*** ash2307 has left #openstack-ironic | 15:28 | |
rpioso | etingof: Was that undesirable situation introduced by https://review.opendev.org/670579 ? | 15:28 |
patchbot | patch 670579 - sushy - Make message registries available to all resources (MERGED) - 3 patch sets | 15:28 |
rpioso | etingof: In other words, did the dependent change fix a bug in that one ^^^? | 15:30 |
*** mkrai_ has quit IRC | 15:32 | |
NobodyCam | Good Morning Ironic folks | 15:32 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Switch to image building jobs from ironic-python-agent-builder https://review.opendev.org/679087 | 15:33 |
etingof | rpioso, the first patch allowed caching once downloaded message registries - that's what the second (dependent) patch does | 15:34 |
*** trandles has joined #openstack-ironic | 15:35 | |
etingof | rpioso, second patch (caching registries) seems more like a feature, excessive registries download and logging has been introduced by some older patch perhaps implementing message registries | 15:35 |
dtantsur | folks, looking for reviews on https://review.opendev.org/#/c/679033/ please | 15:36 |
patchbot | patch 679033 - ironic-python-agent-builder - Correct the relative path calculation - 1 patch set | 15:36 |
etingof | rpioso, so we can only revert both patches at once (some more dependencies can potentially come up along the way) | 15:36 |
etingof | rpioso, I am not yet convinced we should revert the patches because (in my mind) one exposed public API property does not justify (1) disabling registries cache and (2) risking regressions | 15:38 |
*** niceplace_ has joined #openstack-ironic | 15:39 | |
*** dsneddon has joined #openstack-ironic | 15:44 | |
*** bdodd has joined #openstack-ironic | 15:53 | |
*** bdodd_ has quit IRC | 15:54 | |
*** lucasagomes has quit IRC | 16:00 | |
openstackgerrit | Rachit Kapadia proposed openstack/ironic master: DRAC: Drives conversion from JBOD to RAID https://review.opendev.org/679093 | 16:01 |
*** whoami-rajat has joined #openstack-ironic | 16:01 | |
openstackgerrit | Rachit Kapadia proposed openstack/ironic master: [WIP]DRAC: Drives conversion from JBOD to RAID https://review.opendev.org/679093 | 16:03 |
*** ricolin has quit IRC | 16:04 | |
*** ricolin has joined #openstack-ironic | 16:04 | |
*** tesseract has quit IRC | 16:08 | |
*** ricolin has quit IRC | 16:12 | |
*** alexmcleod has quit IRC | 16:14 | |
openstackgerrit | Merged openstack/networking-baremetal master: Update api-ref location https://review.opendev.org/673598 | 16:15 |
*** rachit7 has quit IRC | 16:16 | |
openstackgerrit | Merged openstack/networking-baremetal master: Update Python 3 test runtimes for Train https://review.opendev.org/665073 | 16:17 |
trandles | Question/comment about ironic console and ipmitool-socat. I'm using ironic installed from CentOS 7 stein RPMs. It's standalone (nothing but ironic-api and ironic-conductor). I can't get serial console because the ironic services are running as user ironic, but it's trying to bind socat to a privileged port (623) and gets permission denied. Am I missing something in the config? I would have expected socat to bind a local unprivileged port. | 16:19 |
trandles | | last_error | Error enabling the console on node 2f9fd223-839c-4bf8-ae22-fc8d7394c786. Reason: Console subprocess failed to start. Command: socat -T600 -L/tmp/2f9fd223-839c-4bf8-ae22-fc8d7394c786.pid TCP4-LISTEN:623,bind=192.168.100.254,reuseaddr,fork,max-children=1 EXEC:"ipmitool -I lanplus -H 192.168.100.253 -L ADMINISTRATOR -U NULL -f /tmp/2f9fd223-839c-4bf8-ae22-fc8d7394c786.pw sol activate",pty,stderr. | | 16:20 |
trandles | | | Exit code: 1. | | 16:20 |
trandles | | | Stderr: '2019/08/28 10:16:12 socat[99114] E bind(15, {AF=2 192.168.100.254:623}, 16): Permission denied\n' | 16:20 |
openstackgerrit | Nisha Brahmankar proposed openstack/ironic-python-agent master: Adds CI job that uses DIB build images https://review.opendev.org/676239 | 16:20 |
priteau | trandles: Did you set the ipmi_terminal_port on each node to a unique, unprivileged port? | 16:30 |
trandles | Ok, answering my own question. ipmi_terminal_port is the LOCAL port, not the remote port. Maybee documentation ambiguity. | 16:30 |
trandles | priteau: yes, that fixed it... | 16:31 |
trandles | One of those days | 16:31 |
trandles | I read the ipmi_port note on https://docs.openstack.org/ironic/stein/install/enrollment.html and didn't distinguish local vs. remote | 16:33 |
*** derekh has quit IRC | 16:57 | |
*** dougsz has quit IRC | 17:01 | |
* TheJulia reads | 17:03 | |
TheJulia | o/ trandles | 17:03 |
TheJulia | \o/ | 17:04 |
*** dtantsur is now known as dtantsur|afk | 17:08 | |
dtantsur|afk | g'night | 17:08 |
*** ijw has joined #openstack-ironic | 17:14 | |
*** ijw has quit IRC | 17:16 | |
*** ijw has joined #openstack-ironic | 17:16 | |
*** ijw has quit IRC | 17:17 | |
*** ijw has joined #openstack-ironic | 17:18 | |
trandles | Hi TheJulia | 17:39 |
trandles | Sorry, poorly timed message earlier. I had to run to a meeting for the past hour. | 17:40 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Remove support for CoreOS images https://review.opendev.org/679064 | 17:46 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: Switch to image building jobs from ironic-python-agent-builder https://review.opendev.org/679087 | 17:47 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent master: DNM testing the new CI jobs https://review.opendev.org/679106 | 17:48 |
*** mmethot has quit IRC | 17:58 | |
*** zbitter has joined #openstack-ironic | 17:58 | |
*** mmethot has joined #openstack-ironic | 17:58 | |
*** zaneb has quit IRC | 17:58 | |
*** priteau has quit IRC | 18:12 | |
*** ociuhandu has joined #openstack-ironic | 18:12 | |
*** igordc has quit IRC | 18:23 | |
*** trandles has quit IRC | 18:57 | |
*** cdearborn has joined #openstack-ironic | 19:12 | |
*** ociuhandu has quit IRC | 19:14 | |
*** jcoufal_ has joined #openstack-ironic | 19:57 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Change [deploy]/default_boot_option to local https://review.opendev.org/492114 | 19:57 |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Fix get_boot_option logic https://review.opendev.org/679132 | 19:57 |
*** jcoufal has quit IRC | 19:59 | |
TheJulia | dtantsur|afk: ^^^ okay, that _should_ fix software raid local boot when we change the boot option in the following patch. I'll edit the commit message of the tempest test fix. | 20:01 |
openstackgerrit | Julia Kreger proposed openstack/ironic-tempest-plugin master: Enable boot_option to be specified https://review.opendev.org/678633 | 20:02 |
jtwill98 | Does anyone have experience using the Virtual Redfish Emulator who I could talk with? | 20:02 |
*** gyee has quit IRC | 20:05 | |
TheJulia | jtwill98: well, we use it in our CI, so maybe ask your questions and someone might know. If all else fails etingof may be a good person to chat with | 20:06 |
jtwill98 | Well, I am trying to use the sushy-emulator and can see the Systems, however all efforts to see Managers and Chassis are failing. If I use static with mockups I see them fine. | 20:09 |
jtwill98 | In fact the only data items under my ServiceRoot is "Systems"/ | 20:13 |
TheJulia | I suspect that is because it is not implemted in the dynamic emulator since that is used in our CI for testing and I think has very exclusive paths for manager urls. | 20:14 |
jtwill98 | I suspect that may be the case. How does one follow the instructions in the "virtual media boot" section found at URL: https://docs.openstack.org/sushy-tools/latest/user/dynamic-emulator.html. | 20:19 |
jtwill98 | It looks like you need Managers to map images to virtual media. | 20:20 |
TheJulia | are you installing a release or are you installing directly from the master branch of git? | 20:20 |
jtwill98 | I installed a release but would be willing to use the master branch, if required. | 20:21 |
TheJulia | jtwill98: well, that would be why your having issues | 20:21 |
TheJulia | latest documentation is from master branch | 20:21 |
jtwill98 | sushy-tools .60 if I recall correctly. | 20:21 |
TheJulia | What your searching for merged on August 6th, thus hasn't been released yet | 20:22 |
jtwill98 | using pip install. | 20:22 |
*** gyee has joined #openstack-ironic | 20:23 | |
TheJulia | https://review.opendev.org/#/c/656792 and https://review.opendev.org/#/c/656938 | 20:23 |
patchbot | patch 656792 - sushy-tools - Add Virtual Media support (MERGED) - 26 patch sets | 20:23 |
patchbot | patch 656938 - sushy-tools - Add `get/set_boot_image` to libvirt driver (MERGED) - 28 patch sets | 20:23 |
jtwill98 | OK, at least I now something new to try. | 20:24 |
TheJulia | jtwill98: enjoy! | 20:24 |
jtwill98 | Is there an easy way to apply the patches? | 20:24 |
TheJulia | jtwill98: what patches are you trying to apply? | 20:29 |
TheJulia | jtwill98: the ones I specified? There are more to just those, you'll need to just pull master down and install it | 20:30 |
jtwill98 | I'm good. | 20:34 |
jtwill98 | thanks Julie | 20:34 |
jtwill98 | Thanks Julia! | 20:35 |
*** trandles has joined #openstack-ironic | 20:41 | |
openstackgerrit | Merged openstack/ironic master: Follow-up for deploy steps for Redfish BIOS interface https://review.opendev.org/677695 | 20:42 |
*** jcoufal_ has quit IRC | 20:44 | |
*** jcoufal has joined #openstack-ironic | 20:45 | |
*** jcoufal has quit IRC | 20:50 | |
*** bdodd has quit IRC | 20:51 | |
*** trandles has quit IRC | 20:56 | |
*** belmoreira has joined #openstack-ironic | 20:59 | |
*** patchbot has quit IRC | 20:59 | |
*** patchbot has joined #openstack-ironic | 21:00 | |
*** Goneri has quit IRC | 21:15 | |
*** belmoreira has quit IRC | 21:16 | |
etingof | jtwill98, sorry for being late to the party... with latest master both Managers and Chassis are indeed supported | 21:22 |
*** belmoreira has joined #openstack-ironic | 21:25 | |
*** jtwill98 has quit IRC | 21:37 | |
openstackgerrit | Ilya Etingof proposed openstack/sushy-tools master: Do not reuse sqlite connection https://review.opendev.org/678797 | 22:02 |
*** belmoreira has quit IRC | 22:11 | |
*** patchbot has quit IRC | 22:38 | |
*** patchbot has joined #openstack-ironic | 22:48 | |
*** rcernin has joined #openstack-ironic | 23:21 | |
*** rh-jelabarre has quit IRC | 23:42 | |
*** sthussey has quit IRC | 23:47 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!