| *** pmannidi is now known as pmannidi|Away | 02:58 | |
| *** pmannidi|Away is now known as pmannidi | 04:49 | |
| rpittau | good morning ironic! o/ | 06:32 |
|---|---|---|
| arne_wiebalck | Good morning rpittau and Ironic+ | 07:13 |
| arne_wiebalck | ! | 07:13 |
| arne_wiebalck | here we go :) | 07:13 |
| * arne_wiebalck is not sure why disconnecting the keyboard resets the layout | 07:13 | |
| rpittau | hey arne_wiebalck :) | 07:51 |
| janders | good morning rpittau arne_wiebalck and Ironic o/ | 10:19 |
| rpittau | hey janders :) | 10:19 |
| opendevreview | Merged openstack/ironic-python-agent-builder master: Update qemu version https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/776507 | 10:24 |
| iurygregory | good morning Ironic o/ | 11:05 |
| dtantsur | morning iurygregory | 11:06 |
| *** ChanServ 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." | 11:56 | |
| *** osmanlicilegi is now known as Guest2 | 11:59 | |
| dtantsur | hmm, what's the difference ^^^? | 11:59 |
| arne_wiebalck | hey janders iurygregory and dtantsur o/ | 12:05 |
| TheJulia | good morning | 12:55 |
| cbouchar_ | good morning | 12:56 |
| cbouchar_ | TheJulia: the root_gb changes didn't work. Seeing "last_error": "Failed to prepare to deploy. Exception: 'root_gb'" | 12:56 |
| cbouchar_ | TheJulia: I'll dig into it after meeting | 12:58 |
| TheJulia | arne_wiebalck: adding some more questions to the survey | 13:05 |
| TheJulia | cbouchar_: ack | 13:05 |
| TheJulia | that gives me a really good idea of where it is occuring | 13:06 |
| arne_wiebalck | TheJulia: great, thanks! | 13:14 |
| iurygregory | good morning TheJulia =) | 13:29 |
| iurygregory | dtantsur, thoughts on doing a major release for bifrost since we merged https://opendev.org/openstack/bifrost/commit/f65b52ed8d0b2644657589cf8f22ac12efd98c25 ? Elod is asking in https://review.opendev.org/c/openstack/releases/+/834316 , I think we good with the minor bump since we are also cutting the stable/branch | 14:19 |
| dtantsur | iurygregory: a bit on the fence... maybe a major version is fine (to be strict with semver) | 14:20 |
| iurygregory | ack, I will update the patch | 14:21 |
| opendevreview | OpenStack Release Bot proposed openstack/bifrost stable/yoga: Update .gitreview for stable/yoga https://review.opendev.org/c/openstack/bifrost/+/834704 | 15:32 |
| opendevreview | OpenStack Release Bot proposed openstack/bifrost stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga https://review.opendev.org/c/openstack/bifrost/+/834705 | 15:32 |
| opendevreview | OpenStack Release Bot proposed openstack/bifrost master: Update master for stable/yoga https://review.opendev.org/c/openstack/bifrost/+/834706 | 15:32 |
| opendevreview | OpenStack Release Bot proposed openstack/bifrost master: Add Python3 zed unit tests https://review.opendev.org/c/openstack/bifrost/+/834707 | 15:32 |
| TheJulia | cbouchar_: first patch updated | 15:44 |
| cbouchar_ | TY VM! | 15:44 |
| TheJulia | cbouchar_: second patch I may end up abandoning. dtantsur gave me an idea with his question in an unrelated direction. You *did* run into that issue where image_source if a directory or even if it has any redirects, things go sideways | 15:45 |
| TheJulia | so I want to fix that, but also it seemed like it was closely wired in | 15:45 |
| TheJulia | but! I don't remember exactly where it was pulling that from, so if you have that exception/error handy in memory or chat or something, send it my way | 15:45 |
| TheJulia | I'm likely loosing power in a little bit, don't know if I'll be back on since I have a semi-complex electrical system they likely will need me available for the panels to be rewired properly. | 15:46 |
| opendevreview | Julia Kreger proposed openstack/ironic master: Allow an image_url override https://review.opendev.org/c/openstack/ironic/+/834542 | 15:46 |
| opendevreview | Julia Kreger proposed openstack/ironic master: WIP: Make non-image deploys more sane... https://review.opendev.org/c/openstack/ironic/+/834709 | 15:46 |
| dtantsur | TheJulia: we should really stop putting internal stuff like image_url to instance_info... | 15:48 |
| dtantsur | less a comment to your patch, more shouting in the darkness | 15:49 |
| dtantsur | hold on, your release note reads as if image_url was something we expected users to provide | 15:49 |
| TheJulia | Agree, in her case they are trying to use image url in part because we don’t handle cases gracefully | 15:50 |
| TheJulia | So there are like…. 4 paths to take | 15:50 |
| TheJulia | I kind of lake the latest patch if I can find where the lookup happens at again | 15:50 |
| dtantsur | okay, let's fix it properly *somehow*. what I'd not like to see is image_url getting a documented part of the public API | 15:50 |
| TheJulia | I literally had it on my screen yesterday | 15:50 |
| dtantsur | because it's a duplicate of image_source (I like the name image_url better, but the ship has sailed) | 15:50 |
| TheJulia | Yeah, that is the direction I’m thinking too | 15:51 |
| TheJulia | I think making image source more sane | 15:51 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent-builder stable/yoga: Update .gitreview for stable/yoga https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/834732 | 15:52 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent-builder stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/834733 | 15:52 |
| opendevreview | OpenStack Release Bot proposed openstack/ironic-python-agent-builder master: Update master for stable/yoga https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/834734 | 15:52 |
| dtantsur | https://review.opendev.org/c/openstack/ironic/+/834542/3/ironic/tests/unit/drivers/modules/test_deploy_utils.py#2080 feels weird | 15:54 |
| dtantsur | is image_source a stage1 image or an ISO? | 15:54 |
| dtantsur | we also have stage2 in instance_info, how does it all play together? | 15:54 |
| TheJulia | Yeah, I think stsge2 was an explicit route around based on upstream anaconda docs | 15:57 |
| TheJulia | I *think* | 15:57 |
| TheJulia | But I think with what I know as of yesterday, we can ci this thing | 15:57 |
| opendevreview | Dmitry Tantsur proposed openstack/ironic master: Link from deploy interface docs to anaconda docs and move it to advanced https://review.opendev.org/c/openstack/ironic/+/834739 | 16:01 |
| dtantsur | TheJulia: ^^^ | 16:01 |
| cbouchar_ | TheJulia: I forward an email which contains my host screen | 16:01 |
| cbouchar_ | TheJulia: I'm in a meeting for an hour. Will come back and review this discussion | 16:02 |
| opendevreview | Roman Popelka proposed openstack/ironic-tempest-plugin master: Remove _create_network & _get_server_port_id_and_ip4 methods https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/834424 | 16:05 |
| opendevreview | Roman Popelka proposed openstack/ironic-tempest-plugin master: Remove setup_clients method https://review.opendev.org/c/openstack/ironic-tempest-plugin/+/834425 | 16:05 |
| TheJulia | dtantsur: thanks | 16:09 |
| arne_wiebalck | bye everyone o/ | 16:51 |
| *** tosky is now known as Guest38 | 17:04 | |
| *** tosky_ is now known as tosky | 17:04 | |
| cbouchar_ | TheJulia: With the exchange above, my understand is it needs more discussion. | 17:23 |
| rpittau | good night! o/ | 17:40 |
| zer0c00l | Redhat support case for %onerror issues in anaconda is 03179089 | 19:37 |
| zer0c00l | https://github.com/rhinstaller/anaconda/pull/3974 | 19:53 |
| opendevreview | Merged openstack/ironic-python-agent-builder stable/yoga: Update .gitreview for stable/yoga https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/834732 | 20:24 |
| opendevreview | Merged openstack/ironic-python-agent-builder stable/yoga: Update TOX_CONSTRAINTS_FILE for stable/yoga https://review.opendev.org/c/openstack/ironic-python-agent-builder/+/834733 | 20:24 |
| iurygregory | WOOT, I was invited to participate in the OpenInfra Live: OpenStack Yoga =O | 20:34 |
| iurygregory | TheJulia, https://review.opendev.org/c/openstack/ironic/+/818299 is something urgent we want in Yoga or it can wait? | 20:56 |
| TheJulia | iurygregory: wait, the upstream side of that is Z release I believe | 20:57 |
| iurygregory | TheJulia, ack =), I was checking all open patches for master to see if there is something urgent | 20:59 |
| opendevreview | Merged openstack/ironic master: Link from deploy interface docs to anaconda docs and move it to advanced https://review.opendev.org/c/openstack/ironic/+/834739 | 20:59 |
| TheJulia | iurygregory: congrats w/r/t openinfra live | 21:00 |
| iurygregory | tks =) | 21:00 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!