Friday, 2019-08-09

*** ricolin__ has joined #openstack-ironic00:13
*** ricolin_ has quit IRC00:16
*** betherly has joined #openstack-ironic00:26
*** betherly has quit IRC00:31
*** whoami-rajat has quit IRC00:34
*** gyee has quit IRC00:43
*** betherly has joined #openstack-ironic01:09
*** betherly has quit IRC01:14
*** igordc has quit IRC01:25
*** rh-jelabarre has quit IRC01:28
*** betherly has joined #openstack-ironic01:30
*** whoami-rajat has joined #openstack-ironic01:32
*** betherly has quit IRC01:35
*** betherly has joined #openstack-ironic02:34
*** betherly has quit IRC02:39
*** altlogbot_3 has quit IRC02:44
*** altlogbot_3 has joined #openstack-ironic02:45
*** betherly has joined #openstack-ironic02:55
*** betherly has quit IRC03:00
*** kaifeng has joined #openstack-ironic03:01
*** dsneddon has quit IRC03:03
*** dsneddon has joined #openstack-ironic03:11
*** dsneddon has quit IRC03:16
*** betherly has joined #openstack-ironic03:28
*** betherly has quit IRC03:32
*** trungnv has quit IRC03:53
*** dsneddon has joined #openstack-ironic03:57
*** dsneddon has quit IRC04:05
*** ricolin__ is now known as ricolin04:09
*** betherly has joined #openstack-ironic04:21
*** betherly has quit IRC04:26
*** mkrai has joined #openstack-ironic04:29
*** dsneddon has joined #openstack-ironic04:34
*** betherly has joined #openstack-ironic04:43
*** ianychoi_ has quit IRC04:44
*** betherly has quit IRC04:47
openstackgerritparesh sao proposed openstack/ironic master: Out-of-band `erase_devices` clean step for Proliant Servers  https://review.opendev.org/64158204:57
*** rachit718 has joined #openstack-ironic05:07
openstackgerritPradip Kadam proposed openstack/ironic master: DRAC : Fix issue for RAID-0 creation for multiple disks for PERC H740P  https://review.opendev.org/67476505:07
*** betherly has joined #openstack-ironic05:15
*** betherly has quit IRC05:19
openstackgerritAlexandra Settle proposed openstack/ironic master: Fixing broken links  https://review.opendev.org/67552205:23
*** bdodd has quit IRC05:26
*** e0ne has joined #openstack-ironic05:40
*** dsneddon has quit IRC05:54
*** e0ne has quit IRC05:59
openstackgerritKaifeng Wang proposed openstack/ironic-inspector master: Split API and conductor services  https://review.opendev.org/67373206:06
*** betherly has joined #openstack-ironic06:08
*** betherly has quit IRC06:13
openstackgerritKaifeng Wang proposed openstack/ironic-inspector master: Adds job to run inspector in non-standalone mode  https://review.opendev.org/67503006:15
*** dsneddon has joined #openstack-ironic06:21
*** mcarden has quit IRC06:26
*** dtantsur|afk is now known as dtantsur06:28
*** betherly has joined #openstack-ironic06:29
dtantsurmorning06:31
*** iurygregory has joined #openstack-ironic06:33
iurygregorygood morning06:33
*** betherly has quit IRC06:35
dtantsurarne_wiebalck, xXraphXx, morning folks! I've got some good progress with a CI job with software RAID, but on deployment our ramdisk doesn't seem to be assembling the RAID: https://logs.opendev.org/02/675102/1/check/ironic-standalone/e7a6770/controller/logs/ironic-bm-logs/node-1_no_ansi_2019-08-08-14:44:03_log.txt.gz06:38
dtantsurDoes it mean something has to be fixed in the way we build the ramdisk? or should we add smth like "mdadm --assemble --scan" to IPA itself?06:38
*** mrtmr has left #openstack-ironic06:39
*** dsneddon has quit IRC06:42
*** betherly has joined #openstack-ironic06:51
*** rpittau|afk is now known as rpittau06:52
rpittaugood morning ironic! happy friday!06:52
dtantsurFRIDAY \o/06:52
rpittau:D06:52
kaifengmorning folks o/06:53
openstackgerritM V P Nitesh proposed openstack/ironic master: UEFI PXE Provisioning for wholedisk images  https://review.opendev.org/66486406:54
iurygregorymorning rpittau kaifeng o/06:54
rpittauhey iurygregory :)06:54
mkraiGood morning Ironic!06:55
*** betherly has quit IRC06:56
mkraihi dtantsur rpittau kaifeng iurygregory o/06:56
iurygregorymkrai, o/06:56
mkraiConfirming a behaviour, Adding a port to an active node is allowed but deleting the port is not allowed.06:57
rpittauhey mkrai :)06:57
mkraiIs this a correct behaviour?06:57
kaifengmkrai: it's true if memory serves06:57
iurygregoryI would say so06:58
mkraikaifeng, Yes I confirmed it. But my question is, if the port is not bound on active node, it should be allowed to delete.06:58
mkraiIsn't it?06:58
*** verma-varsha has joined #openstack-ironic06:59
*** dsneddon has joined #openstack-ironic07:00
iurygregoryhummmm07:00
iurygregoryif the node is not active i would say it would say that you could delete the prot07:01
iurygregoryport*07:01
mkraiiurygregory, yes that's valid and already supported :)07:02
kaifengmkrai: sounds reasonable if no vif attached, but currently we only check provision state07:03
iurygregorybasically it's a bug =)07:04
*** tesseract has joined #openstack-ironic07:04
kaifengi remember we don't allow port add/remove when node is active, but some release before, we relaxed the limit to create a port07:05
mkraikaifeng, iurygregory thanks. I will add a bug for it07:05
mkraikaifeng, do you remember why it was done?07:05
mkraiIs it because a port can be bound later after a node is active?07:06
kaifengurh, memory cheats me07:06
kaifenghere is the patch to prohibit port removal https://review.opendev.org/#/c/470273/07:06
patchbotpatch 470273 - ironic - Prevents deletion of ports for active nodes (MERGED) - 7 patch sets07:06
iurygregoryI would say to still not accept if the node is active, but if it is maintenance you should be able to07:07
mkraiiurygregory, Ok but i still think that if the port is not bound, it should be okay to allow deleting it07:08
iurygregorywell if we can ensure that I'm ok =)07:09
kaifengi would prefer to avoid port remove for an active node, even there is no vif attached07:09
kaifenguser may want to attach a port for an instance07:10
mkraikaifeng, ok so you mean allow only if in maintenance mode? bound, not bound doesn't matter07:11
openstackgerritDigambar proposed openstack/ironic stable/rocky: DRAC: Fix OOB introspection to use pxe_enabled flag in idrac driver  https://review.opendev.org/64836007:11
mkraikaifeng, Just to have a similar behaviour. I think either the addition should also be not in active state or deletion should be allowed in active state.07:12
mkraiWDYT?07:12
*** betherly has joined #openstack-ironic07:12
kaifengmkrai: we assume user handle things themself when the node is set to maintenance07:17
kaifengmkrai: do you suggest to add an option to control this behavior?07:17
*** betherly has quit IRC07:17
*** tssurya has joined #openstack-ironic07:18
mkraikaifeng, yes either allow deletion of non provisioned port in active state or reject addition of port in active state.07:18
* mkrai brb, out for lunch.07:20
*** etingof has joined #openstack-ironic07:20
mrdaThanks dtantsur07:24
* dtantsur had a long "what for" moment :D07:25
dtantsurhi mrda, you're welcome07:25
mrda:-)07:25
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: [WIP] use openstacksdk for ironic module  https://review.opendev.org/67217907:33
*** betherly has joined #openstack-ironic07:34
kaifengmkrai: since removing an available node could be changed via option, I think we can do the same, maybe you can file a rfe and describe your use case and see how others think.07:36
xXraphXxgood morning!07:37
*** betherly has quit IRC07:38
xXraphXxdtantsur, yep there probably misses some kind of 'autostart/autoassembly' in the mdadm config. Checking why my ramdisk actually autoassembles the array. But calling it explicitely in the IPA would avoid this07:42
*** rcernin has quit IRC07:46
xXraphXxlooks like adding a line DEVICE partitions in /etc/mdadm/mdadm.conf could fix it. What is weird is that when not present it is assumed to be DEVICE partitions containers, which should already be ok07:50
openstackgerritAlexandra Settle proposed openstack/ironic master: Fixing broken links  https://review.opendev.org/67552207:59
openstackgerritAlexandra Settle proposed openstack/ironic stable/stein: Fixing broken links  https://review.opendev.org/67555007:59
*** bobmel has joined #openstack-ironic08:00
dtantsurxXraphXx: maybe tinycorelinux problem?08:01
*** diga has joined #openstack-ironic08:03
*** betherly has joined #openstack-ironic08:05
*** betherly has quit IRC08:10
*** dougsz has joined #openstack-ironic08:11
xXraphXxMay be. Is the ramdisk available somewhere (or easy to build ?). Mine is deb 9 based, so the behaviour might be different indeed. Isn't the mdarray already assembled but called /dev/md127 instead of /dev/md0 ?08:11
xXraphXxdtantsur08:12
*** rachit718 has quit IRC08:12
xXraphXxhum nope no md127 from the log08:14
*** lucasagomes has joined #openstack-ironic08:16
mkraikaifeng, What option do you mean in "removing an available node could be changed via option"?08:18
*** ociuhandu has joined #openstack-ironic08:18
dtantsurxXraphXx: no traces of assembling RAID in the logs08:18
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: Software RAID: try to assemble RAID on start-up  https://review.opendev.org/67555408:19
dtantsurxXraphXx: wdyt ^^^?08:19
rpittauxXraphXx: the tinyipa files can be found here https://tarballs.openstack.org/ironic-python-agent/tinyipa/files/08:19
dtantsurxXraphXx: ramdisks are built after each merge ^^^08:19
kaifengmkrai: [conductor]allow_deleting_available_nodes08:19
mkraikaifeng, thanks! I will file an rfe08:20
*** betherly has joined #openstack-ironic08:22
*** verma-varsha has quit IRC08:22
*** jistr is now known as jistr|afk08:24
xXraphXxrpittau, thanks for info08:24
arne_wiebalckdtantsur: We don't use a config file, leave things to the kernel (centos in our case).08:27
dtantsurI guess centos is saner than tinyipa..08:27
dtantsurarne_wiebalck: wdyt about the patch above? I guess it won't hurt to try assemble if RAID is already assembled?08:27
arne_wiebalckdtantsur: should be ok, but will say that no raids were assembled even if the kernel has assembled them08:28
dtantsurhmm08:28
dtantsurany ideas for a better wording?08:29
arne_wiebalckdtantsur: this may be confusing ... how about adding "from mdadm.conf" or "from config file" ?08:29
dtantsurarne_wiebalck: it's not necessary from config file, it will try to detect partitions of the config file is empty08:30
arne_wiebalckdtantsur: what happens if the devices are already assembled?08:31
rpittauAFAIK it will ignore volumes not listed in mdadm.conf, but take into consideration /proc/mdstat08:31
* arne_wiebalck is at the aiport, on his phone, can't check 08:32
dtantsurarne_wiebalck: the man page says it only considers unused devices08:33
arne_wiebalckdtantsur: seems ok then08:34
rpittaudtantsur: can always test adding --verbose :)08:34
arne_wiebalckdtantsur: I guess noone will care if it works ;)08:34
rpittaulool08:34
dtantsuroh, --verbose is a nice idea, adding08:34
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: Software RAID: try to assemble RAID on start-up  https://review.opendev.org/67555408:35
dtantsurlet's try this ^^08:35
dtantsurand sorry for keeping distracting you arne_wiebalck :)08:35
mkraikaifeng, Here's the RFE https://storyboard.openstack.org/#!/story/2006385. Thanks!08:36
arne_wiebalckdtantsur : no worries ;) latest patch lgtm08:36
*** dsneddon has quit IRC08:37
rpittaudtantsur: it also might be that there is a difference in config between tinyipa and other distro, maybe mdadm is not in autostart08:38
* arne_wiebalck gets on a plane now o/08:38
rpittauarne_wiebalck: have a safe flight!08:39
dtantsurrpittau: it's probably the case, yes08:40
*** verma-varsha has joined #openstack-ironic08:42
*** alexmcleod has joined #openstack-ironic08:44
*** verma-varsha has quit IRC08:49
arne_wiebalckrpittau: thx!08:53
dtantsur++08:54
*** Lucas_Gray has joined #openstack-ironic09:01
*** derekh has joined #openstack-ironic09:05
*** verma-varsha has joined #openstack-ironic09:05
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: Software RAID: try to assemble RAID on start-up  https://review.opendev.org/67555409:08
*** dsneddon has joined #openstack-ironic09:10
dtantsurTheJulia, jroll, could you help me clean up my open stable patches? https://review.opendev.org/#/q/owner:dtantsur%2540protonmail.com+status:open+NOT+branch:master09:12
*** verma-varsha has quit IRC09:19
*** e0ne has joined #openstack-ironic09:24
*** tesseract has quit IRC09:28
*** tesseract has joined #openstack-ironic09:28
*** tesseract has quit IRC09:29
*** tesseract has joined #openstack-ironic09:29
kaifenggoodnight, have a nice weekend o/09:46
*** kaifeng has quit IRC09:46
openstackgerritIlya Etingof proposed openstack/ironic master: Add Redfish Virtual Media Boot support  https://review.opendev.org/63845309:50
*** hoonetorg has quit IRC09:57
*** verma-varsha has joined #openstack-ironic10:00
*** mkrai has quit IRC10:05
*** yaawang has quit IRC10:09
*** yaawang has joined #openstack-ironic10:16
*** jistr|afk is now known as jistr10:18
*** hoonetorg has joined #openstack-ironic10:20
*** betherly has quit IRC10:38
*** dsneddon has quit IRC10:40
*** marta_lais has quit IRC10:48
jrollmorning10:49
jrolldtantsur: will do10:49
iurygregorymorning jroll10:55
jrollmorning iurygregory10:56
jrolldtantsur: done, just waiting on newer branches for a couple of +W10:56
*** verma-varsha1 has joined #openstack-ironic11:12
*** bfournie has joined #openstack-ironic11:12
*** verma-varsha has quit IRC11:13
*** verma-varsha1 is now known as verma-varsha11:13
*** dsneddon has joined #openstack-ironic11:15
dtantsurmorning jroll, thanks!11:24
*** verma-varsha has quit IRC11:24
dtantsurjroll: and if you have another minute: https://review.opendev.org/#/c/675554/ (this is for the CI job)11:25
patchbotpatch 675554 - ironic-python-agent - Software RAID: try to assemble RAID on start-up - 3 patch sets11:25
jrollsure11:25
*** rh-jelabarre has joined #openstack-ironic11:25
* jroll is watching downstream CI run at the moment :P11:25
*** verma-varsha has joined #openstack-ironic11:26
jrolldtantsur: is there a story for sw raid?11:26
dtantsurthere is, I think11:26
dtantsurjroll: https://storyboard.openstack.org/#!/story/200458111:26
jrolldtantsur: cool, can you tag it on this patch while I review?11:26
jroll+2 though, looks good11:27
dtantsurjroll: wanna me update the commit message?11:28
jrolldtantsur: yeah, when you have time please11:28
openstackgerritDmitry Tantsur proposed openstack/ironic-python-agent master: Software RAID: try to assemble RAID on start-up  https://review.opendev.org/67555411:29
dtantsurjroll: done ^^11:29
* dtantsur is sad that updating commit message reruns the CI11:29
jrollyeah, wish it would only do it if depends-on was changed11:29
TheJuliaGood morning everyone11:40
openstackgerritDmitry Tantsur proposed openstack/python-ironic-inspector-client stable/stein: Switch base job to ironic-base  https://review.opendev.org/67559111:40
dtantsuraaand this ^^ is probably needed now to repair the CI11:41
dtantsurmorning TheJulia11:41
dtantsura question to the audience: will I stop fixing CI jobs one day? :D11:41
openstackgerritJulia Kreger proposed openstack/ironic master: Fixing broken links  https://review.opendev.org/67552211:45
jrollwhat is a software engineer's job, if not to fix CI?11:46
openstackgerritJulia Kreger proposed openstack/ironic stable/stein: Fixing broken links  https://review.opendev.org/67555011:46
iurygregoryjroll, not only fix, we also break the CI hehehe11:47
jrollvery true :)11:47
*** mrtmr has joined #openstack-ironic11:50
mrtmrhi everyone, Im writing new playbook  and roles for bifrost but there is some point I couldn't understand. in redeploy-dynamic playbook, there is provision_state variable. I searched this variable in bifrost repo but i couldnt find where is it come from11:53
jrollmrtmr: that probably comes from the API, it's a property of a node11:59
*** iurygregory has quit IRC11:59
*** jcoufal has joined #openstack-ironic12:04
*** Lucas_Gray has quit IRC12:07
openstackgerritMerged openstack/metalsmith stable/stein: Ensure instance_info is clean before deploy and after failure  https://review.opendev.org/65978412:12
*** dsneddon has quit IRC12:18
openstackgerritMerged openstack/ironic master: Uses IPA-B to build in addition to CoreOS  https://review.opendev.org/67383412:22
openstackgerritMerged openstack/ironic-python-agent master: Implements OpenStack CI job having DIB-built IPA  https://review.opendev.org/67468812:22
*** diga has quit IRC12:34
*** iurygregory has joined #openstack-ironic12:35
*** mrtmr has quit IRC12:38
*** cdearborn has joined #openstack-ironic12:39
*** dsneddon has joined #openstack-ironic12:41
*** dsneddon has quit IRC12:45
openstackgerritraphael.glon proposed openstack/ironic master: Ansible driver: fix partition_configdrive for logical root_devices.  https://review.opendev.org/67464312:47
*** verma-varsha has quit IRC12:49
openstackgerritraphael.glon proposed openstack/ironic-python-agent master: Softraid: refactor partitioning  https://review.opendev.org/67481912:52
*** Lucas_Gray has joined #openstack-ironic12:53
*** Lucas_Gray has quit IRC12:55
*** Lucas_Gray has joined #openstack-ironic12:57
*** mkrai has joined #openstack-ironic12:58
*** henriqueof has joined #openstack-ironic12:58
*** Lucas_Gray has quit IRC13:02
*** ociuhandu has quit IRC13:03
*** ociuhandu has joined #openstack-ironic13:03
*** Lucas_Gray has joined #openstack-ironic13:03
*** Lucas_Gray has quit IRC13:04
*** ociuhandu has quit IRC13:07
*** Lucas_Gray has joined #openstack-ironic13:13
openstackgerritraphael.glon proposed openstack/ironic-python-agent master: image extension, install_bootloader improvements  https://review.opendev.org/67487913:16
openstackgerritraphael.glon proposed openstack/ironic-python-agent master: Fixes get_holder disks with nvme drives  https://review.opendev.org/67562013:16
*** ociuhandu has joined #openstack-ironic13:27
*** verma-varsha has joined #openstack-ironic13:29
*** jistr is now known as jistr|call13:29
openstackgerritMerged openstack/ironic-inspector stable/rocky: CI: clean up the required projects list  https://review.opendev.org/67395913:31
openstackgerritDigambar proposed openstack/ironic stable/rocky: DRAC: Fix OOB introspection to use pxe_enabled flag in idrac driver  https://review.opendev.org/64836013:33
*** jistr|call is now known as jistr13:38
*** mkrai has quit IRC13:41
*** ociuhandu has quit IRC13:43
*** ociuhandu has joined #openstack-ironic13:44
*** bnemec is now known as beekneemech13:45
*** verma-varsha has quit IRC13:47
*** mkrai has joined #openstack-ironic13:48
*** ociuhandu has quit IRC13:49
openstackgerritIury Gregory Melo Ferreira proposed openstack/ironic-prometheus-exporter master: Support for metrics description  https://review.opendev.org/67534513:51
*** ociuhandu has joined #openstack-ironic13:55
openstackgerritMerged openstack/ironic-python-agent stable/stein: CI: stop using pyghmi from git master  https://review.opendev.org/67385314:00
*** verma-varsha has joined #openstack-ironic14:04
*** jaypipes has joined #openstack-ironic14:06
*** mkrai has quit IRC14:07
openstackgerritIlya Etingof proposed openstack/ironic master: Add set_boot_device hook in `redfish` boot interface  https://review.opendev.org/67212314:19
openstackgerritIlya Etingof proposed openstack/ironic master: Add `filename` parameter to Redfish virtual media boot URL  https://review.opendev.org/67105414:19
*** iurygregory_ has joined #openstack-ironic14:23
*** iurygregory has quit IRC14:25
*** verma-varsha has quit IRC14:30
*** ociuhandu has quit IRC14:34
*** ociuhandu has joined #openstack-ironic14:35
openstackgerritIlya Etingof proposed openstack/ironic master: Allow Legacy boot over virtual media  https://review.opendev.org/67225314:39
openstackgerritIlya Etingof proposed openstack/ironic master: [WIP] Add iDRAC boot interface  https://review.opendev.org/67249814:40
*** ociuhandu has quit IRC14:40
*** ociuhandu has joined #openstack-ironic14:41
*** dsneddon has joined #openstack-ironic14:41
*** ociuhandu has quit IRC14:42
*** dsneddon has quit IRC14:46
openstackgerritRiccardo Pittau proposed openstack/ironic-inspector master: [WIP] use openstacksdk for ironic module  https://review.opendev.org/67217914:54
*** ricolin has quit IRC14:57
*** ricolin has joined #openstack-ironic15:01
*** bdodd has joined #openstack-ironic15:07
*** Lucas_Gray has quit IRC15:12
*** jcoufal has quit IRC15:15
*** Lucas_Gray has joined #openstack-ironic15:23
*** cdearborn has quit IRC15:40
openstackgerritMerged openstack/ironic stable/stein: Enable testing adoption in the CI  https://review.opendev.org/67446615:42
openstackgerritMerged openstack/ironic-lib stable/stein: Use last digit to determine paritition naming scheme  https://review.opendev.org/67382015:42
openstackgerritMerged openstack/ironic master: Fixing broken links  https://review.opendev.org/67552215:42
openstackgerritMerged openstack/ironic-python-agent master: Software RAID: try to assemble RAID on start-up  https://review.opendev.org/67555415:42
openstackgerritMerged openstack/ironic stable/stein: Fixing broken links  https://review.opendev.org/67555015:42
*** iurygregory_ has quit IRC15:43
*** e0ne has quit IRC15:44
*** cmurphy has left #openstack-ironic15:45
openstackgerritSurya Seetharaman proposed openstack/ironic master: Support power state change callbacks to nova using ksa_adapter  https://review.opendev.org/66484215:50
*** gyee has joined #openstack-ironic15:50
*** ash2307 has joined #openstack-ironic15:55
*** ash2307 has left #openstack-ironic15:57
*** tssurya has quit IRC16:03
*** jcoufal has joined #openstack-ironic16:10
*** jcoufal has quit IRC16:11
*** jcoufal has joined #openstack-ironic16:11
*** lucasagomes has quit IRC16:12
*** henriqueof has quit IRC16:14
*** ash2307 has joined #openstack-ironic16:18
*** alexmcleod has quit IRC16:21
*** sabedevops has joined #openstack-ironic16:23
openstackgerritraphael.glon proposed openstack/ironic-python-agent master: Fixes get_holder disks with nvme drives  https://review.opendev.org/67562016:23
rpittaubye all! have a great weekend! o/16:35
*** rpittau is now known as rpittau|afk16:35
*** ricolin_ has joined #openstack-ironic16:41
*** dsneddon has joined #openstack-ironic16:42
*** ricolin has quit IRC16:44
*** csmart has quit IRC16:45
*** ajya[m] has quit IRC16:45
*** irclogbot_2 has quit IRC16:47
*** dsneddon has quit IRC16:47
*** irclogbot_1 has joined #openstack-ironic16:48
*** irclogbot_1 has quit IRC16:49
*** irclogbot_3 has joined #openstack-ironic16:50
*** dtantsur is now known as dtantsur|afk16:50
dtantsur|afkhave a nice weekend!16:50
*** csmart has joined #openstack-ironic16:57
*** dougsz has quit IRC16:59
*** tesseract has quit IRC17:00
*** Lucas_Gray has quit IRC17:01
*** sabedevops has quit IRC17:09
*** jcoufal has quit IRC17:10
*** sabedevops has joined #openstack-ironic17:10
*** dsneddon has joined #openstack-ironic17:13
*** efried has joined #openstack-ironic17:14
efriedo/ ironicers!17:14
efriedironickers?17:15
efriediron knickers?17:15
* efried needs sleep17:15
efriedCould I please ask someone to (re) ack the series cutting nova over to using sdk instead of python-ironicclient? Basically making sure we're doing the mapping properly. Series starts here: https://review.opendev.org/#/c/642899/17:17
patchbotpatch 642899 - nova - Introduces SDK to IronicDriver and uses for node.get - 36 patch sets17:17
*** igordc has joined #openstack-ironic17:17
efriedAlso, is there a CI job other than ironic-tempest-ipa-wholedisk-bios-agent_ipmitool-tinyipa (run in nova's check queue) that we could use to exercise the ironic driver, hopefully through these code paths?17:19
*** betherly has joined #openstack-ironic17:22
*** ash2307 has left #openstack-ironic17:24
jrollefried: you could make an ironic patch with a depends-on to that stack to run all of our CI jobs, but the nova bits they exercise are pretty similar17:24
efriedjroll: okay, swhat I'm working on now, we'll see if I get it right.17:25
jrolllooking at commit titles, those should all be exercised in the job in the nova check queue17:25
jroll(boot/delete/rebuild)17:25
efriedokay17:25
* jroll puts the stack on his list but makes no promises17:25
openstackgerritEric Fried proposed openstack/ironic master: DNM: Test nova SDK cutover  https://review.opendev.org/67566917:26
efriedI'm not very good at this, but we'll see if that works ^17:26
jrollthat should work17:26
efriedThanks jroll17:26
efriedI just kinda didn't want to start merging these patches without having *someone* ironic look through them17:26
efrieddtantsur|afk would probably be the most qualified, since he knows SDK as well. I know he looked at the bottom one a while back.17:27
jrollif they pass CI, I'd trust it, personally17:27
efriedOkay, cool. I was concerned about our level of confidence that the CI job covered "everything".17:27
efriedmm, need to take a closer look, I thought the nova patches were passing that job, but only the first one is...17:30
*** sabedevops has quit IRC17:32
*** betherly has quit IRC17:39
openstackgerritMerged openstack/ironic master: devstack: allow creating more than one volume for a VM  https://review.opendev.org/67380017:48
openstackgerritMerged openstack/ironic-lib stable/rocky: Use last digit to determine paritition naming scheme  https://review.opendev.org/67382217:48
*** betherly has joined #openstack-ironic17:50
openstackgerritraphael.glon proposed openstack/ironic-python-agent master: Additional command to build efi partitions  https://review.opendev.org/67567217:51
openstackgerritraphael.glon proposed openstack/ironic-python-agent master: Additional command to build efi partitions  https://review.opendev.org/67567217:52
*** sabedevops has joined #openstack-ironic17:55
*** ajya[m] has joined #openstack-ironic17:56
*** xXraphXx has quit IRC17:58
*** betherly has quit IRC18:03
*** betherly has joined #openstack-ironic18:15
*** jtomasek has quit IRC18:21
*** betherly has quit IRC18:23
*** efried is now known as efried_pto18:42
TheJuliaIf anyone has spare spoons, the python3 changes folks are trying to make for testing are failing on networking-baremetal. :( http://logs.openstack.org/84/652384/3/check/openstack-tox-py37/44ce986/testr_results.html.gz18:57
TheJuliaIt very well may be that it is broken at the moment :\18:58
* TheJulia can look later though, but still18:58
*** ricolin_ has quit IRC19:02
openstackgerritMerged openstack/python-ironic-inspector-client stable/stein: Switch base job to ironic-base  https://review.opendev.org/67559119:09
openstackgerritMerged openstack/ironic-inspector stable/queens: CI: clean up the required projects list  https://review.opendev.org/67396019:12
*** betherly has joined #openstack-ironic19:25
*** betherly has quit IRC19:34
openstackgerritAndreas Jaeger proposed openstack/ironic master: Docu: Fix broken link  https://review.opendev.org/67568519:41
*** e0ne has joined #openstack-ironic19:41
jungleboyjTheJulia:  Was the ovirt driver removed from Ironic?19:57
jungleboyjOh, it is in a different repo.20:05
*** whoami-rajat has quit IRC20:12
jungleboyjIs there a pointer anywhere to how I can integrate the staging-drivers into a devstack deployment.  Not seeing a lot of documentation for the staging-drivers.20:14
*** betherly has joined #openstack-ironic20:17
*** e0ne has quit IRC20:18
*** jaypipes has quit IRC20:22
*** betherly has quit IRC20:27
*** betherly has joined #openstack-ironic20:36
TheJuliayeah, different repo20:39
TheJuliaI guess we wouldn't object if you were to pip install the staging drivers repo in our devstack plugin20:40
TheJulialike... if "need staging drivers"; do xyz; done20:40
*** dsneddon has quit IRC20:42
*** derekh has quit IRC20:44
openstackgerritMerged openstack/networking-baremetal stable/pike: Install Neutron from stable/pike  https://review.opendev.org/66203020:45
*** derekh has joined #openstack-ironic20:46
openstackgerritJulia Kreger proposed openstack/networking-baremetal master: Update neutron requirement  https://review.opendev.org/67570020:47
*** ijw has joined #openstack-ironic20:48
openstackgerritJulia Kreger proposed openstack/networking-baremetal master: Update neutron requirement  https://review.opendev.org/67570020:50
*** betherly has quit IRC20:53
*** EmilienM is now known as EmilienM|pto21:01
*** igordc has quit IRC21:01
openstackgerritJulia Kreger proposed openstack/networking-baremetal master: Remove vif_details check  https://review.opendev.org/67570121:09
*** dsneddon has joined #openstack-ironic21:13
*** e0ne has joined #openstack-ironic21:21
*** betherly has joined #openstack-ironic21:31
*** e0ne has quit IRC21:32
*** e0ne has joined #openstack-ironic21:35
*** e0ne has quit IRC21:45
*** dsneddon has quit IRC21:47
*** betherly has quit IRC21:48
*** dsneddon has joined #openstack-ironic21:50
*** dsneddon has quit IRC22:10
*** betherly has joined #openstack-ironic22:11
*** betherly has quit IRC22:15
*** betherly has joined #openstack-ironic22:22
*** betherly has quit IRC22:27
*** sabedevops has quit IRC22:33

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!