*** Lucas_Gray has quit IRC | 00:11 | |
*** dsneddon has joined #openstack-ironic | 00:47 | |
*** dsneddon has quit IRC | 00:52 | |
*** mkowalski_ has joined #openstack-ironic | 01:28 | |
*** mkowalski__ has quit IRC | 01:30 | |
*** dsneddon has joined #openstack-ironic | 02:48 | |
*** dsneddon has quit IRC | 02:53 | |
*** mkrai has joined #openstack-ironic | 03:19 | |
*** ricolin has joined #openstack-ironic | 03:20 | |
*** dhellmann has quit IRC | 04:25 | |
*** dhellmann has joined #openstack-ironic | 04:30 | |
*** mkowalski_ has quit IRC | 04:35 | |
*** mkowalski_ has joined #openstack-ironic | 04:35 | |
*** mkowalski_ has quit IRC | 04:41 | |
*** monica_pardhi has joined #openstack-ironic | 04:44 | |
*** dsneddon has joined #openstack-ironic | 04:49 | |
*** dsneddon has quit IRC | 04:54 | |
*** whoami-rajat has joined #openstack-ironic | 05:11 | |
*** iurygregory has joined #openstack-ironic | 05:12 | |
iurygregory | good morning Ironic | 05:12 |
---|---|---|
rpioso | Good morning iurygregory | 05:13 |
iurygregory | rpioso, o/ | 05:13 |
*** rachit7 has joined #openstack-ironic | 05:14 | |
openstackgerrit | Steve Baker proposed openstack/ironic master: Allow vendor_data to be included in a configdrive dict https://review.opendev.org/683847 | 05:22 |
openstackgerrit | Richard G. Pioso proposed openstack/ironic master: Add Redfish vmedia boot interface to idrac HW type https://review.opendev.org/672498 | 05:41 |
*** yolanda has quit IRC | 05:42 | |
*** yolanda has joined #openstack-ironic | 05:49 | |
openstackgerrit | Richard G. Pioso proposed openstack/ironic master: Add Redfish vmedia boot interface to idrac HW type https://review.opendev.org/672498 | 06:04 |
rpioso | etingof: Please take a look at ^^^ | 06:05 |
*** rpioso is now known as rpioso|afk | 06:05 | |
openstackgerrit | RotanChen proposed openstack/ironic-inspector master: Remove unused import statement. https://review.opendev.org/682330 | 06:05 |
*** yolanda has quit IRC | 06:22 | |
*** TxGirlGeek has joined #openstack-ironic | 06:23 | |
arne_wiebalck | Good morning, ironic! | 06:38 |
iurygregory | morning arne_wiebalck o/ | 06:38 |
arne_wiebalck | hey iurygregory o/ | 06:39 |
*** rpittau|afk is now known as rpittau | 06:40 | |
rpittau | good morning ironic! o/ | 06:40 |
iurygregory | morning rpittau o/ | 06:41 |
rpittau | hey iurygregory :) | 06:41 |
arne_wiebalck | hey rpittau o/ | 06:41 |
rpittau | hi arne_wiebalck :) | 06:41 |
*** TxGirlGeek has quit IRC | 06:42 | |
*** dsneddon has joined #openstack-ironic | 06:50 | |
*** dsneddon has quit IRC | 06:55 | |
arne_wiebalck | When adding new controllers, I noticed that the conductor affinity is updated on most but not all nodes, some nodes keep the affinity to deleted controllers (and I update them manually) ... can someone remind me why for some node the affinity is not re-calculated? | 06:57 |
*** tesseract has joined #openstack-ironic | 07:00 | |
*** rcernin has quit IRC | 07:04 | |
*** alexmcleod has joined #openstack-ironic | 07:13 | |
*** kiseok7 has quit IRC | 07:18 | |
*** monica_pardhi has quit IRC | 07:22 | |
mkrai | good morning ironic! | 07:31 |
mkrai | arne_wiebalck, rpittau iurygregory hi o/ | 07:31 |
*** e0ne has joined #openstack-ironic | 07:34 | |
mkrai | I was seeing conductor do_node_clean code and found that there is two power,network validate call. Is it intended? | 07:34 |
mkrai | https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L1177-L1178 and https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L1316-L1317 | 07:35 |
*** e0ne has quit IRC | 07:35 | |
*** pcaruana has joined #openstack-ironic | 07:41 | |
rpittau | mkrai: have you seen this comment? https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L1171 | 07:44 |
mkrai | rpittau, Yes but the second validate in _do_node_clean is also immediate. I mean no actual processing | 07:46 |
*** jtomasek has quit IRC | 07:47 | |
*** jtomasek has joined #openstack-ironic | 07:48 | |
rpittau | mkrai: sorry, I'm not sure I follow, the power and network validate are done as first steps, before starting anu processing | 07:48 |
etingof | rpioso|afk, o/ reviewed, LGTM | 07:48 |
rpittau | any* | 07:48 |
etingof | rpioso|afk, I accidentally posted my older comments to bdodd's review as well. you can just ignore those.. | 07:49 |
mkrai | rpittau, https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L1289-L1311, is it actual processing? | 07:50 |
rpittau | mkrai: in terms of clean steps, no, that is preparation to cleaning. | 07:53 |
mkrai | rpittau, Ok thanks | 07:54 |
rpittau | mkrai: not even preparation actually, we're just deciding if we're going to clean or not a node | 07:55 |
mkrai | rpittau, Yes and it isn't a real processing | 07:56 |
mkrai | Just some condition check | 07:56 |
rpittau | mkrai: correct | 07:56 |
mkrai | That's why the validation seemed redundant to me. | 07:56 |
mkrai | Does it makes sense? | 07:56 |
*** toabctl has joined #openstack-ironic | 08:02 | |
rpittau | mkrai: I understand why it looks redundant, it is indeed, although I think it's necessary, something could change betweem method calls | 08:02 |
rpittau | ok, probably not network :) | 08:03 |
*** yolanda has joined #openstack-ironic | 08:09 | |
*** e0ne has joined #openstack-ironic | 08:18 | |
*** iurygregory has quit IRC | 08:19 | |
*** gregoryo has joined #openstack-ironic | 08:21 | |
openstackgerrit | pengyuesheng proposed openstack/python-ironic-inspector-client master: Update the constraints url https://review.opendev.org/683882 | 08:44 |
*** rcernin has joined #openstack-ironic | 08:45 | |
*** iurygregory has joined #openstack-ironic | 08:45 | |
*** derekh has joined #openstack-ironic | 08:49 | |
*** yolanda has quit IRC | 08:50 | |
*** dsneddon has joined #openstack-ironic | 08:51 | |
*** gregoryo has quit IRC | 08:55 | |
*** dsneddon has quit IRC | 08:55 | |
*** ociuhandu has joined #openstack-ironic | 09:05 | |
*** dtantsur|afk is now known as dtantsur | 09:07 | |
dtantsur | morning ironic | 09:07 |
dtantsur | mgoddard: hey, sorry, I may have to skip the 2nd part (even 2/3) of the meeting today. could you take it over once I leave? | 09:08 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/stein: devstack: wait for conductor to start and register itself https://review.opendev.org/683900 | 09:09 |
*** mkowalski_ has joined #openstack-ironic | 09:09 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/rocky: devstack: wait for conductor to start and register itself https://review.opendev.org/683901 | 09:09 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic stable/queens: devstack: wait for conductor to start and register itself https://review.opendev.org/683902 | 09:10 |
iurygregory | morning dtantsur | 09:12 |
mkrai | rpittau, Sorry I was away, what could change? | 09:15 |
mkrai | morning dtantsur | 09:15 |
*** bobmel has joined #openstack-ironic | 09:24 | |
*** bobmel has quit IRC | 09:25 | |
*** Lucas_Gray has joined #openstack-ironic | 09:30 | |
mgoddard | dtantsur: sure, no problem. If you give me your IRC credentials I could impersonate you, and maybe no one will ever know :p (except meeting quality will quickly drop) | 09:40 |
*** dsneddon has joined #openstack-ironic | 09:40 | |
dtantsur | :D | 09:40 |
*** Lucas_Gray has quit IRC | 09:46 | |
*** Lucas_Gray has joined #openstack-ironic | 09:48 | |
*** ricolin has quit IRC | 09:48 | |
*** ociuhandu has quit IRC | 09:59 | |
*** ociuhandu has joined #openstack-ironic | 10:00 | |
dtantsur | xXraphXx: hey, around re https://review.opendev.org/#/c/683421/ ? | 10:04 |
patchbot | patch 683421 - ironic-python-agent - Revert "Software raid: mbr/gpt partition table alt... - 3 patch sets | 10:04 |
dtantsur | I managed to figure out from IRC scrollback why we are doing it. But to have an understandable git history, we need to provide a reasoning in the commit message. | 10:05 |
dtantsur | if you could update the commit message, I could approve right away | 10:05 |
*** bobmel has joined #openstack-ironic | 10:05 | |
xXraphXx | dtantsur, hi, sure doing | 10:05 |
dtantsur | thx! | 10:05 |
*** ociuhandu has quit IRC | 10:06 | |
*** rachit7 has quit IRC | 10:18 | |
openstackgerrit | raphael.glon proposed openstack/ironic-python-agent master: Revert "Software raid: mbr/gpt partition table alternative" https://review.opendev.org/683421 | 10:21 |
xXraphXx | dtantsur, done, would the commit message be ok ? | 10:22 |
openstackgerrit | raphael.glon proposed openstack/ironic-python-agent master: Revert "Software raid: mbr/gpt partition table alternative" https://review.opendev.org/683421 | 10:26 |
*** rachit7 has joined #openstack-ironic | 10:39 | |
*** ociuhandu has joined #openstack-ironic | 10:41 | |
*** kaifeng has joined #openstack-ironic | 10:42 | |
kaifeng | morning ironic o/ | 10:44 |
*** ociuhandu has quit IRC | 10:46 | |
*** Lucas_Gray has quit IRC | 10:46 | |
*** Lucas_Gray has joined #openstack-ironic | 10:48 | |
*** ociuhandu has joined #openstack-ironic | 10:50 | |
*** ociuhandu has quit IRC | 10:54 | |
dtantsur | xXraphXx: done (sorry, was on lunch) | 10:57 |
dtantsur | morning (evening?) kaifeng :) | 10:57 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: Explicitly tear down software RAID after testing it https://review.opendev.org/683921 | 11:08 |
dtantsur | this ^^ may help with the standalone job | 11:08 |
openstackgerrit | Merged openstack/ironic master: Don't resume deployment or cleaning on heartbeat when polling https://review.opendev.org/681272 | 11:09 |
kaifeng | dtantsur: yep | 11:17 |
*** dsneddon has quit IRC | 11:17 | |
kaifeng | dtantsur: there is a fix may help as well https://review.opendev.org/682270 | 11:17 |
patchbot | patch 682270 - ironic-tempest-plugin - Fix an issue when nodes are not available in stand... - 1 patch set | 11:17 |
*** ociuhandu has joined #openstack-ironic | 11:19 | |
*** ociuhandu has quit IRC | 11:23 | |
*** lucasagomes has joined #openstack-ironic | 11:31 | |
*** rh-jelabarre has joined #openstack-ironic | 11:31 | |
dtantsur | kaifeng: yep, +2 | 11:32 |
kaifeng | dtantsur: just a look on your patch, i don't think the state transition works this way, I wonder why the delete_node is set to False? | 11:34 |
* kaifeng insert add/take/whatever into above words | 11:35 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-tempest-plugin master: Explicitly tear down software RAID after testing it https://review.opendev.org/683921 | 11:37 |
dtantsur | kaifeng: delete_node is set to False to prevent tearDown from failing (it will try unprovision the already unprovisioned node) | 11:38 |
*** bobmel has quit IRC | 11:40 | |
kaifeng | so sometimes terminate_node is only used for vif cleanup right? | 11:42 |
openstackgerrit | Merged openstack/ironic master: Add iDRAC RAID deploy steps https://review.opendev.org/641731 | 11:43 |
kaifeng | oh and I left a suggestion on the patch of conductor startup checking, i think we could use the /v1/conductors for the same purpose | 11:45 |
*** rh-jelabarre has quit IRC | 11:45 | |
*** rh-jelabarre has joined #openstack-ironic | 11:45 | |
kaifeng | for a multinode environment, checking driver list is not sufficient | 11:46 |
*** dsneddon has joined #openstack-ironic | 11:47 | |
*** rcernin has quit IRC | 11:49 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent-builder master: DIB: do not use /tmp for a virtualenv in the source build https://review.opendev.org/683929 | 11:50 |
dtantsur | kaifeng: yep, we could. I can try following up with it, but I may not have time in the near future. | 11:51 |
kaifeng | dtantsur: np, anytime :) | 11:52 |
kaifeng | i gotta go now, see you later o/ | 11:53 |
*** kaifeng has quit IRC | 11:53 | |
*** bfournie has quit IRC | 11:57 | |
*** ociuhandu has joined #openstack-ironic | 11:57 | |
*** goldyfruit_ has quit IRC | 11:57 | |
*** ociuhandu has quit IRC | 11:58 | |
*** ociuhandu has joined #openstack-ironic | 11:59 | |
*** iurygregory has quit IRC | 12:01 | |
TheJulia | dtantsur: did you see my note regarding the metalsmith job only ever using network booting? | 12:17 |
TheJulia | Which likely explains some of the confusion. | 12:17 |
*** dsneddon has quit IRC | 12:22 | |
openstackgerrit | Dmitry Tantsur proposed openstack/python-ironicclient master: Fix TypeError when endpoint cannot be detected from a session https://review.opendev.org/683944 | 12:27 |
*** rpioso|afk is now known as rpioso | 12:28 | |
dtantsur | TheJulia: the centos job should (at least used to) use local booting | 12:28 |
dtantsur | I think it falls back to networking booting because we don't configure the boot loader | 12:29 |
*** rloo has joined #openstack-ironic | 12:29 | |
dtantsur | TheJulia: re bifrost failure: looking. the above ironicclient patch is one of the parts of it. | 12:29 |
dtantsur | but I suspect ironicclient 3.0.0 breaks no-auth :( | 12:30 |
rpioso | etingof: Thank you! I'll revise the commit message and release note to mention the introduction of the new dependency. | 12:30 |
*** rcernin has joined #openstack-ironic | 12:33 | |
dtantsur | oh wow, SessionClient is SUCH A MESS | 12:33 |
*** dsneddon has joined #openstack-ironic | 12:33 | |
etingof | rpioso, what should we do with our new drivers-requirements dependency that apparently fails a few CI jobs? | 12:36 |
etingof | rpioso, comment it out or release sushy-oem-idrac on pypi (if that helps at all?) | 12:37 |
*** dsneddon has quit IRC | 12:38 | |
rpioso | etingof: Good question. The plan is to add that repo to opendev's 'x' namespace. That will make it available on PyPI. However, that'll take some time. | 12:41 |
rpioso | etingof: Could it be commented out or left out and later added? Or would that not comply with stable branch policy? | 12:42 |
etingof | dtantsur, might know ^ | 12:44 |
mbuil | guys, I am trying ironic stand-alone and suddenly, when executing "openstack baremetal node list" I get "Missing value auth-url required for auth plugin password". Is it not enough with export OS_TOKEN=fake-token and export OS_URL=http://localhost:6385/? Any hint? thanks! | 12:44 |
dtantsur | etingof, rpioso, we cannot merge code that is not usable by end users, if that's your question | 12:45 |
dtantsur | if there is a dependency, it has to be installable | 12:45 |
etingof | rpioso, perhaps you've researched already what needs to be done to move the library to `x`...? | 12:46 |
*** bfournie has joined #openstack-ironic | 12:47 | |
rpioso | etingof: I have. | 12:48 |
* rpioso digs up the link to the document. | 12:49 | |
*** jcoufal has joined #openstack-ironic | 12:50 | |
openstackgerrit | Ilya Etingof proposed openstack/ironic master: Add indicator management to redfish hw type https://review.opendev.org/652740 | 12:51 |
openstackgerrit | Dmitry Tantsur proposed openstack/python-ironicclient master: Fix TypeError when using endpoint_override with SessionClient https://review.opendev.org/683944 | 12:51 |
dtantsur | TheJulia: this ^^ may be the fix | 12:51 |
*** iurygregory has joined #openstack-ironic | 12:51 | |
dtantsur | etingof: some parts of https://docs.openstack.org/infra/manual/creators.html | 12:51 |
openstackgerrit | Ilya Etingof proposed openstack/ironic master: Add indicators REST API endpoints https://review.opendev.org/651785 | 12:51 |
mbuil | found the solution ==> https://github.com/openstack/bifrost/commit/6df83e0bc56d519bb6d37190228e0977d89da6a3 | 12:54 |
rpioso | etingof: https://docs.openstack.org/infra/manual/creators.html | 12:55 |
dtantsur | mbuil: right, OS_TOKEN was removed from openstackclient | 12:55 |
mbuil | dtantsur: I'll update https://docs.openstack.org/ironic/latest/install/standalone.html | 12:57 |
dtantsur | thanks! | 12:57 |
openstackgerrit | Merged openstack/ironic master: Allow retrying PXE boot if it takes too long https://review.opendev.org/683127 | 12:58 |
openstackgerrit | Rachit Kapadia proposed openstack/ironic master: DRAC: Fix a bug for delete_config with multiple controllers https://review.opendev.org/680660 | 12:59 |
*** ricolin has joined #openstack-ironic | 12:59 | |
*** e0ne_ has joined #openstack-ironic | 12:59 | |
*** goldyfruit_ has joined #openstack-ironic | 13:01 | |
*** e0ne has quit IRC | 13:01 | |
etingof | rpioso, on the bright side, sushy-oem-idrac is structured similarly to sushy i.e. it should hopefully be easy to adopt as an OS project... | 13:01 |
etingof | rpioso, if you are short of cycles, I can probable pursue this | 13:02 |
rpioso | dtantsur, etingof: Could it be completed in time for Train? | 13:03 |
dtantsur | rpioso: feature work is essentially out of question for Train | 13:04 |
rpioso | etingof: I'll pursue it and of course keep you looped in. | 13:06 |
rpioso | etingof: I'd like to explore your review and code comments about needing to reboot the server to set the boot device. It appears to describe two reboots. In my experience, only one reboot has been required. During the next reboot, the setting is applied and utilized. Would later today work? | 13:08 |
*** ociuhandu has quit IRC | 13:10 | |
*** ociuhandu has joined #openstack-ironic | 13:10 | |
openstackgerrit | Dmitry Tantsur proposed openstack/python-ironicclient master: Remove the requirement on either endpoint_override or os_ironic_api_version https://review.opendev.org/683952 | 13:10 |
etingof | rpioso, are you saying that it is possible to submit two iDRAC jobs in a row (set-boot-device and set-boot-mode) without a reboot in-between? | 13:11 |
*** e0ne has joined #openstack-ironic | 13:12 | |
*** e0ne_ has quit IRC | 13:12 | |
openstackgerrit | Manuel Buil proposed openstack/ironic master: Update env. variables in the documentation https://review.opendev.org/683953 | 13:13 |
rpioso | etingof: Let's start simple by keeping boot mode out of the discussion. | 13:14 |
*** dsneddon has joined #openstack-ironic | 13:14 | |
rpioso | etingof: But later today :-) | 13:14 |
rpioso | etingof: If that works for you. | 13:14 |
etingof | rpioso, sure! | 13:15 |
*** ociuhandu has quit IRC | 13:17 | |
*** mkrai has quit IRC | 13:18 | |
openstackgerrit | Dmitry Tantsur proposed openstack/python-ironicclient master: Make the dependency on oslo.config explicit https://review.opendev.org/683955 | 13:18 |
openstackgerrit | Dmitry Tantsur proposed openstack/python-ironicclient master: Make the dependency on oslo.config explicit https://review.opendev.org/683955 | 13:20 |
*** mkrai has joined #openstack-ironic | 13:21 | |
*** beekneemech is now known as bnemec | 13:24 | |
*** sthussey has joined #openstack-ironic | 13:26 | |
jroll | morning ironic | 13:27 |
dtantsur | morning jroll | 13:27 |
*** ociuhandu has joined #openstack-ironic | 13:27 | |
rpittau | hey jroll :) | 13:28 |
*** goldyfruit___ has joined #openstack-ironic | 13:30 | |
openstackgerrit | Dmitry Tantsur proposed openstack/python-ironicclient master: Fix TypeError when using endpoint_override with SessionClient https://review.opendev.org/683944 | 13:31 |
*** ociuhandu has quit IRC | 13:33 | |
*** goldyfruit_ has quit IRC | 13:33 | |
*** whoami-rajat has quit IRC | 13:33 | |
arne_wiebalck | jroll: o/ | 13:36 |
*** yolanda has joined #openstack-ironic | 13:44 | |
*** rcernin has quit IRC | 13:45 | |
*** diga has joined #openstack-ironic | 13:45 | |
*** dsneddon has quit IRC | 13:46 | |
*** bdodd has joined #openstack-ironic | 13:50 | |
*** yolanda has quit IRC | 13:52 | |
*** dsneddon has joined #openstack-ironic | 13:54 | |
*** yolanda has joined #openstack-ironic | 13:56 | |
*** dsneddon has quit IRC | 14:00 | |
*** jcoufal_ has joined #openstack-ironic | 14:00 | |
*** ociuhandu has joined #openstack-ironic | 14:00 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic master: Document PXE retries https://review.opendev.org/683967 | 14:02 |
*** yolanda has quit IRC | 14:03 | |
*** jcoufal has quit IRC | 14:04 | |
*** tzumainn has joined #openstack-ironic | 14:04 | |
*** ociuhandu has quit IRC | 14:04 | |
mbuil | guys, our ironic-conductor logs show this when we try to do the inspection: "ConnectFailure: Unable to establish connection to http://localhost:5050: HTTPConnectionPool(host='localhost', port=5050): Max retries exceeded with url: / (Caused by NewConnectionError('<urllib3.connection.HTTPConnection object at 0x7ff1b52a7c90>: Failed to establish a new connection: [Errno 111] ECONNREFUSED',))". I can see that a process should be | 14:05 |
mbuil | listening on port 5050 but that port is not open. Any hint what might be the problem? | 14:05 |
openstackgerrit | Riccardo Pittau proposed openstack/networking-baremetal master: Use openstacksdk for ironic connection https://review.opendev.org/683971 | 14:12 |
*** dsneddon has joined #openstack-ironic | 14:16 | |
openstackgerrit | Gaƫtan Trellu proposed openstack/python-ironicclient master: Fix usage of --owner filter during node list https://review.opendev.org/682666 | 14:17 |
dtantsur | mbuil: maybe it's not listening on localhost? | 14:18 |
*** yolanda has joined #openstack-ironic | 14:19 | |
*** dsneddon has quit IRC | 14:21 | |
*** iurygregory has quit IRC | 14:23 | |
*** jtwill98 has joined #openstack-ironic | 14:29 | |
*** TxGirlGeek has joined #openstack-ironic | 14:35 | |
mbuil | dtantsur: by reading the conf. it seems the inspector should be listening to port 5050. Should I see through 'ps' any process called inspector? Or what is the name of the process? Maybe I can try to start it manually and check if there are any problems | 14:37 |
dtantsur | mbuil: usually, ironic-inspector. | 14:37 |
mbuil | dtantsur: thanks. That throws "ERROR oslo.messaging._drivers.impl_rabbit [-] Connection failed: [Errno 111] ECONNREFUSED (retrying in 2.0 seconds): error: [Errno 111] ECONNREFUSED" | 14:38 |
rpittau | mbuil: that's a problem with rabbitmq (or any other message broker you're using) | 14:40 |
rpittau | mbuil: you should check that rabbitmq is running and the rabbit conf is correct in your setup | 14:41 |
mbuil | rpittau: is rabbitmq also used in bifrost with standalone ironic? | 14:42 |
rpittau | mbuil: it recently switched to json-rpc by default | 14:44 |
openstackgerrit | Merged openstack/ironic-python-agent-builder master: DIB: rename files and paths to match the element name https://review.opendev.org/682647 | 14:48 |
openstackgerrit | Merged openstack/ironic-python-agent-builder master: Correct variables in the POST job image_name -> image_type https://review.opendev.org/682899 | 14:48 |
*** S4ren has joined #openstack-ironic | 14:49 | |
*** ociuhandu has joined #openstack-ironic | 14:51 | |
mbuil | rpittau: I have rpc_transport = json-rpc in my ironic.conf. Do you have the command to start ironic-inspector handy? I'd like to try | 14:51 |
*** kaifeng has joined #openstack-ironic | 14:52 | |
openstackgerrit | Mark Goddard proposed openstack/ironic master: Release notes cleanup for 13.0.0 (mk2) https://review.opendev.org/683983 | 14:53 |
mbuil | rpittau: I still get the rabbitmq error when executing "ironic-inspector --config-file /etc/ironic/ironic.conf" even if ironic.conf has "rpc_transport = json-rpc", any idea? Maybe I need another parameter there? | 14:54 |
*** yolanda has quit IRC | 14:54 | |
*** dsneddon has joined #openstack-ironic | 14:55 | |
*** nick_kar has joined #openstack-ironic | 14:56 | |
*** ociuhandu has quit IRC | 14:57 | |
*** mkowalski_ has quit IRC | 14:58 | |
kaifeng | mbuil: it seems you are feeding inspector service with ironic configuration file | 14:58 |
rpittau | mbuil: out of curiosity, which version of ironic-inspector are you using? | 14:58 |
rpittau | and yes, also that kaifeng :D | 14:58 |
kaifeng | hmm, i never tried that, what the purpose behind this? | 14:59 |
dtantsur | #startmeeting ironic | 15:00 |
openstack | Meeting started Mon Sep 23 15:00:02 2019 UTC and is due to finish in 60 minutes. The chair is dtantsur. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:00 |
openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:00 |
*** openstack changes topic to " (Meeting topic: ironic)" | 15:00 | |
openstack | The meeting name has been set to 'ironic' | 15:00 |
dtantsur | hey all, meeting time! | 15:00 |
etingof | o/ | 15:00 |
rpittau | o/ | 15:00 |
rloo | o/ | 15:00 |
kaifeng | o/ | 15:00 |
bdodd | o/ | 15:00 |
dtantsur | I'll have to run quite soon, will pass my chair rights to mgoddard then | 15:00 |
dtantsur | #chair mgoddard | 15:00 |
openstack | Current chairs: dtantsur mgoddard | 15:00 |
jroll | \o | 15:00 |
dtantsur | the empty agenda is located at: | 15:00 |
dtantsur | #link https://wiki.openstack.org/wiki/Meetings/Ironic | 15:00 |
mgoddard | lo | 15:00 |
rpioso | o/ | 15:00 |
dtantsur | #topic Announcements / Reminder | 15:01 |
*** openstack changes topic to "Announcements / Reminder (Meeting topic: ironic)" | 15:01 | |
dtantsur | #info This week we must have stable/train for all our branched project | 15:01 |
*** cdearborn has joined #openstack-ironic | 15:01 | |
dtantsur | #link https://releases.openstack.org/train/schedule.html | 15:01 |
cdearborn | o/ | 15:01 |
arne_wiebalck | o/ | 15:01 |
dtantsur | #info We're in feature freeze, please hold on approving features until the branches are created | 15:01 |
rpittau | it's release-3 week | 15:02 |
openstackgerrit | Mark Goddard proposed openstack/ironic master: Update release mappings for Train https://review.opendev.org/683984 | 15:02 |
dtantsur | good call ^^^ | 15:02 |
dtantsur | anything else to announce? | 15:02 |
mgoddard | semi-plug, semi-useful info, I wrote a blog about bios, raid, scheduling, deploy steps and deploy templates: http://www.stackhpc.com/bespoke-bare-metal.html | 15:02 |
mgoddard | based on the talk I gave in denver | 15:03 |
mgoddard | not really an announcement, but hey :) | 15:03 |
dtantsur | #link http://www.stackhpc.com/bespoke-bare-metal.html | 15:03 |
dtantsur | why not :) | 15:03 |
arne_wiebalck | nice! | 15:03 |
dtantsur | well, kind of an announcement: our CI feels better, please try not to break it :) | 15:04 |
dtantsur | any ideas for further improvements are very welcome | 15:04 |
*** ociuhandu has joined #openstack-ironic | 15:04 | |
dtantsur | anything else? | 15:04 |
*** whoami-rajat has joined #openstack-ironic | 15:04 | |
*** ociuhandu has quit IRC | 15:04 | |
rloo | dtantsur: we may get to it, but if we don't. before you take off. please let us know what you think needs to land before we cut a train rc1. | 15:05 |
dtantsur | I'm not aware of anything that MUST land | 15:05 |
rloo | also, I don't know if we did this, but I want to congratulate (and THANK!) TheJulia for being Ussuri PTL. | 15:05 |
*** ociuhandu has joined #openstack-ironic | 15:05 | |
rpittau | and when we are cutting train RC1 :) | 15:05 |
*** stendulker has joined #openstack-ironic | 15:05 | |
mgoddard | rloo: you could #thank | 15:05 |
*** pcaruana has quit IRC | 15:05 | |
dtantsur | however, I have a few fixes for ironicclient, that seem quite urgent | 15:05 |
*** w14161_1 has joined #openstack-ironic | 15:06 | |
dtantsur | rpittau: for us RC1 == stable/train | 15:06 |
jroll | rloo: +1000 | 15:06 |
dtantsur | we don't do proper RCs, so the last release before branching is an RC | 15:06 |
rloo | dtantsur: i suspect rpittau wants to know what day we'll be cutting that. | 15:06 |
dtantsur | Thursday the latest | 15:06 |
mkrai | o/ sorry I am late | 15:06 |
rpittau | rloo: yeah | 15:06 |
rpittau | dtantsur: thanks | 15:06 |
*** dsneddon has quit IRC | 15:06 | |
dtantsur | doing it later may cause us troubles if grenade gets branched | 15:06 |
dtantsur | actually, I'm doing to check tomorrow if we can release or not | 15:07 |
rloo | #thank TheJulia for volunteering to be the PTL for ussuri | 15:07 |
dtantsur | so consider it happening at any point | 15:07 |
dtantsur | unless somebody brings up anything blocking | 15:07 |
dtantsur | #topic Review subteam status reports (capped at ten minutes) | 15:07 |
*** openstack changes topic to "Review subteam status reports (capped at ten minutes) (Meeting topic: ironic)" | 15:07 | |
dtantsur | #link https://etherpad.openstack.org/p/IronicWhiteBoard line 266 | 15:07 |
dtantsur | I don't expect a lot of changes, but there's been some progress around IPA-builder and RAID | 15:07 |
*** ociuhandu has quit IRC | 15:08 | |
openstackgerrit | Merged openstack/ironic stable/queens: devstack: wait for conductor to start and register itself https://review.opendev.org/683902 | 15:08 |
openstackgerrit | Merged openstack/ironic-inspector master: Docs for ironic inspector with tooz https://review.opendev.org/674542 | 15:08 |
openstackgerrit | Merged openstack/ironic-python-agent master: Revert "Software raid: mbr/gpt partition table alternative" https://review.opendev.org/683421 | 15:08 |
*** ociuhandu has joined #openstack-ironic | 15:08 | |
rloo | ooo. reduced rfe bugs by 5 :) | 15:09 |
rloo | well,not bugs. stories i guess. | 15:09 |
dtantsur | yep :) | 15:09 |
mkrai | https://review.opendev.org/#/c/665835/ do we need this one for train? | 15:09 |
patchbot | patch 665835 - ironic - Block port deletions where vif is present - 3 patch sets | 15:09 |
*** w14161_1 is now known as jrwang | 15:10 | |
dtantsur | I'm not sure how I feel about a behavior change after feature freeze | 15:10 |
dtantsur | is it critical for anything mkrai? | 15:10 |
rloo | wrt Software RAID, L361. The 2 patches. I think we're punting those to U. | 15:11 |
dtantsur | seems so | 15:12 |
arne_wiebalck | rloo: yes | 15:12 |
mkrai | dtantsur, Not urgent if team thinks so. | 15:12 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-python-agent-builder master: DIB: do not use /tmp for a virtualenv in the source build https://review.opendev.org/683929 | 15:12 |
*** ociuhandu has quit IRC | 15:13 | |
rloo | am going to delete the "power state callbacks to Nova" section (L369ish) | 15:13 |
dtantsur | yep | 15:13 |
dtantsur | are we still going to land https://review.opendev.org/#/c/651785/ in train? | 15:13 |
patchbot | patch 651785 - ironic - Add indicators REST API endpoints - 16 patch sets | 15:13 |
arne_wiebalck | shall we remove all the "old" s/w RAID lines 350-363 ? | 15:13 |
dtantsur | mgoddard: this ^^ may affect release_mappings | 15:14 |
dtantsur | arne_wiebalck: likely | 15:14 |
mgoddard | dtantsur: I didn't think we were aiming for landing indicators? | 15:14 |
*** TxGirlGeek has quit IRC | 15:14 | |
dtantsur | mgoddard: it's probably a bit too late, yes | 15:14 |
mgoddard | hasn't had a +2 yet | 15:14 |
rloo | Yay, we now have pdf docs! | 15:15 |
dtantsur | we have landed https://review.opendev.org/#/c/649675/, but it's not user-visible | 15:15 |
patchbot | patch 649675 - ironic - Add indicator management harness to ManagementInte... (MERGED) - 13 patch sets | 15:15 |
dtantsur | let's hold on the API until we release then | 15:15 |
dtantsur | anything else on the statuses? | 15:16 |
mgoddard | I guess we'd need the redfish implementation patch too to make the API useful? | 15:16 |
dtantsur | mgoddard: they're ordered correctly now, the API one goes last | 15:16 |
mgoddard | right | 15:17 |
kaifeng | rloo: it doesn't look nice for some projects :) | 15:17 |
rloo | kaifeng: well... as long as the info is there, that's better than nothing. now people can pretty-ify them :D | 15:17 |
dtantsur | kaifeng: PDF docs? well, now people may start improving them | 15:17 |
kaifeng | i expect there would some common tooling for the purpose | 15:18 |
rloo | and many thanks kaifeng, for working on the pdfs! | 15:18 |
dtantsur | kaifeng++ | 15:18 |
rpioso | Thank you for reviewing the idrac changes. Two of them have merged \o/ | 15:18 |
dtantsur | np | 15:19 |
dtantsur | are we ready to move on? | 15:19 |
kaifeng | np :) | 15:19 |
mgoddard | kaifeng++, especially since I shamelessly stole the patch for kolla :D | 15:19 |
rloo | ++ movin' | 15:19 |
dtantsur | heh | 15:19 |
dtantsur | #topic Deciding on priorities for the coming week | 15:19 |
*** openstack changes topic to "Deciding on priorities for the coming week (Meeting topic: ironic)" | 15:19 | |
*** ociuhandu has joined #openstack-ironic | 15:19 | |
kaifeng | mgoddard: the root source is from the doc team :) | 15:20 |
mgoddard | kaifeng: we're all thieves when it comes to it | 15:20 |
*** ociuhandu has quit IRC | 15:21 | |
*** ociuhandu has joined #openstack-ironic | 15:22 | |
rloo | imitation is the sincerest form of flattery | 15:22 |
* kaifeng quickly learns | 15:23 | |
rloo | rpioso: are there still idrac changes that need to land in train? | 15:23 |
rloo | rpioso: *need* :) | 15:23 |
dtantsur | okay, I've cleaned up the list. is there anything that has to be there? | 15:23 |
rpioso | rloo: Glad you asked. | 15:24 |
dtantsur | note that "High Priority" is before branching, "Weekly Priorities" go after | 15:24 |
rpioso | We would truly like to land https://review.opendev.org/#/c/672498 | 15:24 |
patchbot | patch 672498 - ironic - Add Redfish vmedia boot interface to idrac HW type - 20 patch sets | 15:24 |
rpioso | It's an idrac HW type change. | 15:24 |
dtantsur | I think this ^^ goes into after-train bucket, especially since it adds a dependency | 15:25 |
dtantsur | with this said, I have to go. I'm leaving finishing the list in mgoddard's capable hands :) | 15:25 |
mgoddard | ok. dtantsur's gone, chuck whatever you like into train :p | 15:25 |
dtantsur | hey!! | 15:25 |
mgoddard | (kidding) | 15:26 |
arne_wiebalck | :-D | 15:26 |
dtantsur | :D | 15:26 |
*** dtantsur is now known as dtantsur|afk | 15:26 | |
* rpioso waves farewell to dtantsur | 15:26 | |
dtantsur|afk | o/ | 15:26 |
rpioso | ;-) | 15:26 |
mgoddard | I think it's the right call rpioso - we'd like to cut the release tomorrow ideally, so I think anything without at least one +2 at this point needs a solid argument behind it | 15:26 |
rpioso | mgoddard: There's high demand for it in Train. | 15:27 |
rpioso | mgoddard: It highly leverages an already merged change. | 15:28 |
rloo | rpioso: i haven't reviewed it; CI isn't happy and no +2. I guess if you can find two cores willing to review and approve before whenever the branch/rc1 is done... | 15:29 |
rpittau | rpioso: there are some comments that need still to be addressed | 15:29 |
rloo | rpioso: i don't know that it is a high priority though. | 15:29 |
*** rachit7 has quit IRC | 15:29 | |
rpioso | etingof and I are feverishly working on addressing the remaining comments. | 15:30 |
mgoddard | the addition of sushy-oem-idrac as a dependency could be risky | 15:30 |
rloo | rpioso: well, it IS a high priority from your point of view | 15:30 |
rpioso | rloo: Understood and understood. | 15:30 |
mgoddard | I think rloo has the answer here | 15:30 |
mgoddard | treat it as an FFE, and find two cores to sponsor | 15:31 |
rpittau | has sushy-oem-idrac actually been released yet ? | 15:31 |
mgoddard | unless the rest of the team thinks it is too risky | 15:31 |
rpioso | Whatever the community could reasonably do would be much appreciated. | 15:31 |
rpioso | We're willing to submit an FFE. | 15:32 |
mgoddard | I don't see any releases on https://github.com/etingof/sushy-oem-idrac | 15:32 |
rpioso | mgoddard: It's being published on PyPI as I keyboard. | 15:32 |
etingof | mgoddard need to reload his browser ;) | 15:32 |
mgoddard | nope | 15:33 |
rloo | that seems risky then. releasing a package just before this feature needs to land. not enough testing/feedback :-( | 15:33 |
mgoddard | ok | 15:33 |
mgoddard | I see i now | 15:33 |
mgoddard | it is a driver-req, so optional | 15:33 |
rpioso | The package is outside of OpenStack/OpenDev | 15:33 |
rpioso | Out-of-tree | 15:34 |
rloo | ok, let's take it to the mailing list then :) | 15:34 |
rpioso | +1 | 15:34 |
mgoddard | rpioso: can you raise an FFE on openstack-discuss for this? | 15:35 |
rpioso | mgoddard: Absolutely! | 15:35 |
mgoddard | thanks | 15:35 |
rpioso | Thank you! | 15:35 |
mgoddard | ok, priorities | 15:36 |
mgoddard | are we missing anything? | 15:36 |
rloo | no. i don't think so. | 15:37 |
rpittau | all the priorities from last week where addressed, in a way or another | 15:37 |
rloo | but it just occurred to me. didn't julia do something wrt shanghai forum or... something? | 15:37 |
mgoddard | she proposed two sessions AFAIK | 15:37 |
rpittau | rloo: she proposed some sessions for the forum | 15:37 |
rloo | ah, yeah. that. worth an announcement. | 15:38 |
mgoddard | operator feedback and one other about a specific feature request | 15:38 |
mgoddard | anyone have a link? | 15:38 |
*** yolanda has joined #openstack-ironic | 15:40 | |
mgoddard | snapshotting, that was it | 15:40 |
* rpioso doesn't see it on https://wiki.openstack.org/wiki/Forum/Shanghai2019, yet. | 15:40 | |
mgoddard | #info Two forum sessions proposed for Shanghai summit - Operator Feedback and snapshots requirements gathering | 15:40 |
mgoddard | http://eavesdrop.openstack.org/irclogs/%23openstack-ironic/%23openstack-ironic.2019-09-19.log.html#t2019-09-19T18:37:14 | 15:40 |
rpittau | yeah I don't think the schedule is up-to-date yet | 15:41 |
mgoddard | ok, no new RFEs to review | 15:41 |
mgoddard | #topic Open Discussion | 15:41 |
*** openstack changes topic to "Open Discussion (Meeting topic: ironic)" | 15:41 | |
mgoddard | Has anyone got anything to discuss today? | 15:42 |
TheJulia | I have nothing but I need to go to airplane mode now | 15:42 |
rpittau | I should have told before when dtantsur|afk was also here, but if there | 15:42 |
tzumainn | hi! if anyone has time to take a look at this spec - https://review.opendev.org/#/c/681066/ - that would be great! | 15:42 |
patchbot | patch 681066 - ironic-specs - Expose node owner information to oslo.policy checks - 6 patch sets | 15:42 |
mgoddard | have a nice flight TheJulia | 15:42 |
TheJulia | Thanks | 15:42 |
rpittau | whoops | 15:42 |
rpittau | have a safe flight TheJulia | 15:43 |
TheJulia | tzumainn: I'll try this week | 15:43 |
rpioso | Safe and uneventful travels, TheJulia. | 15:43 |
tzumainn | TheJulia, thanks, no worries if oyu can't - I know this is a very busy time! | 15:43 |
kaifeng | have a safe travel TheJulia | 15:44 |
rpittau | anything that needs to be done for the releases, I would be glad to help :) | 15:44 |
*** dsneddon has joined #openstack-ironic | 15:45 | |
*** mkowalski_ has joined #openstack-ironic | 15:45 | |
mgoddard | rpittau: thanks. I'm going through the TODO list: https://docs.openstack.org/ironic/latest/contributor/releasing.html | 15:45 |
mgoddard | reviews appreciated | 15:45 |
rpittau | mgoddard: oh handy :) | 15:45 |
rpittau | mgoddard: I left a comment on the release-mappings update patch | 15:47 |
mgoddard | tzumainn: I'll add your spec to the list in our priorities | 15:47 |
tzumainn | mgoddard, thanks! | 15:47 |
mgoddard | should give it a bit more visibility | 15:47 |
mgoddard | rpittau: thanks, I'll respond | 15:48 |
mgoddard | I hear crickets | 15:49 |
*** mkowalski__ has joined #openstack-ironic | 15:49 | |
mgoddard | thanks everyone | 15:49 |
rpioso | mgoddard: Thank you | 15:49 |
kaifeng | thanks mgoddard o/ | 15:49 |
rpittau | mgoddard: thanks! | 15:49 |
mgoddard | with a little luck we'll have train behind us for the next meeting | 15:49 |
mgoddard | onto Ussuri! | 15:50 |
rloo | +++ | 15:50 |
mgoddard | #endmeeting | 15:50 |
*** openstack changes topic to "Bare Metal Provisioning | Status: http://bit.ly/ironic-whiteboard | Docs: http://docs.openstack.org/ironic/ | Bugs: https://storyboard.openstack.org/#!/project_group/75 | Contributors are generally present between 6 AM and 12 AM UTC, If we do not answer, please feel free to pose questions to openstack-discuss mailing list." | 15:50 | |
rpittau | mgoddard: hopefully not literally a train :D | 15:50 |
openstack | Meeting ended Mon Sep 23 15:50:08 2019 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:50 |
openstack | Minutes: http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-09-23-15.00.html | 15:50 |
openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-09-23-15.00.txt | 15:50 |
openstack | Log: http://eavesdrop.openstack.org/meetings/ironic/2019/ironic.2019-09-23-15.00.log.html | 15:50 |
rpioso | rpittau: lol | 15:50 |
mgoddard | rpittau: these jokes will be not nearly as easy with Ussuri | 15:50 |
rpittau | mgoddard: Ussuri is a river, so they could be even worse :D | 15:51 |
mgoddard | uh oh | 15:51 |
rpittau | anyhow, good night! o/ | 15:52 |
*** mkowalski_ has quit IRC | 15:52 | |
*** rpittau is now known as rpittau|afk | 15:52 | |
*** TxGirlGeek has joined #openstack-ironic | 15:52 | |
rpioso | bdodd: I asked a question on https://review.opendev.org/#/c/672498/ I'm hoping you or mraineri could help answer. It's at https://review.opendev.org/#/c/672498/19/ironic/drivers/modules/drac/boot.py@106 | 15:52 |
patchbot | patch 672498 - ironic - Add Redfish vmedia boot interface to idrac HW type - 20 patch sets | 15:52 |
patchbot | patch 672498 - ironic - Add Redfish vmedia boot interface to idrac HW type - 20 patch sets | 15:52 |
* kaifeng heard an old saying that you can't jump into the same Ussuri twice. | 15:52 | |
bdodd | rpioso I'll take a look. I think mraineri is on leave for the rest of the month. | 15:56 |
*** ociuhandu has quit IRC | 15:56 | |
*** ociuhandu has joined #openstack-ironic | 15:57 | |
rpioso | bdodd: Thank you so much! There's another, related comment several lines after that one. Ack on mraineri's leave. | 15:57 |
*** kaifeng has quit IRC | 15:58 | |
*** ociuhandu has quit IRC | 16:01 | |
*** ociuhandu has joined #openstack-ironic | 16:02 | |
*** Lucas_Gray has quit IRC | 16:04 | |
*** stendulker has quit IRC | 16:04 | |
openstackgerrit | Gaƫtan Trellu proposed openstack/python-ironicclient master: Fix usage of --owner filter during node list https://review.opendev.org/682666 | 16:05 |
*** pcaruana has joined #openstack-ironic | 16:06 | |
*** Lucas_Gray has joined #openstack-ironic | 16:06 | |
*** ociuhandu has quit IRC | 16:06 | |
*** alexmcleod has quit IRC | 16:07 | |
openstackgerrit | Merged openstack/python-ironicclient master: Make the dependency on oslo.config explicit https://review.opendev.org/683955 | 16:09 |
*** ociuhandu has joined #openstack-ironic | 16:10 | |
*** mkrai has quit IRC | 16:12 | |
*** e0ne has quit IRC | 16:16 | |
openstackgerrit | Mark Goddard proposed openstack/ironic-inspector master: Clean up release notes for 10.0.0 https://review.opendev.org/683992 | 16:18 |
*** etingof has quit IRC | 16:23 | |
*** Lucas_Gray has quit IRC | 16:25 | |
*** Lucas_Gray has joined #openstack-ironic | 16:29 | |
*** problem_v has joined #openstack-ironic | 16:41 | |
*** dtruong has joined #openstack-ironic | 16:41 | |
*** pcaruana has quit IRC | 16:46 | |
*** Goneri has joined #openstack-ironic | 16:54 | |
openstackgerrit | Merged openstack/python-ironicclient master: Fix TypeError when using endpoint_override with SessionClient https://review.opendev.org/683944 | 17:05 |
*** whoami-rajat has quit IRC | 17:14 | |
*** etingof has joined #openstack-ironic | 17:16 | |
*** derekh has quit IRC | 17:24 | |
*** ociuhandu has quit IRC | 17:27 | |
*** ociuhandu has joined #openstack-ironic | 17:28 | |
*** ociuhandu has quit IRC | 17:28 | |
*** ociuhandu has joined #openstack-ironic | 17:28 | |
*** dsneddon has quit IRC | 17:38 | |
*** S4ren has quit IRC | 17:38 | |
*** e0ne has joined #openstack-ironic | 17:42 | |
*** jtomasek has quit IRC | 17:42 | |
*** yaawang has quit IRC | 17:57 | |
*** yaawang has joined #openstack-ironic | 17:59 | |
*** e0ne has quit IRC | 18:01 | |
*** ricolin has quit IRC | 18:02 | |
*** igordc has joined #openstack-ironic | 18:06 | |
*** goldyfruit___ has quit IRC | 18:11 | |
*** dsneddon has joined #openstack-ironic | 18:12 | |
*** Lucas_Gray has quit IRC | 18:16 | |
*** Goneri has quit IRC | 18:18 | |
*** dsneddon has quit IRC | 18:18 | |
*** ociuhandu has quit IRC | 18:19 | |
*** dsneddon has joined #openstack-ironic | 18:24 | |
*** diga has quit IRC | 18:28 | |
*** jtomasek has joined #openstack-ironic | 18:36 | |
*** goldyfruit___ has joined #openstack-ironic | 18:41 | |
*** e0ne has joined #openstack-ironic | 18:43 | |
*** e0ne has quit IRC | 18:45 | |
*** igordc has quit IRC | 18:51 | |
*** e0ne has joined #openstack-ironic | 18:51 | |
*** goldyfruit___ has quit IRC | 19:01 | |
*** lucasagomes has quit IRC | 19:06 | |
*** lucasagomes has joined #openstack-ironic | 19:06 | |
*** lucasagomes has quit IRC | 19:14 | |
*** mgoddard has quit IRC | 19:17 | |
*** mgoddard has joined #openstack-ironic | 19:20 | |
*** goldyfruit has joined #openstack-ironic | 19:23 | |
*** jcoufal_ has quit IRC | 19:30 | |
*** jcoufal has joined #openstack-ironic | 19:31 | |
*** jtwill98 has quit IRC | 19:35 | |
*** pcaruana has joined #openstack-ironic | 19:43 | |
*** e0ne has quit IRC | 19:46 | |
*** derekh has joined #openstack-ironic | 20:01 | |
*** derekh has quit IRC | 20:01 | |
*** pcaruana has quit IRC | 20:12 | |
*** Lucas_Gray has joined #openstack-ironic | 20:31 | |
*** jtomasek has quit IRC | 20:43 | |
TheJulia | Ussuri jokes... oh my | 20:57 |
TheJulia | I guess we should start early! | 20:57 |
TheJulia | dtantsur|afk: thanks for the ironicclient fix | 21:02 |
TheJulia | dtantsur|afk: re: centos job, we need something like https://review.opendev.org/#/c/683262/ to actually cause the image to be part of the equation | 21:03 |
patchbot | patch 683262 - ironic - WIP: if wholedisk set boot dev to disk - 2 patch sets | 21:03 |
TheJulia | dtantsur|afk: I'm goin gto remove the wip flag, since scarily, it passes. I really don't have time to whip up unit test on it though. | 21:03 |
TheJulia | Actually, I'll create a bug and all that stuff | 21:03 |
*** jcoufal has quit IRC | 21:04 | |
rpioso | TheJulia: During today's meeting, we discussed landing https://review.opendev.org/#/c/672498/ in Train. We agreed I would send an FFE to the ML today. Is that agreeable with you? | 21:07 |
patchbot | patch 672498 - ironic - Add Redfish vmedia boot interface to idrac HW type - 20 patch sets | 21:07 |
TheJulia | ironic-cores: https://review.opendev.org/#/c/679332/ would be good to review. It basically hardens the code path on IPA such that we properly tear-down the iscsi connection. Ultimately this likely will help reduce some of the more odd race conditions. | 21:10 |
patchbot | patch 679332 - ironic-python-agent - Attempt tgtd session detach - 4 patch sets | 21:10 |
TheJulia | rpioso: It is agreeable with me | 21:10 |
rpioso | Thank you! The FFE will be sent out over the next few hours. | 21:11 |
TheJulia | dtantsur|afk: oh wow. it is not really deterministic | 21:12 |
TheJulia | oh... | 21:13 |
TheJulia | https://7f88627495221102c59d-fa02bede2ecdc79d66ce6b922331a4c1.ssl.cf2.rackcdn.com/679332/4/check/metalsmith-integration-ipa-src/1a382a3/controller/logs/ and https://openstack.fortnebula.com:13808/v1/AUTH_e8fd161dc34c421a979a9e6421f823e9/zuul_opendev_logs_919/641731/24/check/metalsmith-integration-glance-localboot-centos7 seem to be doing the right thing. | 21:14 |
TheJulia | I've +2'ed https://review.opendev.org/#/c/681838/4 based upon what is basically a glance. arne_wiebalck if you would do the honors of a quick review, I think that would cover it. | 21:18 |
patchbot | patch 681838 - ironic-python-agent - Delete_configuration, consider removed raid member... - 4 patch sets | 21:18 |
* TheJulia gives up writing story because can't find a easily reproduced error, but decides to remove the wip flag anyway | 21:18 | |
*** tesseract has quit IRC | 21:23 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: if wholedisk set boot dev to disk https://review.opendev.org/683262 | 21:26 |
TheJulia | dtantsur|afk: ^^^ has passed the gate twice, 3rd time is a charm right? | 21:27 |
*** Wryhder has joined #openstack-ironic | 21:27 | |
*** igordc has joined #openstack-ironic | 21:27 | |
*** Lucas_Gray has quit IRC | 21:27 | |
*** Wryhder is now known as Lucas_Gray | 21:28 | |
*** bdodd has quit IRC | 21:28 | |
*** goldyfruit has quit IRC | 21:32 | |
*** bfournie has quit IRC | 21:33 | |
*** rcernin has joined #openstack-ironic | 21:38 | |
*** rcernin has quit IRC | 21:40 | |
*** rcernin has joined #openstack-ironic | 21:40 | |
*** cdearborn has quit IRC | 21:41 | |
*** goldyfruit has joined #openstack-ironic | 21:43 | |
openstackgerrit | Julia Kreger proposed openstack/ironic master: Fix get_boot_option logic https://review.opendev.org/679132 | 21:46 |
openstackgerrit | Merged openstack/ironic master: Document PXE retries https://review.opendev.org/683967 | 22:03 |
openstackgerrit | Merged openstack/ironic master: Release notes cleanup for 13.0.0 (mk2) https://review.opendev.org/683983 | 22:11 |
openstackgerrit | Merged openstack/ironic master: Update env. variables in the documentation https://review.opendev.org/683953 | 22:13 |
*** bfournie has joined #openstack-ironic | 22:33 | |
*** jrwang has quit IRC | 22:40 | |
*** mkowalski__ has quit IRC | 22:46 | |
*** mkowalski__ has joined #openstack-ironic | 22:47 | |
*** rh-jelabarre has quit IRC | 23:06 | |
*** TxGirlGeek has quit IRC | 23:06 | |
*** Lucas_Gray has quit IRC | 23:26 | |
*** sthussey has quit IRC | 23:45 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!