openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [database] https://review.openstack.org/301815 | 00:03 |
---|---|---|
*** rloo has quit IRC | 00:05 | |
*** piet has quit IRC | 00:06 | |
*** kwazar has quit IRC | 00:07 | |
JayF | that apache :8088 issue in the gate appears to be happening reliably on ipa-src jobs | 00:08 |
JayF | http://logs.openstack.org/89/330789/1/check/gate-tempest-dsvm-ironic-agent_ssh-coreos-src/7306594/logs/devstacklog.txt.gz | 00:08 |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [dhcp] https://review.openstack.org/301830 | 00:08 |
*** wajdi has joined #openstack-ironic | 00:12 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [iboot] https://review.openstack.org/301856 | 00:12 |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [ilo] https://review.openstack.org/301947 | 00:15 |
*** amotoki has quit IRC | 00:17 | |
*** smoriya has joined #openstack-ironic | 00:17 | |
JayF | Working with #openstack-infra, looks like a new zuul launcher thing | 00:19 |
JayF | bug is updated, and it'll be patched up | 00:19 |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [inspector] https://review.openstack.org/301955 | 00:20 |
*** mgoddard has joined #openstack-ironic | 00:22 | |
openstackgerrit | Ruby Loo proposed openstack/ironic: Don't use dict.get(key) in api & conductor unit tests https://review.openstack.org/330830 | 00:22 |
*** hoangcx has joined #openstack-ironic | 00:22 | |
*** mgoddard has quit IRC | 00:27 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [ipmi] https://review.openstack.org/301967 | 00:30 |
*** Sukhdev has quit IRC | 00:30 | |
JayF | infra is going to change their port, but effectiveness is likely on the multiple-days/a week timeframe | 00:31 |
JayF | so I'm also going to push a change up to change the port we use | 00:31 |
JayF | 4011 was suggested as the port that's listed in /etc/services as pxe, so going to use that | 00:31 |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [irmc] https://review.openstack.org/302476 | 00:33 |
JayF | actually, not that, it's netboot-pxe 3928/tcp # PXE NetBoot Manager | 00:33 |
JayF | so choosing that. | 00:33 |
JayF | Does anyone know the historical reason we didn't use the service already on :80? | 00:34 |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [keystone] https://review.openstack.org/303093 | 00:36 |
openstackgerrit | Jay Faulkner proposed openstack/ironic: Change port used for Ironic static http to 3928 https://review.openstack.org/330832 | 00:37 |
*** sacharya_ has quit IRC | 00:38 | |
JayF | jroll: jlvillal ^ 330832 is our gate fix | 00:41 |
* JayF wishes there were more cores awake at this time pst, lol | 00:41 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [oneview] https://review.openstack.org/303103 | 00:49 |
jroll | this is amazing | 00:52 |
jroll | lol | 00:52 |
jlvillal | JayF, Awesome :) | 00:52 |
JayF | I was literally about to leave for the day at 5, saw the bugfix to ipa failed across the board agani | 00:53 |
JayF | asked in infra and just threwaway comment of 'looks like 8088 got ninja'd out from under us' | 00:53 |
JayF | and then everything clicked and what happened was clear | 00:53 |
jroll | JayF: +A | 00:53 |
jroll | thank you for that | 00:53 |
JayF | there was a very valid question though: why weren't we using port 80? | 00:53 |
jroll | I have no idea, I actually missed the switch to using apache at all in the gate | 00:54 |
JayF | I didn't know if there was some reason (I presumed maybe to have it at a different url than ironic api or something?) | 00:54 |
JayF | we don't use wsgi-apache | 00:54 |
jroll | oh, ipxe? | 00:54 |
JayF | we use apache to serve images to ipxe | 00:54 |
JayF | static hosting | 00:54 |
jroll | ya | 00:54 |
JayF | so it's something lucasagomes must have added when he did that forever-ago | 00:54 |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [seamicro] https://review.openstack.org/303113 | 00:54 |
JayF | either way, we should consider just hosting that on :80 at some path | 00:54 |
JayF | but that's a bigger change and I just wanted to get the obvious fix in asap | 00:54 |
jlvillal | JayF, +2 | 00:55 |
jroll | so my first instinct is that horizon or keystone serves on 80 and nobody could be bothered to do the apache config | 00:55 |
JayF | well multiple services live on :80 already | 00:55 |
JayF | and the infra is clearly there for multiple services to share apache | 00:55 |
jroll | yeah | 00:55 |
jroll | mind filing a bug? | 00:55 |
JayF | I'm not sure it's a bug :) | 00:55 |
JayF | maybe an RFE | 00:55 |
jroll | well, whatever | 00:56 |
jroll | file a thing under the thing called bugs in launchpad :P | 00:56 |
jroll | thank you for helping with finding that | 00:56 |
JayF | lol | 00:56 |
JayF | it's fine, all I did was wtf in infra | 00:56 |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [snmp] https://review.openstack.org/303644 | 00:56 |
JayF | and they did all the figuring out | 00:56 |
JayF | if all bugs were fixed by lines in irc | 00:56 |
JayF | me and you would fix more than anyone else | 00:56 |
jroll | heh | 00:57 |
jroll | bark > bite? :| | 00:57 |
*** YiboCai has joined #openstack-ironic | 00:57 | |
jlvillal | JayF, Awesome figuring that out :) | 00:57 |
JayF | the scariest dogs do both | 00:58 |
*** sdake has joined #openstack-ironic | 00:58 | |
jlvillal | JayF, What I don't get is why it didn't kill all jobs. Why did most jobs pass. | 00:59 |
JayF | jlvillal: it's the zuul launcher migration. | 01:00 |
JayF | jlvillal: jobs that ran on "new zuul launcher" failed | 01:00 |
*** mbound has joined #openstack-ironic | 01:00 | |
JayF | jobs on old jenkins worked | 01:00 |
jlvillal | Ah... | 01:00 |
JayF | when all of them changed to zuul today, we went to hard-broken | 01:00 |
JayF | https://bugs.launchpad.net/ironic/+bug/1593510 | 01:00 |
openstack | Launchpad bug 1593510 in Ironic "[RFE] Host ipxe static assets in devstack on :80" [Undecided,New] | 01:00 |
jroll | that's why frequency increased too | 01:00 |
jroll | thanks JayF | 01:01 |
JayF | no problemo | 01:01 |
jroll | low-hanging-fruit++ | 01:01 |
JayF | there's nothing more satisfying | 01:01 |
JayF | than fixing a major bug | 01:01 |
JayF | by doing nearly nothing | 01:01 |
jlvillal | JayF, The priority of that bug would have really gone up tomorrow morning for us ;) | 01:01 |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [ssh] https://review.openstack.org/303649 | 01:01 |
JayF | heh. I was just angry the ipa fix wasn't going in so i rage-diagnosed it | 01:01 |
JayF | lol | 01:01 |
JayF | breaking all the time is way easier to diagnose than sometimes breaking, anyway | 01:02 |
* jroll thinks dinner is ready, see y'all tomorrow | 01:03 | |
jroll | thanks again JayF | 01:03 |
JayF | hey it's no problem | 01:03 |
jlvillal | ++++++ | 01:03 |
JayF | I lost an hour fixing flights for my little bro so just stayed and was able to fix it | 01:03 |
jroll | just read that :P | 01:03 |
JayF | I'm "off" tomorrow but given my brother isn't going to be in until a little later, might be in a little in the morning | 01:03 |
JayF | I fixed a lot of stuff today. None of that 'stuff' was the policy spec :( | 01:04 |
*** mbound has quit IRC | 01:05 | |
JayF | you know what's additionally funny | 01:05 |
JayF | I made this way worse... by making all jobs use ipxe except one | 01:05 |
jlvillal | What? | 01:05 |
JayF | lol | 01:05 |
JayF | not like it matters if a voting job was still broken anyway | 01:06 |
jlvillal | true | 01:06 |
JayF | but ipa broken across the board was because it's all ipxe | 01:06 |
jlvillal | So I guess infra started that change over around 6-June | 01:06 |
JayF | Probably so. And ramped it up. | 01:06 |
JayF | Honestly, this is best case | 01:06 |
JayF | nothing actually broken | 01:06 |
JayF | just a silly thing that happens | 01:06 |
jlvillal | Yep. | 01:06 |
jlvillal | JayF, I updated the whiteboard | 01:07 |
JayF | way better than if ironic or devstack had some crazy race condition | 01:07 |
JayF | I updated the 'random' to ALL OF THE JOERBS :) | 01:07 |
jlvillal | Heh | 01:07 |
jlvillal | Okay time for me to go spend time with the wife :) | 01:07 |
JayF | yep, same here | 01:07 |
JayF | I'll be in a little tomorrow if for nothing else than to play recheck fairy o/ | 01:08 |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [swift] https://review.openstack.org/303653 | 01:09 |
*** sdake has quit IRC | 01:14 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [virtualbox] https://review.openstack.org/303655 | 01:15 |
*** sdake has joined #openstack-ironic | 01:16 | |
*** praneshp has joined #openstack-ironic | 01:16 | |
*** Goneri has joined #openstack-ironic | 01:17 | |
*** YiboCai has quit IRC | 01:18 | |
*** YiboCai has joined #openstack-ironic | 01:18 | |
openstackgerrit | Ruby Loo proposed openstack/ironic: Replace dict.get(key) in api & conductor tests https://review.openstack.org/330830 | 01:20 |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [glance] https://review.openstack.org/304829 | 01:22 |
openstackgerrit | Ruby Loo proposed openstack/ironic: Replace dict.get(key) in drivers/modules/*/ tests https://review.openstack.org/330838 | 01:22 |
*** sdake_ has joined #openstack-ironic | 01:22 | |
*** sdake has quit IRC | 01:24 | |
*** YiboCai has quit IRC | 01:27 | |
*** YiboCai has joined #openstack-ironic | 01:27 | |
*** baoli has joined #openstack-ironic | 01:28 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [neutron] https://review.openstack.org/304838 | 01:30 |
*** Goneri has quit IRC | 01:34 | |
jlvillal | JayF, jroll: I backported JayF's patch to stable/mitaka. https://review.openstack.org/330839 | 01:36 |
*** rama_y has quit IRC | 01:38 | |
*** YiboCai has quit IRC | 01:39 | |
*** YiboCai has joined #openstack-ironic | 01:39 | |
*** YiboCai has quit IRC | 01:40 | |
*** YiboCai has joined #openstack-ironic | 01:40 | |
jlvillal | And stable/liberty via devstack: https://review.openstack.org/330846 | 01:43 |
*** mtanino has quit IRC | 01:45 | |
*** Goneri has joined #openstack-ironic | 01:48 | |
*** hoangcx has quit IRC | 01:51 | |
*** hoangcx has joined #openstack-ironic | 01:51 | |
*** Sukhdev has joined #openstack-ironic | 01:57 | |
*** sacharya has joined #openstack-ironic | 01:59 | |
*** Goneri has quit IRC | 02:00 | |
*** sacharya_ has joined #openstack-ironic | 02:09 | |
*** garthb has joined #openstack-ironic | 02:10 | |
*** sacharya has quit IRC | 02:13 | |
openstackgerrit | Merged openstack/ironic: Change port used for Ironic static http to 3928 https://review.openstack.org/330832 | 02:14 |
*** Goneri has joined #openstack-ironic | 02:15 | |
*** sdake has joined #openstack-ironic | 02:16 | |
*** sdake_ has quit IRC | 02:19 | |
*** piet has joined #openstack-ironic | 02:22 | |
*** mtanino has joined #openstack-ironic | 02:22 | |
*** Goneri has quit IRC | 02:26 | |
openstackgerrit | Ruby Loo proposed openstack/ironic: Replace dict.get(key) in drivers unit tests https://review.openstack.org/330852 | 02:28 |
*** Goneri has joined #openstack-ironic | 02:28 | |
*** piet has quit IRC | 02:35 | |
*** Goneri has quit IRC | 02:46 | |
*** garthb has quit IRC | 02:52 | |
*** rama_y has joined #openstack-ironic | 02:53 | |
*** ChrisAusten has joined #openstack-ironic | 02:57 | |
*** Goneri has joined #openstack-ironic | 02:57 | |
*** ChrisAusten has quit IRC | 02:58 | |
*** joprovost has quit IRC | 03:03 | |
jlvillal | dtantsur|afk: Need this to fix stable/mitaka and grenade: https://review.openstack.org/330839 | 03:05 |
*** amotoki has joined #openstack-ironic | 03:07 | |
*** Goneri has quit IRC | 03:08 | |
*** jcoufal has joined #openstack-ironic | 03:08 | |
*** amotoki has quit IRC | 03:09 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [DEFAULT] https://review.openstack.org/309070 | 03:15 |
*** Goneri has joined #openstack-ironic | 03:21 | |
*** baoli has quit IRC | 03:25 | |
*** Goneri has quit IRC | 03:28 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [api] https://review.openstack.org/309186 | 03:39 |
openstackgerrit | Merged openstack/ironic: Active Node Creation via adopt state https://review.openstack.org/275766 | 03:40 |
jlvillal | TheJulia, ^^^ :) | 03:41 |
openstackgerrit | Merged openstack/ironic: Centralize config options - [console] https://review.openstack.org/295946 | 03:44 |
*** vmud213 has joined #openstack-ironic | 03:46 | |
*** Goneri has joined #openstack-ironic | 03:46 | |
openstackgerrit | Merged openstack/ironic: Centralize config options - [cimc] https://review.openstack.org/301376 | 03:47 |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [deploy] https://review.openstack.org/309206 | 03:49 |
*** Goneri has quit IRC | 03:56 | |
*** amotoki has joined #openstack-ironic | 04:00 | |
openstackgerrit | Merged openstack/ironic-inspector: Fix response code for rule creating API https://review.openstack.org/327492 | 04:12 |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [agent] https://review.openstack.org/309603 | 04:14 |
*** Nisha_away has joined #openstack-ironic | 04:18 | |
*** amotoki has quit IRC | 04:18 | |
*** links has joined #openstack-ironic | 04:20 | |
openstackgerrit | Merged openstack/python-ironicclient: Implementation of baremetal power state commands https://review.openstack.org/172517 | 04:20 |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [pxe] https://review.openstack.org/309612 | 04:21 |
*** klindgren has quit IRC | 04:24 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize config options - [iscsi] https://review.openstack.org/309615 | 04:35 |
*** yuanying has quit IRC | 04:41 | |
*** krsacme has quit IRC | 04:41 | |
openstackgerrit | Yuiko Takada Mori proposed openstack/ironic: Fix parameter create-node.sh https://review.openstack.org/325732 | 04:43 |
*** amotoki has joined #openstack-ironic | 04:45 | |
*** SiRiuS__ has quit IRC | 04:45 | |
*** ijw_ has joined #openstack-ironic | 04:46 | |
*** ijw has quit IRC | 04:47 | |
*** sdake_ has joined #openstack-ironic | 04:50 | |
*** sdake has quit IRC | 04:53 | |
*** garthb has joined #openstack-ironic | 04:57 | |
*** garthb has quit IRC | 04:58 | |
*** ijw_ has quit IRC | 04:59 | |
*** ijw has joined #openstack-ironic | 04:59 | |
*** sacharya_ has quit IRC | 05:06 | |
openstackgerrit | Ramamani Yeleswarapu proposed openstack/ironic: Centralize Config Options - patch merge, cleanup https://review.openstack.org/330160 | 05:10 |
*** ChubYann has quit IRC | 05:11 | |
*** yuriyz|2 has quit IRC | 05:15 | |
*** links has quit IRC | 05:27 | |
*** sdake has joined #openstack-ironic | 05:27 | |
*** links has joined #openstack-ironic | 05:28 | |
*** sdake_ has quit IRC | 05:31 | |
*** cfarquhar has quit IRC | 05:35 | |
*** klindgren has joined #openstack-ironic | 05:37 | |
*** sdake_ has joined #openstack-ironic | 05:38 | |
*** sdake has quit IRC | 05:41 | |
openstackgerrit | Yuiko Takada Mori proposed openstack/ironic: [WIP] Nova-compatible serial console: socat console_utils https://review.openstack.org/328168 | 05:43 |
openstackgerrit | Yuiko Takada Mori proposed openstack/ironic: IPMINative: add NativeIPMISocatConsole class https://review.openstack.org/293874 | 05:43 |
*** skramaja has joined #openstack-ironic | 05:46 | |
*** vmud213 has quit IRC | 05:47 | |
*** sdake_ has quit IRC | 05:50 | |
*** bcornec1 has quit IRC | 05:50 | |
*** mbound has joined #openstack-ironic | 05:55 | |
*** amotoki has quit IRC | 05:56 | |
*** mbound has quit IRC | 06:00 | |
*** ccamacho has joined #openstack-ironic | 06:00 | |
*** amotoki has joined #openstack-ironic | 06:02 | |
*** yuanying has joined #openstack-ironic | 06:02 | |
*** tosc_fiberdata has joined #openstack-ironic | 06:03 | |
*** rama_y has quit IRC | 06:08 | |
*** vmud213 has joined #openstack-ironic | 06:10 | |
*** vmud213 has quit IRC | 06:12 | |
*** vmud213 has joined #openstack-ironic | 06:13 | |
*** Sukhdev has quit IRC | 06:17 | |
*** rcernin has joined #openstack-ironic | 06:23 | |
*** mtanino has quit IRC | 06:24 | |
*** david-lyle has quit IRC | 06:26 | |
*** pcaruana has joined #openstack-ironic | 06:29 | |
*** livelace has joined #openstack-ironic | 06:31 | |
*** _vsaienko has quit IRC | 06:46 | |
*** _milan_ has joined #openstack-ironic | 06:59 | |
*** fragatina has quit IRC | 06:59 | |
_milan_ | morning Ironic | 06:59 |
lazy_prince | Hi.. just wanted to confirm that https://github.com/openstack/ironic/blob/stable/mitaka/etc/ironic/ironic.conf.sample#L498 is only consumed by Ironic api service and not by conductor service.. is that right understanding..? | 07:00 |
*** milan has quit IRC | 07:01 | |
*** yuanying has quit IRC | 07:02 | |
*** amotoki has quit IRC | 07:03 | |
*** tesseract has joined #openstack-ironic | 07:10 | |
*** max_lobur has joined #openstack-ironic | 07:11 | |
*** max_lobur has left #openstack-ironic | 07:11 | |
*** dtantsur|afk is now known as dtantsur | 07:15 | |
dtantsur | Morning Ironic | 07:15 |
dtantsur | jlvillal, +2 | 07:19 |
aarefiev | morning! | 07:21 |
Nisha_away | dtantsur, morning | 07:25 |
dtantsur | morning Nisha_away, aarefiev | 07:25 |
*** Nisha_away is now known as Nisha1 | 07:26 | |
*** ohamada has joined #openstack-ironic | 07:26 | |
aarefiev | morning dtantsur, Nisha_away | 07:30 |
*** dmk0202 has joined #openstack-ironic | 07:32 | |
*** ijw has quit IRC | 07:33 | |
*** dmk0202 has quit IRC | 07:33 | |
*** rcernin has quit IRC | 07:45 | |
*** wagiel has joined #openstack-ironic | 07:46 | |
_milan_ | morning aarefiev dtantsur Nisha1 | 07:46 |
aarefiev | morning _milan_ | 07:46 |
wagiel | morning openstackers | 07:46 |
dtantsur | morning _milan_, wagiel | 07:47 |
wagiel | I have a question about dnsmasq in ironic, i see process is running with all the options, but when telnet port 67 nothing is listening, my instance is spawning but when it comes to dhcp part nothing is happening | 07:48 |
dtantsur | wagiel, note that we don't manage dnsmasq, it's neutron's job. I think they're using network namespaces, but it's better to ask them (#openstack-neutron) | 07:50 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Add dbapi and objects functions to get a node by associated MAC addresses https://review.openstack.org/330938 | 07:51 |
wagiel | dtantsur: thank you, I will | 07:52 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: [WIP] Promote agent vendor passthru to core API https://review.openstack.org/330018 | 07:53 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Add manage_boot parameter to introspection API https://review.openstack.org/316801 | 07:54 |
*** yolanda has joined #openstack-ironic | 07:55 | |
*** zzzeek has quit IRC | 08:00 | |
*** zzzeek has joined #openstack-ironic | 08:00 | |
_milan_ | guys, I want to introduce inspector experimental grenade gate. I want to base it on the ironic job from devstack-gate.yaml jenkins definitnion in infra. I'm wondering how to make the job enable inspector's grenade plugin (under review now) without having to insert ref spec in the grenade plugin line. | 08:00 |
_milan_ | e.g https://github.com/openstack-infra/project-config/blob/d09a1a898d34931f7fc49e027bf665950e66a2b2/jenkins/jobs/devstack-gate.yaml#L2096 | 08:02 |
_milan_ | vs https://github.com/dparalen/devstack-gate-test/blob/with_inspector/ansible/roles/devstack-gate/files/inspector-grenade.sh#L99 | 08:05 |
*** yuanying has joined #openstack-ironic | 08:05 | |
dtantsur | _milan_, if I got your question right, zuul is doing it automatically for gate jobs, hence it's not there for ironic | 08:06 |
dtantsur | but it's a bit too early to think about gate jobs for me :D | 08:07 |
_milan_ | dtantsur, I'm not sure how the ref is propagated. I'd like to be able to: "check experimental" as I've spent most my time debugging fake gates :-/ | 08:08 |
dtantsur | _milan_, I'm not sure either, it Just Works (tm) | 08:08 |
*** vmud213 has quit IRC | 08:08 | |
_milan_ | :D | 08:08 |
dtantsur | you can ask on #openstack-infra ofc | 08:08 |
*** vmud213 has joined #openstack-ironic | 08:08 | |
_milan_ | yeah | 08:08 |
_milan_ | anyways, how about we make such an experimental job? | 08:09 |
_milan_ | might make life easier for reviewers too | 08:09 |
*** rcernin has joined #openstack-ironic | 08:10 | |
dtantsur | _milan_, definitely. I was going to ask for it :) | 08:10 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 08:11 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 08:11 |
_milan_ | dtantsur, yeah, I wasn't able to figure out how to move on from this error: http://paste.openstack.org/show/516792/ | 08:12 |
_milan_ | dtantsur, it's tiny IPA | 08:12 |
_milan_ | I tried with core os but ran into different errors of course | 08:12 |
dtantsur | hmm | 08:13 |
dtantsur | we can look into it today | 08:13 |
_milan_ | in result, my fake gate stops right there and that's the furthest I can get right now | 08:13 |
_milan_ | yeah, would be nice | 08:13 |
* _milan_ about to relocate | 08:14 | |
_milan_ | oh, and of course, ironic node-list says: wait for callback | 08:14 |
_milan_ | obviously | 08:14 |
_milan_ | dtantsur, so, I shall post a review w/r the experimental inspector grenade job for devstack-gate.yaml(/projects.yaml) | 08:17 |
dtantsur | cool | 08:17 |
lucasagomes | morning all | 08:17 |
*** mgoddard has joined #openstack-ironic | 08:20 | |
*** jlvillal has quit IRC | 08:22 | |
*** jlvillal has joined #openstack-ironic | 08:22 | |
dtantsur | lucasagomes, morning | 08:22 |
*** dmk0202 has joined #openstack-ironic | 08:27 | |
*** daemontool has joined #openstack-ironic | 08:28 | |
openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: WIP: Tempest: Use tempest call_until_true https://review.openstack.org/330950 | 08:30 |
*** athomas has joined #openstack-ironic | 08:30 | |
*** alineb has joined #openstack-ironic | 08:31 | |
_milan_ | lucasagomes, morning | 08:31 |
* _milan_ relocate | 08:32 | |
*** _milan_ has quit IRC | 08:32 | |
*** Romanenko_K has quit IRC | 08:36 | |
*** Romanenko_K has joined #openstack-ironic | 08:36 | |
dtantsur | aarefiev, something seems broken about our tempest: http://logs.openstack.org/01/316801/5/check/gate-tempest-dsvm-ironic-inspector-nv/d5ec08c/console.html#_2016-06-17_08_27_57_090450 | 08:36 |
*** zer0c00l has quit IRC | 08:37 | |
*** jlvillal has quit IRC | 08:37 | |
*** _jmp_ has joined #openstack-ironic | 08:38 | |
aarefiev | dtantsur: I'm looking, but gerrit super slow :( | 08:38 |
*** zer0c00l has joined #openstack-ironic | 08:38 | |
*** wagiel has quit IRC | 08:38 | |
*** _jmp_ is now known as Guest39421 | 08:38 | |
*** wagiel has joined #openstack-ironic | 08:39 | |
*** harlowja_ has quit IRC | 08:41 | |
*** electrofelix has joined #openstack-ironic | 08:41 | |
*** jlvillal has joined #openstack-ironic | 08:42 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: [WIP] Promote agent vendor passthru to core API https://review.openstack.org/330018 | 08:42 |
*** jmccrory has quit IRC | 08:42 | |
*** daemontool_ has joined #openstack-ironic | 08:42 | |
*** anteaya has quit IRC | 08:42 | |
*** ishant|away has quit IRC | 08:42 | |
*** vmud213 has quit IRC | 08:42 | |
*** ekarlso has quit IRC | 08:42 | |
*** ishant|away has joined #openstack-ironic | 08:42 | |
*** livelace has quit IRC | 08:42 | |
*** kirankv has quit IRC | 08:42 | |
*** m3m0 has quit IRC | 08:42 | |
*** lazy_prince has quit IRC | 08:42 | |
*** kirankv has joined #openstack-ironic | 08:42 | |
*** lynxman_ has joined #openstack-ironic | 08:43 | |
*** igordcard has quit IRC | 08:43 | |
*** aignatov has quit IRC | 08:43 | |
*** _jmp__ has quit IRC | 08:43 | |
*** rcarrillocruz has quit IRC | 08:43 | |
*** mgould|afk is now known as mgould | 08:43 | |
mgould | morning Ironic | 08:43 |
*** xek has quit IRC | 08:43 | |
*** lynxman has quit IRC | 08:43 | |
*** persia has quit IRC | 08:43 | |
*** lynxman_ is now known as lynxman | 08:43 | |
*** lazy_prince has joined #openstack-ironic | 08:43 | |
*** jmccrory has joined #openstack-ironic | 08:43 | |
*** yhvh has quit IRC | 08:43 | |
dtantsur | morning mgould | 08:43 |
*** aignatov has joined #openstack-ironic | 08:43 | |
*** livelace has joined #openstack-ironic | 08:43 | |
*** xek has joined #openstack-ironic | 08:43 | |
*** persia has joined #openstack-ironic | 08:43 | |
*** vmud213 has joined #openstack-ironic | 08:44 | |
*** mtreinish has quit IRC | 08:44 | |
*** daemontool has quit IRC | 08:44 | |
*** e0ne has joined #openstack-ironic | 08:44 | |
*** yhvh has joined #openstack-ironic | 08:44 | |
*** rcarrillocruz has joined #openstack-ironic | 08:44 | |
dtantsur | lucasagomes, mind approving https://review.openstack.org/#/c/306418/ please? has 3x +2 already.. | 08:45 |
*** m3m0 has joined #openstack-ironic | 08:45 | |
lucasagomes | dtantsur, will take a look, 1 se | 08:45 |
lucasagomes | sec* | 08:45 |
*** anteaya has joined #openstack-ironic | 08:45 | |
*** mtreinish has joined #openstack-ironic | 08:46 | |
*** Nisha_away has joined #openstack-ironic | 08:47 | |
*** igordcard has joined #openstack-ironic | 08:49 | |
*** Nisha1 has quit IRC | 08:50 | |
openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: WIP: Fix tempest test https://review.openstack.org/330955 | 08:51 |
*** ekarlso has joined #openstack-ironic | 08:51 | |
aarefiev | dtantsur: ^ should fix | 08:51 |
dtantsur | aarefiev, looks good, why is it WIP? | 08:52 |
aarefiev | dtantsur: I'll change commit message once gates pass | 08:55 |
dtantsur | aarefiev, that will require 2nd gate run | 08:55 |
dtantsur | aarefiev, you can mark your patch W-1 instead, then you will be able to unmark it without rerunning gate | 08:55 |
dtantsur | (workflow -1 button in gerrit UI) | 08:56 |
aarefiev | dtantsur: yeah, I know :) | 08:56 |
aarefiev | dtantsur: I was marked as wip because I'm not sure this is only one reason, let see | 08:58 |
dtantsur | when I'm not sure about the patch I usually use W-1, but as you wish :) | 08:58 |
*** hoangcx has quit IRC | 09:05 | |
*** hoangcx_ has joined #openstack-ironic | 09:05 | |
*** dtantsur is now known as dtantsur|bbl | 09:07 | |
*** sambetts|afk is now known as sambetts | 09:12 | |
sambetts | Morning all | 09:12 |
aarefiev | morning sambetts | 09:13 |
mgould | morning sambetts lucasagomes dtantsur|bbl Nisha_away aarefiev | 09:13 |
aarefiev | morning mgould | 09:13 |
*** e0ne has quit IRC | 09:13 | |
Nisha_away | mgould, o/ | 09:14 |
wagiel | @sambetts: Hi, thanks for the local.conf, worked like a charm, instance is spawning but dont get any dhcp responses, tcpdumped br-int interface but dont see any traffic on 67 | 09:15 |
*** tloeb_ has joined #openstack-ironic | 09:17 | |
*** tloeb_ is now known as tloeb__ | 09:17 | |
*** openstackgerrit has quit IRC | 09:18 | |
sambetts | wagiel: is the physical nic that is being used by your controller node to talk to your baremetal (in my case eth2) up? I've found that devstack does bring that interface up by default | 09:18 |
*** openstackgerrit has joined #openstack-ironic | 09:18 | |
wagiel | @sambets: yes it is up, when tcpdump br-eth1 i can see the udp 67 requests coming in, but no on br-int | 09:21 |
*** _milan_ has joined #openstack-ironic | 09:22 | |
openstackgerrit | Merged openstack/ironic: Remove workaround for nova removing instance_uuid during cleaning https://review.openstack.org/328929 | 09:22 |
*** alex_xu has quit IRC | 09:26 | |
*** YiboCai has quit IRC | 09:27 | |
*** YiboCai has joined #openstack-ironic | 09:27 | |
sambetts | wagiel: instead of doing tcpdump on br-int can you tcpdump on the tap interface for the dhcp server? | 09:28 |
*** alex_xu has joined #openstack-ironic | 09:28 | |
openstackgerrit | Anton Arefiev proposed openstack/ironic-inspector: Fix tempest tests https://review.openstack.org/330955 | 09:30 |
aarefiev | dtantsur sambetts: ^tempest fix works, job pass | 09:30 |
sambetts | wagiel: if you do a sudo ip netns list, you'll see the list of namespaces, one of those should have the UUID of the subnet your dhcp server is in | 09:31 |
sambetts | subnet or network I can rememeber which sorry :( | 09:32 |
sambetts | wagiel: once you've worked that out you can do a "sudo ip netns exec <namespace name> ifconfig" and it'll list the ports in that namespace | 09:33 |
sambetts | wagiel: then you can do a "sudo ip netns exec <namespace name> tcpdump -i <tap interface>" to see the packets actually making it to your dhcp server IP | 09:34 |
openstackgerrit | Merged openstack/ironic-python-agent: Fix unwanted unmounting of /usr/share/oem https://review.openstack.org/330789 | 09:36 |
vsaienko | morning sambetts, aarefiev, dtantsur, vdrok and all Ironic'ers! | 09:39 |
aarefiev | morning vsaienko | 09:39 |
hkominos | morning openstackers and ironicers! | 09:39 |
*** mgoddard_ has joined #openstack-ironic | 09:42 | |
*** mgoddard has quit IRC | 09:42 | |
hkominos | IS there anyone online that I can ask about VLANs and Ironic? | 09:42 |
*** ipukha has joined #openstack-ironic | 09:42 | |
hkominos | (FUEL deployment) | 09:42 |
sambetts | hkominos: ask away | 09:43 |
wagiel | @sambetts: just did the tap int, unfortunately silence on port 67, generally there is traffic but no 67 | 09:45 |
hkominos | hi sambetts:I am trying to deploy Ironic with FUEL. For the openstack nodes ,all the trafic except the pXE is Vlan Tagged. I was told yesterday that the ironic slave (Not the conductor,i mean the machine which I am trying to deploy) must be able to receive all teh traffic. SO i changed the switch configuration and all the Tagged traffic and the PXE is allowed on another port which reaches the NIC of my machine. However I am | 09:46 |
hkominos | I am assuming that the provisioning works kinda like the provisioning on the machines for FUEL: they boot through pxe and then they get an image | 09:48 |
*** sbog has joined #openstack-ironic | 09:49 | |
sbog | Hey guys | 09:49 |
sbog | I wanna ask if there a way to use Ironic without other openstack services? Just standalone installation to provision baremetal nodes. | 09:50 |
sbog | As I understand I can use noauth to get rid of keystone and use my own dhcp for pointing new nodes to Ironic. But what about images? And maybe there are other problems? | 09:51 |
sambetts | hkominos: Right now, all Ironic slaves have to run on an access port, there is no VLAN awareness on the node at any point during provisioning | 09:52 |
*** Nisha_away has quit IRC | 09:52 | |
sambetts | wagiel: none of my test nodes are running right now so I can't compare setups, are your DHCP packets coming in on br-eth1 untagged? | 09:54 |
sambetts | sbog: Yes, totally possible :) https://github.com/openstack/bifrost | 09:55 |
sbog | sambetts: got it, thanks for the link :) | 09:55 |
_milan_ | morning sambetts hkominos vsaienko sbog and all :) | 09:56 |
vsaienko | morning _milan_ | 09:56 |
vsaienko | morning hkominos | 09:56 |
hkominos | goodmorning vsaienko | 09:56 |
_milan_ | inspector guys: FYI I've just submitted an infra pull request for experimental grenade testing to enhance debugging experience ;) https://review.openstack.org/#/c/331001/ | 09:57 |
* sambetts might just disappear suddenly, hoping the thunderstorm going on outside won't knock out his power | 09:58 | |
_milan_ | sambetts, you unplugging mains or risking lightning impact ;) | 09:59 |
sambetts | _milan_: not sure :/ /me really needs to get a UPS | 09:59 |
* _milan_ usually feels an urge to go unplug all | 10:00 | |
sambetts | vdrok: ping | 10:00 |
_milan_ | even fridge :) | 10:00 |
hkominos | sambetts: So the ironic slave must receive only utagged traffic? | 10:00 |
sambetts | hkominos: yes | 10:00 |
vdrok | morning ironic! | 10:00 |
vdrok | sambetts: pong | 10:00 |
_milan_ | hey vdrok :) | 10:00 |
vdrok | morning _milan_ ! | 10:01 |
sambetts | vdrok: Re: https://review.openstack.org/#/c/315766/11/ironic/api/controllers/v1/node.py, the UT for validate with node name works, how were you testing it/ | 10:01 |
sambetts | ? | 10:01 |
vdrok | sambetts: on devstack :) | 10:01 |
hkominos | But all the traffic in Fuel is either tagged or Untagged like the PXE. So you are saying that allowing all the Vlans and the Untagged PXE on the ironic slaves interface will not work since it is not the same | 10:01 |
sambetts | vdrok: I mean, curl or ironiclient or ... | 10:02 |
vdrok | ironicclient | 10:02 |
vdrok | https://www.irccloud.com/pastebin/aYJJe5tB/validate-failure | 10:03 |
*** openstackgerrit has quit IRC | 10:03 | |
vdrok | so it gets passed to node_uuid instead of node | 10:03 |
*** openstackgerrit has joined #openstack-ironic | 10:03 | |
vdrok | the query itself is correct - curl -g -i -X GET http://127.0.0.1:6385/v1/nodes/node-0/validate | 10:04 |
*** rcernin has quit IRC | 10:05 | |
sambetts | vdrok: errr, our UT tests a different URL ... /nodes/validate?node=%s | 10:05 |
vdrok | heh | 10:06 |
vdrok | we should test both I think | 10:06 |
vdrok | both are correct ways to validate | 10:06 |
sambetts | vdrok: depends which we want to support | 10:06 |
vdrok | iirc it was agreed upon long time ago to support both | 10:06 |
vdrok | lucasagomes: dtantsur|bbl maybe you remember ^ | 10:07 |
* lucasagomes reads | 10:07 | |
lucasagomes | vdrok, yeah, sounds correct we should support both | 10:08 |
sambetts | vdrok: adding a UT for it to see if I can fix | 10:08 |
sambetts | sigh... now that is duplicated API :-P | 10:09 |
vdrok | sambetts: I mean, I guess we can do it the same way as with port controller, just add a parameter to the constructor? | 10:09 |
vdrok | of states controller etc | 10:09 |
sambetts | vdrok: we shouldn't need to | 10:09 |
openstackgerrit | Davanum Srinivas (dims) proposed openstack/ironic: [WIP] Testing latest u-c https://review.openstack.org/318440 | 10:10 |
*** e0ne has joined #openstack-ironic | 10:12 | |
*** YiboCai has quit IRC | 10:14 | |
hkominos | Can somebody tell me which fuel networks does an ironic slave need access to? | 10:18 |
vsaienko | hkominos: I assume you configured some vlan for baremetal network, lets say this vlan is 333. Ironic-conductor will send baremetal traffic tagged by vlan 333 on the nic to which baremetal network is pinned on the node, lets say it is nic eth1 and plugged to gig0/1 on the switch SW-A. You have to configure Gig0/1 on SW-A to trunk mode and allow 333 vlan. Ironic slave is connected to SW-A port gig0/2, this port should be configured in access mode | 10:19 |
hkominos | So should Ironic have its own PXE network? | 10:21 |
hkominos | (ironic slave) | 10:22 |
*** hoangcx_ has quit IRC | 10:22 | |
*** zhs_ss has joined #openstack-ironic | 10:23 | |
vsaienko | baremetal network is ironic PXE network in fuel | 10:23 |
*** e0ne has quit IRC | 10:24 | |
zhs_ss | hi everyone, I built a fedora image with disk-image-builder. What is the username/password by default? | 10:24 |
sambetts | zhs_ss: deploy ramdisk or tenant image? | 10:25 |
vsaienko | ironic slave should receive untagged frames from baremetal network, eg access port mode/ | 10:25 |
zhs_ss | tenant image | 10:25 |
zhs_ss | sambetts:tenant image | 10:25 |
sambetts | zhs_ss: there isn't a default username and password, unless you include the devuser element | 10:25 |
hkominos | vsaienko:I think I understand. So the ironic slaves needs access only to the Baremetal network and not the others (FUel,Management etc) | 10:27 |
vsaienko | hkominos: correct | 10:28 |
*** e0ne has joined #openstack-ironic | 10:29 | |
zhs_ss | sambetts:I built the image with the command "disk-image-create fedora vm dhcp-all-interfaces -o my-image", then i boot a instance with it. the instance is up and running. Now I am infront of the baremetal machine trying to login. I didn't notice a 'devuser' option when i built the image... | 10:30 |
lucasagomes | zhs_ss, you can't SSH into the node? | 10:32 |
sambetts | zhs_ss: in a normal OpenStack setup tenant's don't have direct access to the node, so the images are setup for ssh access only, if you want to login directly then you need to add this element to your disk-image-create https://github.com/openstack/diskimage-builder/tree/master/elements/devuser and configured the environment vars | 10:33 |
zhs_ss | lucasagomes:I can get an ssh connection, but I don't know the password | 10:35 |
sambetts | zhs_ss: there isn't a password, its expect that you setup an OpenStack keypair | 10:35 |
zhs_ss | sambetts: thanks a lot, I will check it. | 10:35 |
lucasagomes | zhs_ss, right, well it should be the key you've passed to nova when you booted the instance | 10:35 |
lucasagomes | nova boot ... --key-name <blah> | 10:36 |
zhs_ss | sambetts:lucasagomes: oh, right, i can use keypair :P thanks a lot | 10:37 |
lucasagomes | zhs_ss, right | 10:37 |
lucasagomes | yw | 10:37 |
hkominos | vsaienko:when i run an ironic validate should i get boot :supported? | 10:41 |
openstackgerrit | Satoru Moriya proposed openstack/ironic-specs: Volume connection information for Ironic nodes https://review.openstack.org/200496 | 10:42 |
vdrok | hkominos: no, everything is done by deploy interface | 10:43 |
vdrok | in case of fuel_ipmitool | 10:43 |
*** e0ne has quit IRC | 10:47 | |
*** sabeen1 has joined #openstack-ironic | 10:48 | |
hkominos | still cant get a pxe message but i think the network is set up correctly though. I looked at https://github.com/openstack/fuel-agent/tree/master/contrib/ironic/ironic-fa-deploy and it specifies a driver_info/fuel_key_filename=PRIVATE_KEY | 10:49 |
hkominos | which i dont know what it is and i did not do because it was not shown in the utube tutorial by mirantis. is it necessary? | 10:49 |
lucasagomes | dtantsur|bbl, JayF jroll rloo, so there's 4+2 here https://review.openstack.org/#/c/323511/, I've answered the comments, do you think I need a new patch-set before approval or just take the notes into consideration when implementing? | 10:50 |
lucasagomes | I assume none are critical/high prio since you wouldn't +2 it if it was | 10:50 |
vdrok | a tiny project config patch, please take a look, grenade related :) https://review.openstack.org/331046 | 10:57 |
vdrok | hkominos: you're not required to specify it, it is a key that will be used by driver to ssh into deployment ramdisk | 11:01 |
vdrok | if you're not getting a pxe prompt it's definitely not a problem | 11:01 |
mgould | vdrok: left a couple of comments | 11:01 |
vdrok | mgould: thanks! | 11:01 |
*** gcb has joined #openstack-ironic | 11:03 | |
mgould | vdrok: LGTM | 11:03 |
*** dtantsur|bbl is now known as dtantsur | 11:04 | |
*** smoriya has quit IRC | 11:05 | |
dtantsur | lucasagomes, I'm fine with +A and a follow-up to clarify | 11:05 |
dtantsur | also morning vsaienko, mgould, sambetts and everyone else | 11:05 |
*** wagiel has quit IRC | 11:06 | |
dtantsur | lucasagomes, although I don't think it will take much time to approve after fixing the nits. I can review right away | 11:07 |
mgould | morning dtantsur | 11:09 |
vsaienko | morning dtantusr, lucasagomes, mgould | 11:20 |
mgould | morning vsaienko | 11:23 |
*** hoangcx has joined #openstack-ironic | 11:25 | |
vdrok | dtantsur: lucasagomes could you take a look at https://review.openstack.org/330089 ? a tiny bugfix | 11:25 |
TheJulia | Good morning everyone | 11:25 |
vdrok | morning TheJulia | 11:25 |
mgould | morning TheJulia | 11:26 |
hkominos | goodmorning TheJulia | 11:26 |
openstackgerrit | Merged openstack/ironic-specs: Promote agent vendor passthru to core API https://review.openstack.org/306418 | 11:27 |
openstackgerrit | Pavlo Shchelokovskyy proposed openstack/ironic: Deny some driver_info updates when console is on https://review.openstack.org/330003 | 11:28 |
lucasagomes | dtantsur, fair enough will do it too | 11:28 |
lucasagomes | vdrok, sure | 11:28 |
lucasagomes | vdrok, are you going to put up a new patch-set for that? | 11:32 |
lucasagomes | code lgtm, but in the release note you said you will change the text | 11:33 |
*** gcb has quit IRC | 11:33 | |
vdrok | lucasagomes: I meant the followup, but can do :) | 11:33 |
lucasagomes | vdrok, right, yeah, well it's easier to just put a new patch-set if you are going to | 11:34 |
vdrok | ok, doing now | 11:34 |
*** hoangcx has quit IRC | 11:34 | |
lucasagomes | ok, I will re-add my +2 as soon as it's up | 11:34 |
aarefiev | dtantsur: hey, one question about this https://review.openstack.org/#/c/305864/ | 11:35 |
aarefiev | There is new always_use_boot_interface option, why we can't use deploy disk if introspected one isn't provided? | 11:35 |
aarefiev | in case it True | 11:36 |
hkominos | Is the dhcp server fro the baremetal network supposed to be in the controler on a different namespace? | 11:37 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Properly set ephemeral size in agent drivers https://review.openstack.org/330089 | 11:37 |
vdrok | lucasagomes: done, thanks for review :) | 11:38 |
TheJulia | ugh, looks like the client change for active node creation is wedged https://review.openstack.org/#/c/275767/ anyone want to add a second +A to see if that unwedges it | 11:38 |
lucasagomes | vdrok, thank you for the code, +2 re-added | 11:38 |
*** lucasagomes is now known as lucas-hungry | 11:39 | |
jroll | lucas-hungry: yeah, rloo said follow-up is fine, will approve | 11:43 |
*** hoangcx has joined #openstack-ironic | 11:44 | |
*** SiRiuS__ has joined #openstack-ironic | 11:45 | |
jroll | morning everyone | 11:46 |
*** rcernin has joined #openstack-ironic | 11:47 | |
*** gcb has joined #openstack-ironic | 11:49 | |
TheJulia | good morning jroll | 11:51 |
sambetts | jroll: is the fact we support v1/ports?node= and v1/nodes/<uuid>/ports which are duplicates just because of the way it was implemented, or was that intentional? | 11:52 |
jroll | sambetts: before my time :) | 11:52 |
jroll | I suspect the former, but I'd bet money both are used in the wild | 11:53 |
sambetts | jroll: I've already found similar case, with nodes validate, we support both v1/nodes/<uuid>/validate and v1/nodes/validate?node= and the UTs tested one version and the ironic client used the other... | 11:54 |
*** sabeen1 has quit IRC | 11:54 | |
jroll | yep | 11:54 |
jroll | I'm inclined to just leave those inconsistencies to be fixed in v2 | 11:54 |
vdrok | morning jroll | 11:56 |
jroll | \o | 11:56 |
openstackgerrit | Merged openstack/ironic-specs: Collect system logs from IPA https://review.openstack.org/323511 | 11:57 |
openstackgerrit | Vladyslav Drok proposed openstack/ironic: Change the logic of selecting image for tests https://review.openstack.org/329625 | 11:58 |
openstackgerrit | Julia Kreger proposed openstack/ironic: Follow-up to Active Node Creation https://review.openstack.org/331107 | 11:59 |
*** dmk0202 has quit IRC | 12:05 | |
openstackgerrit | Merged openstack/python-ironicclient: Client addition for Active Node Creation verb adopt https://review.openstack.org/275767 | 12:06 |
*** e0ne has joined #openstack-ironic | 12:07 | |
*** mtanino has joined #openstack-ironic | 12:07 | |
sambetts | jroll: makes sense :( | 12:08 |
dtantsur | morning TheJulia, jroll | 12:19 |
jroll | hey dtantsur :) | 12:19 |
mgould | morning jroll | 12:19 |
sambetts | hey dtantsur | 12:19 |
*** dprince has joined #openstack-ironic | 12:19 | |
jroll | \o mgould | 12:19 |
TheJulia | good morning dtantsur | 12:20 |
openstackgerrit | Merged openstack/ironic: Change port used for Ironic static http to 3928 https://review.openstack.org/330839 | 12:23 |
openstackgerrit | Dmitry Tantsur proposed openstack/python-ironicclient: Add --wait flag for provision actions and wait_for_provision_state function https://review.openstack.org/328191 | 12:24 |
dtantsur | plz review ^^^ | 12:24 |
_milan_ | morning jroll TheJulia | 12:25 |
_milan_ | and mgould ;) | 12:26 |
jroll | \o | 12:26 |
mgould | morning _milan_ | 12:28 |
_milan_ | :) | 12:28 |
*** baoli has joined #openstack-ironic | 12:34 | |
*** baoli_ has joined #openstack-ironic | 12:36 | |
*** piet has joined #openstack-ironic | 12:37 | |
*** hoangcx has quit IRC | 12:38 | |
*** hoangcx has joined #openstack-ironic | 12:39 | |
*** alaski is now known as lascii | 12:40 | |
*** baoli has quit IRC | 12:40 | |
*** baoli_ has quit IRC | 12:41 | |
*** hoangcx has quit IRC | 12:43 | |
*** rbudden has joined #openstack-ironic | 12:48 | |
dtantsur | sambetts, are you fine with leaving NMI in https://review.openstack.org/186700 ? I'm going to review the spec now, but I'd like to know if your concerns are addresses by naohirot's comments | 12:49 |
*** vmud213 has quit IRC | 12:49 | |
*** lucas-hungry is now known as lucasagomes | 12:50 | |
lucasagomes | jroll, cheers! I will put up a follow up patch soon-ish (after a call in 10m) | 12:51 |
jroll | cool, thanks man | 12:51 |
lucasagomes | jroll, TheJulia sambetts morning :-) | 12:51 |
TheJulia | good morning lucasagomes | 12:52 |
dtantsur | sambetts, also please review a one-liner fixing our tempest: https://review.openstack.org/330955 | 12:53 |
*** ccamacho is now known as ccamacho|lunck | 12:53 | |
*** ccamacho|lunck is now known as ccamacho|lunch | 12:54 | |
jroll | dtantsur: merged | 12:54 |
jroll | er, approved | 12:54 |
dtantsur | thnx | 12:54 |
sambetts | heh jroll was faster than me | 12:54 |
_milan_ | vdrok, vsaienko, jlvillal I've added you guys to the Inspector grenade patches review, feel free to disagree ;) | 12:55 |
_milan_ | aarefiev, you interested as well? ^ | 12:55 |
_milan_ | sambetts, probably ^ ;) | 12:56 |
*** raildo has joined #openstack-ironic | 12:56 | |
sambetts | thanks _milan_ | 12:56 |
sambetts | dtantsur: I'm still not sure, because when I was looking around sometimes a NMI can lead to a power off (halt) or to a reboot, and so sometimes it'll be a power_on to power_on and sometimes it'll be power_on to power_off, and unless we can detect which then in the case of power_on to power_off Ironic is likely to try to turn the node back on, and I'm not sure that would be the expect behaviour | 12:58 |
sambetts | or not | 12:58 |
dtantsur | sambetts, oh, I see | 13:00 |
dtantsur | sambetts, then I'm not sure how to implement NMI at all... or alternatively we can define some policy: e.g. the power state should eventually become the same | 13:00 |
dtantsur | so e.g. starting with power on we should end up with power on again (and enforce if it does not happen) | 13:01 |
dtantsur | but yeah, I'd probably split NMI away from this spec | 13:01 |
*** [1]cdearborn has joined #openstack-ironic | 13:02 | |
*** gcb has quit IRC | 13:02 | |
sambetts | dtantsur: I've not got any experience with NMI, everything I'm quoting here is just from some google fu into the subject, but based on what I've seen it worries me | 13:03 |
dtantsur | yeah. I've suggested the split. you can vote in favor of it, if you feel like | 13:03 |
dtantsur | I think having soft power off in API is valuable and more or less clear | 13:03 |
dtantsur | jroll, do you have any experience with NMI? | 13:03 |
*** trown|outtypewww is now known as trown | 13:04 | |
dtantsur | (this is re https://review.openstack.org/186700) | 13:04 |
NobodyCam | good morning Ironicers ... | 13:04 |
NobodyCam | and ofc... | 13:04 |
dtantsur | morning NobodyCam. TGIF! | 13:04 |
sambetts | NobodyCam: TGIF :D | 13:04 |
dtantsur | we | 13:04 |
NobodyCam | yep :P | 13:04 |
dtantsur | we're faster :D | 13:04 |
NobodyCam | morning dtantsur sambetts :) | 13:04 |
NobodyCam | lol | 13:04 |
*** zhs_ss has quit IRC | 13:05 | |
* dtantsur looks at all the comments on the driver composition spec..... | 13:05 | |
*** wajdi has quit IRC | 13:06 | |
sambetts | dtantsur: I'm so sorry I've not got round to reviewing that yet :( everytime I load the tab, I get a internal ping | 13:06 |
dtantsur | heh, happens | 13:08 |
xavierr | $ ironic --all good-morning | 13:08 |
*** thiagop has joined #openstack-ironic | 13:10 | |
thiagop | Good morning and TGIF, Ironic | 13:11 |
mgould | morning thiagop xavierr | 13:12 |
NobodyCam | morning xavierr thiagop mgould :) | 13:15 |
TheJulia | xavierr: Good morning! | 13:16 |
*** jvgrant_ has joined #openstack-ironic | 13:18 | |
jroll | dtantsur: zero :) | 13:23 |
openstackgerrit | Julia Kreger proposed openstack/ironic: Follow-up to Active Node Creation https://review.openstack.org/331107 | 13:25 |
*** joprovost has joined #openstack-ironic | 13:29 | |
*** mjturek1 has quit IRC | 13:29 | |
*** joprovost1 has joined #openstack-ironic | 13:31 | |
openstackgerrit | Merged openstack/bifrost: split ironic install role into install,bootstrap,start phases https://review.openstack.org/318423 | 13:32 |
openstackgerrit | Merged openstack/ironic-inspector: Fix tempest tests https://review.openstack.org/330955 | 13:32 |
*** joprovost has quit IRC | 13:33 | |
*** joprovost1 is now known as joprovost | 13:33 | |
*** mtanino has quit IRC | 13:35 | |
NobodyCam | morning jroll :) | 13:35 |
jroll | ohai NobodyCam :) | 13:35 |
NobodyCam | :) | 13:35 |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-specs: The driver composition reform https://review.openstack.org/188370 | 13:40 |
dtantsur | fun continues ^^^ | 13:40 |
jroll | yay | 13:41 |
dtantsur | TODOs are still left, the other comments seem resolved | 13:41 |
* dtantsur works on filling in TODOs now | 13:41 | |
* jroll only wanted the todos done :( | 13:41 | |
*** dansmith is now known as superdan | 13:41 | |
dtantsur | then stay tuned | 13:41 |
jroll | will do | 13:41 |
dtantsur | jroll, there is one logic change around /driver vs /hardware_type fields | 13:42 |
jroll | dtantsur: I'll just wait for the todos if that's okay with you | 13:42 |
dtantsur | sure | 13:42 |
*** piet has quit IRC | 13:45 | |
vdrok | morning NobodyCam xavierr and thiagop | 13:46 |
*** mgoddard__ has joined #openstack-ironic | 13:46 | |
xavierr | vdrok o/ | 13:46 |
*** links has quit IRC | 13:46 | |
*** piet has joined #openstack-ironic | 13:48 | |
*** piet has quit IRC | 13:49 | |
*** mgoddard_ has quit IRC | 13:49 | |
*** wajdi has joined #openstack-ironic | 13:49 | |
*** fragatina has joined #openstack-ironic | 13:49 | |
*** PollyZ has joined #openstack-ironic | 13:50 | |
NobodyCam | morning vdrok :) | 13:50 |
*** Goneri has joined #openstack-ironic | 13:50 | |
*** ametts has joined #openstack-ironic | 13:50 | |
*** fragatin_ has joined #openstack-ironic | 13:50 | |
*** e0ne has quit IRC | 13:51 | |
*** e0ne has joined #openstack-ironic | 13:52 | |
*** wajdi has quit IRC | 13:53 | |
dtantsur | jroll, can you think of any RPC impact in the driver composition? I can't think of anything.. | 13:53 |
*** wajdi has joined #openstack-ironic | 13:53 | |
*** fragatina has quit IRC | 13:54 | |
jroll | dtantsur: get_topic_for_node will probably need to change, but I don't think anything major | 13:54 |
dtantsur | aha | 13:54 |
jroll | ah that would all be in the hash ring | 13:55 |
*** Goneri has quit IRC | 13:55 | |
jroll | which in the end looks like dbapi.get_active_driver_dict() | 13:56 |
jroll | if you're just adding hardware_type to that then should be fine? | 13:56 |
dtantsur | I think so | 13:56 |
*** joprovost has quit IRC | 13:58 | |
*** joprovost1 has joined #openstack-ironic | 13:58 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-specs: The driver composition reform https://review.openstack.org/188370 | 14:00 |
dtantsur | jroll, this does not have TODO text ^^^ | 14:00 |
jroll | \o/ | 14:00 |
jroll | thanks | 14:00 |
dtantsur | it's possible I missed something, but now not intentionally :) | 14:00 |
*** joprovost1 is now known as joprovost | 14:00 | |
dtantsur | jroll, the spec is merged, marking https://bugs.launchpad.net/ironic/+bug/1570841 as rfe-approved? | 14:01 |
openstack | Launchpad bug 1570841 in Ironic "[RFE] Promote agent vendor passthru to core API" [Wishlist,In progress] - Assigned to Dmitry Tantsur (divius) | 14:01 |
jroll | dtantsur: go ahead :) | 14:01 |
dtantsur | awesome! the first patch is ready for review, when somebody has time: https://review.openstack.org/#/c/330938/ | 14:01 |
jroll | nice | 14:02 |
sambetts | \0/ | 14:03 |
*** PollyZ has quit IRC | 14:03 | |
*** athomas has quit IRC | 14:06 | |
*** PollyZ has joined #openstack-ironic | 14:07 | |
dtantsur | hmm, the driver composition sounds like a possible summit talk, doesn't it? (assuming we land it, lol) | 14:07 |
* dtantsur will submit | 14:09 | |
*** tloeb__ has quit IRC | 14:09 | |
*** tloeb_ has joined #openstack-ironic | 14:09 | |
*** tloeb_ is now known as tloeb__ | 14:09 | |
jroll | dtantsur++ | 14:09 |
*** athomas has joined #openstack-ironic | 14:10 | |
*** SiRiuS__ has quit IRC | 14:10 | |
*** PollyZ has quit IRC | 14:12 | |
*** sdake has joined #openstack-ironic | 14:12 | |
*** joprovost1 has joined #openstack-ironic | 14:14 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic-inspector: Updated from global requirements https://review.openstack.org/331165 | 14:15 |
jlvillal | Good morning Ironic | 14:15 |
NobodyCam | morning jlvillal | 14:15 |
jlvillal | Big thanks to JayF for figuring out the root cause of the gate breakage last night :) | 14:15 |
*** joprovost has quit IRC | 14:16 | |
*** joprovost1 is now known as joprovost | 14:16 | |
thiagop | jlvillal: what was the cause? | 14:16 |
thiagop | jlvillal: morning | 14:16 |
jlvillal | thiagop, Infra was brining up the new Zuul system. Slowly. | 14:16 |
jlvillal | s/brining/bringing/ | 14:17 |
_milan_ | morning thiagop NobodyCam jlvillal | 14:17 |
NobodyCam | morning _milan_ | 14:17 |
jlvillal | thiagop, In the new Zuul system they are using port 8088 to allow access to the console long. Kind of cool you can telnet to port 8088 and see the log. | 14:17 |
thiagop | jlvillal: but it does conflict with the ports we are using, right? | 14:18 |
jlvillal | thiagop, But we (Ironic) are using port 8088, so anytime our job hit the new Zuul systems they broke | 14:18 |
jlvillal | thiagop, JayF pushed a patch last night to move our port out of the way. | 14:18 |
thiagop | jlvillal: what do we use 8088 for, anyway? It's not the main port of the Ironic API as far as I remember... | 14:20 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/python-ironicclient: Updated from global requirements https://review.openstack.org/331179 | 14:20 |
*** jvgrant_ has quit IRC | 14:21 | |
*** mtanino has joined #openstack-ironic | 14:21 | |
jlvillal | thiagop, We use it for Ironic in devstack. | 14:22 |
*** ccamacho|lunch is now known as ccamacho | 14:22 | |
jlvillal | FYI: I updated the whiteboard to move the gate to golden. And moved the information on the gate failure to the historical page. | 14:23 |
thiagop | jlvillal: so it's just devstack stuff, we won't need to change a lot of docs saying our port has changed, right? | 14:23 |
jlvillal | thiagop, No, only devstack | 14:23 |
thiagop | terrific | 14:23 |
jlvillal | thiagop, This was the fix: https://review.openstack.org/#/c/330832/ | 14:24 |
jlvillal | It has also been backported to stable/mitaka. So the grenade jobs will now work also | 14:24 |
jlvillal | And a proposed fix to stable/liberty, but that has to go through the devstack project. | 14:24 |
*** SiRiuS__ has joined #openstack-ironic | 14:27 | |
vdrok | morning jlvillal | 14:31 |
jlvillal | vdrok, Good morning :) | 14:31 |
vdrok | jlvillal: btw - https://review.openstack.org/331046 | 14:31 |
jlvillal | vdrok, Darn you. You beat me to it ;) | 14:32 |
vdrok | :) | 14:32 |
*** david-lyle has joined #openstack-ironic | 14:32 | |
mgould | morning jlvillal | 14:32 |
jlvillal | vdrok, | 14:33 |
jlvillal | - - name: ^gate-grenade-dsvm-ironic$ | 14:33 |
jlvillal | + - name: ^gate-grenade-dsvm-ironic(|-nv)$ | 14:33 |
*** skramaja has quit IRC | 14:33 | |
jlvillal | vdrok, My thought. | 14:33 |
jlvillal | vdrok, I didn't get around to posting it. | 14:33 |
jlvillal | vdrok, But yours works :) | 14:33 |
vdrok | jlvillal: what about partial? I don't remember how it was called | 14:33 |
dtantsur | morning jlvillal | 14:33 |
jlvillal | mgould, Morning | 14:33 |
*** e0ne has quit IRC | 14:34 | |
jlvillal | vdrok, We won't have a partial. It will now be ????-multinode-???? | 14:34 |
*** PollyZ has joined #openstack-ironic | 14:34 | |
jlvillal | vdrok, partial does not exist/work anymore for grenade | 14:34 |
jlvillal | dtantsur, morning | 14:34 |
vdrok | jlvillal: yeah, but I thought that this may match the new job too | 14:34 |
jlvillal | vdrok, I'm not sure. I'm not sure if it even has the word grenade in the job, from looking at Nova jobs. | 14:35 |
*** e0ne has joined #openstack-ironic | 14:35 | |
jlvillal | vdrok, I'm happy with your patch :) | 14:35 |
jlvillal | vdrok, +1 from me | 14:36 |
vdrok | gate-grenade-dsvm-partial-ironic, so yeah, it won't match by that. OK, thanks :) | 14:36 |
jroll | dtantsur: re-reviewed | 14:36 |
jroll | thiagop: to more fully answer your question, apache is used for serving ipxe images in our devstack jobs | 14:38 |
thiagop | jroll: aww... | 14:38 |
jroll | vsaienko: lol, https://review.openstack.org/#/c/329658/ | 14:38 |
jroll | er, vdrok ^^ | 14:38 |
*** SiRiuS__ has quit IRC | 14:38 | |
jroll | vdrok: also, yeah we don't have partial anymore https://review.openstack.org/#/c/329553/ | 14:39 |
dtantsur | jroll, thanks, will address soon | 14:39 |
vdrok | jroll: :) yeah, I know, I was thinking to make it more general so that newly added grenade jobs can match it too | 14:40 |
jroll | vdrok: indeed | 14:40 |
*** sacharya has joined #openstack-ironic | 14:40 | |
*** tosc_fiberdata has quit IRC | 14:41 | |
*** dprince has quit IRC | 14:42 | |
openstackgerrit | Xavier proposed openstack/ironic: Add Dynamic Allocation feature for the OneView drivers https://review.openstack.org/286192 | 14:42 |
xavierr | reviews here would be great :) ^^^ | 14:43 |
vdrok | dtantsur: you're too fast, it will be a second revision you've pushed while I'm reviewing :) or I am too slow | 14:43 |
*** dprince has joined #openstack-ironic | 14:43 | |
*** baoli has joined #openstack-ironic | 14:45 | |
dtantsur | vdrok, lol :) I'll wait for you to leave the comments then :) | 14:45 |
*** skramaja has joined #openstack-ironic | 14:46 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-specs: Follow-up patch of 8068e1f6d92d22bf25a56b8327c1abd0821b1bcd https://review.openstack.org/331200 | 14:47 |
lucasagomes | follow-up on the logs spec ^ thanks for the reviews! | 14:47 |
*** sacharya_ has joined #openstack-ironic | 14:47 | |
*** sacharya has quit IRC | 14:50 | |
dtantsur | jroll, responded. will update when I address vdrok's comments (if any) | 14:51 |
jroll | dtantsur: cool | 14:53 |
*** skramaja has quit IRC | 14:53 | |
*** krtaylor has joined #openstack-ironic | 14:53 | |
dtantsur | jroll, I remember vdrok has a valid question: do we want to restrict node states in which we allow updating hardware type and interfaces? | 14:54 |
dtantsur | jroll, we currently allow updating a driver whenever we allow updates at all | 14:54 |
*** pcaruana has quit IRC | 14:54 | |
vdrok | yeah, that is one of the comments I have now, maybe it's OK to leave as is now, and fix as a bug afterwards | 14:54 |
jroll | dtantsur: hm, does seem like a good idea, but then they can still go through the driver to get around it :( | 14:54 |
*** hoangcx has joined #openstack-ironic | 14:55 | |
*** hoangcx has quit IRC | 14:55 | |
*** hoangcx has joined #openstack-ironic | 14:55 | |
dtantsur | jroll, true; still it's a chance to make the new API somewhat saner | 14:55 |
dtantsur | but I really don't know if it's a good idea or not | 14:56 |
dtantsur | e.g. there maybe be nothing wrong in changing the power interface even in active state | 14:56 |
jroll | lucasagomes: one nit | 14:56 |
jroll | dtantsur: yeah, I'd say save it for later | 14:56 |
lucasagomes | jroll, sigh, good catch heh | 14:57 |
dtantsur | vdrok, are you ok with ^^^ | 14:57 |
vdrok | dtantsur: yep, OK with fixing later. we also need to decide which ones should be restricted, power should not ofc :) | 14:57 |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic-specs: Follow-up patch of 8068e1f6d92d22bf25a56b8327c1abd0821b1bcd https://review.openstack.org/331200 | 14:58 |
dtantsur | yeah... this spec is complex enough already | 14:58 |
*** mgoddard has joined #openstack-ironic | 14:58 | |
*** mgoddard__ has quit IRC | 14:58 | |
vdrok | ah, dtantsur you've replied about disabled hw types | 15:02 |
dtantsur | yep | 15:03 |
dtantsur | lets leave them alone for now | 15:03 |
*** tesseract has quit IRC | 15:05 | |
*** rcernin has quit IRC | 15:07 | |
*** rama_y has joined #openstack-ironic | 15:08 | |
*** dprince has quit IRC | 15:08 | |
*** amotoki has joined #openstack-ironic | 15:09 | |
*** dprince has joined #openstack-ironic | 15:09 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic-specs: The driver composition reform https://review.openstack.org/188370 | 15:09 |
dtantsur | jroll, vdrok, ^^^ | 15:09 |
dtantsur | hopefully I didn't miss anything | 15:09 |
*** dprince has quit IRC | 15:11 | |
*** electrofelix has quit IRC | 15:15 | |
openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Add power_transition_started_at to the nodes table https://review.openstack.org/327642 | 15:18 |
*** sdake has quit IRC | 15:20 | |
*** ipukha has quit IRC | 15:23 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: [WIP] Promote agent vendor passthru to core API https://review.openstack.org/330018 | 15:24 |
*** mgoddard_ has joined #openstack-ironic | 15:27 | |
*** mgoddard has quit IRC | 15:31 | |
*** rajinir has joined #openstack-ironic | 15:32 | |
*** marios has quit IRC | 15:33 | |
*** ijw has joined #openstack-ironic | 15:34 | |
*** SiRiuS__ has joined #openstack-ironic | 15:36 | |
rajinir | thiago: I missed the IRC qa meeting. Chris mentioned about the Thirdpart CI talk for the summit. I'm interested to join and share the experiences | 15:38 |
rajinir | thiagop: I missed the IRC qa meeting. Chris mentioned about the Thirdpart CI talk for the summit. I'm interested to join and share the experiences | 15:38 |
*** Nisha_away has joined #openstack-ironic | 15:39 | |
dtantsur | fg | 15:42 |
dtantsur | oops | 15:42 |
NobodyCam | hehehe | 15:43 |
*** ohamada has quit IRC | 15:43 | |
*** flwang has quit IRC | 15:44 | |
*** flwang has joined #openstack-ironic | 15:48 | |
*** praneshp has quit IRC | 15:49 | |
lucasagomes | dtantsur, sambetts re: soft reboot/NMI... It seems that injecting NMI does causes the kernel to reboot | 15:50 |
lucasagomes | so I'm good with that being a power state action, not sure if we need a split on that spec | 15:50 |
* lucasagomes voted accordingly with ^ | 15:51 | |
*** sdake has joined #openstack-ironic | 15:51 | |
dtantsur | maybe? I don't know, hence I'm unable to vote properly | 15:52 |
dtantsur | lucasagomes, it's the same for windows and other major OS, right? | 15:52 |
sambetts | lucasagomes: I think Windows just Halts | 15:52 |
lucasagomes | sambetts, it restarts | 15:53 |
lucasagomes | https://support.microsoft.com/en-us/kb/927069 | 15:53 |
lucasagomes | also, nova seems to expect the instance to reboot with nmi as well | 15:54 |
lucasagomes | https://specs.openstack.org/openstack/nova-specs/specs/mitaka/implemented/instance-crash-dump.html | 15:54 |
lucasagomes | so, I think it's aligned with the interfaces | 15:54 |
*** _milan_ has quit IRC | 15:57 | |
sambetts | lucasagomes: I don't see where on that page it says it restarts, it says you need to restart to enable a setting in the registery | 15:58 |
sambetts | lucasagomes: https://support.microsoft.com/en-gb/kb/2750146 | 15:58 |
*** mgoddard_ has quit IRC | 15:59 | |
*** mgoddard has joined #openstack-ironic | 15:59 | |
sambetts | lucasagomes: this page says it may restart based on settings configured by the user in the Advanced System settings | 15:59 |
lucasagomes | sambetts, right yeah I think it's possible to configure windows to enter in a sorta of "debug" mode with it as well | 16:00 |
lucasagomes | "With the introduction of Windows 2000, Microsoft allowed the use of an NMI to cause a system to either break into a debugger, or dump the contents of memory to disk and reboot.[2]" | 16:00 |
lucasagomes | https://en.wikipedia.org/wiki/Non-maskable_interrupt | 16:00 |
lucasagomes | sambetts, but in any case, both soft reboot or nmi requires the some configuration | 16:01 |
lucasagomes | so, it's same-ish | 16:01 |
lucasagomes | IMO | 16:01 |
sambetts | well soft reboot Ironic should expect it to go power on to power on because thats what a reboot should do, but with NMI there doesn't appear to me a "this is what it should do" but rather is down to a number of facots | 16:03 |
sambetts | facotrs* | 16:03 |
*** [1]cdearborn has quit IRC | 16:05 | |
*** jvgrant_ has joined #openstack-ironic | 16:06 | |
openstackgerrit | Dmitry Tantsur proposed openstack/ironic: [WIP] Promote agent vendor passthru to core API https://review.openstack.org/330018 | 16:06 |
lucasagomes | sambetts, right, the soft power boot in that spec does include a timeout | 16:06 |
lucasagomes | maybe we need something similar for nmi? To make sure it's being rebooted? | 16:06 |
lucasagomes | tho it's strange... :-/ | 16:07 |
*** sdake has quit IRC | 16:07 | |
sambetts | lucasagomes: but its down to the tenant on the box to determine if it should be rebooted or not | 16:07 |
openstackgerrit | edan david proposed openstack/ironic-python-agent: Use assertIn and assertNotIn https://review.openstack.org/330451 | 16:07 |
dtantsur | calling it a day folks, have a great weekend | 16:08 |
dtantsur | please don't forget to review the driver composition spec ;) | 16:08 |
lucasagomes | idk, even with the dependency on some factors, it seems that a) nova interface expects a reboot b) in the linux case the node is reboot | 16:08 |
sambetts | lucasagomes: because if they've set no reboot in windows, is it right that Ironic try to turn it back on | 16:08 |
dtantsur | and see you on midcycle | 16:08 |
lucasagomes | seems enough for having it in the power actions | 16:08 |
lucasagomes | c) ipmitool also has it as a power action | 16:08 |
lucasagomes | dtantsur, enjoy the weekend! | 16:08 |
vdrok | night dtantsur | 16:08 |
*** dtantsur is now known as dtantsur|afk | 16:08 | |
*** dmk0202 has joined #openstack-ironic | 16:09 | |
sambetts | Ilo has it a diagnostic setting | 16:09 |
sambetts | not a power one | 16:09 |
*** mgoddard has quit IRC | 16:09 | |
lucasagomes | right, but our api was heavily influenced on ipmitool (the name of boot devices and all) so I think it does fit | 16:10 |
* lucasagomes is starting thinking about the split making sense now after the discussion heh | 16:10 | |
sambetts | I almost think we should should disable Ironic force power state for that node if someone fires an NMI, and allow that nodes power state to be read by Ironic instead of set | 16:10 |
*** hoangcx has quit IRC | 16:11 | |
sambetts | then the node is left alone to do whatever its going to do when it received the NMI | 16:11 |
*** Romanenko_K has quit IRC | 16:11 | |
*** kromanenko__ has quit IRC | 16:11 | |
*** kromanenko_ has quit IRC | 16:11 | |
sambetts | and then turn back on force power state on the next power state change | 16:13 |
lucasagomes | "ironic force power state" the configuration option ? | 16:14 |
sambetts | so the node will either go power on -> power off -> power on, or power on -> power off -> tenant does nova power on my instance -> power on | 16:14 |
sambetts | lucasagomes: yeah, disable it for that node for one power state change if an NMI is requested for that node | 16:15 |
lucasagomes | sambetts, right, so that config is global at the moment isn't it? | 16:15 |
lucasagomes | I can't disable for a node specifically, unless we set it into maintenance | 16:15 |
sambetts | lucasagomes: it is, but we could add an if into the sync loop thats like if config force power state && node not in NMI: | 16:16 |
lucasagomes | hmm maybe yeah, but, I honestly prefer it to be simple | 16:17 |
lucasagomes | NMI is used in an extreme case where the kernel is locked up | 16:17 |
lucasagomes | I'm not sure we can really expect a very well defined behavior on those cases | 16:17 |
lucasagomes | neither I think we should make it super complicated, if the instance is not configured for rebooting after NMI is inject, the user still can go and do a hard reboot | 16:18 |
lucasagomes | it's not a big deal | 16:18 |
sambetts | lucasagomes: its not that the instance isn't configured for rebooting and therefore didn't reboot, its that Ironic will try to turn it back on if it powers off | 16:19 |
*** edand has joined #openstack-ironic | 16:20 | |
sambetts | or will error if Ironic is expecting it to go back to power on | 16:21 |
*** rbrud has joined #openstack-ironic | 16:21 | |
lucasagomes | sambetts, yeah, but that's still aligned with what is expected when the user does a "nova inject-nmi" right? | 16:21 |
lucasagomes | if it's off but is marked as on, ironic will put it back on | 16:21 |
*** rbrud has quit IRC | 16:21 | |
lucasagomes | and nova does expect the instance to reboot | 16:21 |
lucasagomes | maybe we should think about it over beer sambetts :-) | 16:22 |
* lucasagomes gives sambetts a pint of guinness | 16:23 | |
sambetts | ;) heh yeah, I need to read the nova spec, because I wonder if they've considered a case where the instance doesn't reboot automatically | 16:23 |
sambetts | lucasagomes: have you had any of the BrewDog beers? | 16:23 |
lucasagomes | sambetts, yeah, maybe some more investigation on what nova expect, all info I have is that spec and the matrix of the drivers that supports nmi injection | 16:23 |
*** jvgrant_ has quit IRC | 16:23 | |
sambetts | lucasagomes: same, until reading that spec I didn't even know NMI was a thing | 16:24 |
lucasagomes | sambetts, I think I did yes, it's two scottish guys that brew it isn't it? | 16:24 |
lucasagomes | they have a tv show or something | 16:24 |
lucasagomes | sambetts, yup, same for me | 16:24 |
sambetts | lucasagomes: I know its scottish, no idea about the tv show though | 16:24 |
*** mbound has joined #openstack-ironic | 16:25 | |
lucasagomes | sambetts, http://www.imdb.com/title/tt2953726/ | 16:25 |
mat128 | about NMI and the power states, don't forget drivers like SNMP that actually control the power outlet | 16:25 |
mat128 | I presume the outlet must (and will) stay "on" even during NMI | 16:25 |
lucasagomes | I've watched it on the plane one day | 16:25 |
*** praneshp has joined #openstack-ironic | 16:25 | |
sambetts | lucasagomes: heh, thats cool | 16:26 |
*** praneshp has joined #openstack-ironic | 16:26 | |
sambetts | mat128: I'm not sure that driver would be able to support that power state, or soft power off/reboot | 16:26 |
lucasagomes | mat128, right on that good info. That spec is about IPMIPower supporting it but def we should make the interface in ironic to work with all drivers | 16:26 |
lucasagomes | if possible | 16:27 |
*** e0ne has quit IRC | 16:27 | |
lucasagomes | mat128, is injecting NMI possible via SNMP ? | 16:27 |
mat128 | I think soft power control isn't possible with SNMP (think of it as a light switch) | 16:27 |
mat128 | lucasagomes: NMI is (potentially) injected by IPMI | 16:27 |
mat128 | but it's effect on power state isn't clear | 16:27 |
mat128 | if the node reboots on it's own, I'm sure IPMI still reports "on" | 16:27 |
lucasagomes | fair | 16:28 |
*** dprince has joined #openstack-ironic | 16:28 | |
mat128 | so not sure Ironic should try to manipulate power state other than issuing the NMI | 16:28 |
lucasagomes | the target state on that spec is still "on" after nmi | 16:28 |
mat128 | yeah.. we're running the other mode (trust the power device rather than the db) downstream | 16:28 |
mat128 | I keep forgetting that mode :P | 16:28 |
lucasagomes | mat128, yeah, mind taking a look at the spec? I think you have more experience with NMI than I do | 16:28 |
lucasagomes | would be good to have ur opnion on the interface proposed by the author | 16:28 |
lucasagomes | https://review.openstack.org/#/c/186700 | 16:29 |
lucasagomes | when you have some time ofc | 16:29 |
mat128 | oh thanks :) was looking for it | 16:29 |
mat128 | have about 30m before internal sprint demo :) I will look at it | 16:29 |
lucasagomes | thanks a lot | 16:30 |
*** sabeen has joined #openstack-ironic | 16:32 | |
mat128 | lucasagomes: so far, I think SNMP will simply not support any of the soft controls | 16:32 |
mat128 | I wonder if INJECT_NMI really belongs to power driver though | 16:32 |
-openstackstatus- NOTICE: zuul was restarted for a software upgrade; events between 16:08 and 16:30 were missed, please recheck any changes uploaded during that time | 16:33 | |
*** sabeen1 has joined #openstack-ironic | 16:34 | |
lucasagomes | mat128, fair, if you can please add a comment | 16:35 |
lucasagomes | perhaps adding NMI to the management interface as proposed by sambetts before is the way to go | 16:35 |
*** sabeen has quit IRC | 16:36 | |
*** Goneri has joined #openstack-ironic | 16:39 | |
*** PollyZ has quit IRC | 16:41 | |
*** marios has joined #openstack-ironic | 16:44 | |
*** trown is now known as trown|lunch | 16:45 | |
*** PollyZ has joined #openstack-ironic | 16:45 | |
*** ijw has quit IRC | 16:49 | |
*** tloeb__ has quit IRC | 16:49 | |
lucasagomes | im going to call it a day | 16:51 |
lucasagomes | have a great evening all, enjoy the weekend | 16:51 |
mat128 | lucasagomes: you too =) | 16:51 |
lucasagomes | :-) see ya | 16:51 |
*** lucasagomes is now known as lucas-afk | 16:51 | |
*** manjrem has joined #openstack-ironic | 16:52 | |
*** daemontool_ has quit IRC | 16:52 | |
*** harlowja has joined #openstack-ironic | 16:53 | |
vdrok | good night lucas-afk | 16:55 |
openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: Test version do not merge https://review.openstack.org/331257 | 16:56 |
vdrok | sambetts: jroll iirc I've seen someone proposing that we push the portgroups api to the back of multitenancy patches chain, maybe you remember? is it still the case? | 16:56 |
sambetts | vdrok: I think we decided to not change things at this point and just focus on getting everything merged | 17:05 |
vdrok | sambetts: ok. i think i'll change the portgroups controller to be like you propose, with _lookup, as it fixes the only issue that I see now with it | 17:07 |
sambetts | vdrok: cool, I'm just reworking my patches to make the interface for doing it much nicer | 17:07 |
vdrok | yeah, this way will save a bunch of ifs in current code :) | 17:08 |
sambetts | vdrok: i was so close to pushing a patch then managed to break my git repo :/ | 17:08 |
vdrok | :( | 17:08 |
*** rbudden has quit IRC | 17:11 | |
*** e0ne has joined #openstack-ironic | 17:12 | |
openstackgerrit | Andre Aranha proposed openstack/python-oneviewclient: DO NOT MERGE Test version DO NOT MERGE https://review.openstack.org/331257 | 17:12 |
*** amotoki has quit IRC | 17:13 | |
thiagop | rajinir: noted. Sorry, was on a meeting | 17:18 |
thiagop | which one is yours? | 17:18 |
thiagop | rajinir: Sorry, I have to go. Talk to you on monday | 17:23 |
thiagop | Goin' out early today, family business. See ya folks! | 17:23 |
sambetts | cya thiagop | 17:23 |
*** thiagop has quit IRC | 17:23 | |
*** e0ne has quit IRC | 17:24 | |
*** PollyZ has quit IRC | 17:26 | |
*** piet has joined #openstack-ironic | 17:31 | |
*** ijw has joined #openstack-ironic | 17:32 | |
*** milan has joined #openstack-ironic | 17:34 | |
*** edand has quit IRC | 17:36 | |
*** sabeen1 has quit IRC | 17:39 | |
openstackgerrit | Sam Betts proposed openstack/ironic: Fix invalid chassis/nodes/chassis_uuid API url https://review.openstack.org/316149 | 17:39 |
openstackgerrit | Sam Betts proposed openstack/ironic: Fix all possible node subresources routing collisions https://review.openstack.org/315766 | 17:39 |
openstackgerrit | Sam Betts proposed openstack/ironic: Prevent URL collisions with sub-controllers: nodes/ports https://review.openstack.org/314514 | 17:39 |
*** sdake has joined #openstack-ironic | 17:40 | |
*** fragatina has joined #openstack-ironic | 17:43 | |
*** PollyZ has joined #openstack-ironic | 17:43 | |
*** fragatin_ has quit IRC | 17:43 | |
* sambetts -> home | 17:43 | |
NobodyCam | have a great weekend sambetts | 17:44 |
sambetts | NobodyCam: You too! | 17:44 |
sambetts | Night all o/ | 17:44 |
*** sambetts is now known as sambetts|afk | 17:44 | |
vdrok | night sambetts|afk and everyone! | 17:48 |
NobodyCam | noight vdrok | 17:48 |
NobodyCam | night even | 17:48 |
vdrok | good night NobodyCam :) | 17:49 |
*** mbound has quit IRC | 17:49 | |
*** rbudden has joined #openstack-ironic | 17:49 | |
*** joprovost has quit IRC | 17:50 | |
*** ccamacho has quit IRC | 17:52 | |
jlvillal | vdrok: Ciao | 17:54 |
jlvillal | NobodyCam: I thought you spelled like Pinky (as in Pinky and The Brain) :) | 17:54 |
jlvillal | Noight! | 17:54 |
*** trown|lunch is now known as trown | 17:55 | |
*** joprovost has joined #openstack-ironic | 17:55 | |
NobodyCam | lol .... What are going to do tonight?!? :) | 17:55 |
jlvillal | The same thing we do every night Pinky. | 17:55 |
NobodyCam | lol +++ | 17:56 |
jlvillal | Try to take over the world! | 17:56 |
*** manjrem has quit IRC | 17:57 | |
NobodyCam | :) | 17:57 |
*** fragatina has quit IRC | 18:00 | |
TheJulia | taking over world ++ | 18:00 |
*** fragatina has joined #openstack-ironic | 18:01 | |
* mgould -> home; good night, everyone! | 18:03 | |
*** mgould is now known as mgould|afk | 18:04 | |
*** harlowja has quit IRC | 18:04 | |
NobodyCam | night mgould|afk | 18:04 |
*** dprince has quit IRC | 18:06 | |
*** dprince has joined #openstack-ironic | 18:06 | |
*** ccamacho has joined #openstack-ironic | 18:08 | |
openstackgerrit | Merged openstack/ironic: Follow-up to Active Node Creation https://review.openstack.org/331107 | 18:10 |
*** rbudden_ has joined #openstack-ironic | 18:27 | |
*** fragatina has quit IRC | 18:27 | |
*** ChubYann has joined #openstack-ironic | 18:27 | |
*** rbudden has quit IRC | 18:28 | |
*** rbudden_ is now known as rbudden | 18:28 | |
*** Sukhdev has joined #openstack-ironic | 18:32 | |
*** joprovost has quit IRC | 18:34 | |
*** ijw has quit IRC | 18:34 | |
*** joprovost has joined #openstack-ironic | 18:37 | |
*** joprovost1 has joined #openstack-ironic | 18:42 | |
*** joprovost has quit IRC | 18:43 | |
*** joprovost1 is now known as joprovost | 18:43 | |
*** moshele has joined #openstack-ironic | 18:45 | |
openstackgerrit | Julia Kreger proposed openstack/ironic-specs: Boot from Volume - Reference Drivers https://review.openstack.org/294995 | 18:47 |
*** ccamacho has quit IRC | 18:50 | |
*** mbound has joined #openstack-ironic | 18:50 | |
*** ccamacho has joined #openstack-ironic | 18:52 | |
*** mbound has quit IRC | 18:54 | |
*** joprovost1 has joined #openstack-ironic | 18:55 | |
*** joprovost has quit IRC | 18:56 | |
*** joprovost1 is now known as joprovost | 18:56 | |
*** joprovost has quit IRC | 18:59 | |
*** joprovost has joined #openstack-ironic | 19:00 | |
*** moshele has quit IRC | 19:02 | |
*** fragatina has joined #openstack-ironic | 19:04 | |
*** e0ne has joined #openstack-ironic | 19:08 | |
*** shakamunyi has joined #openstack-ironic | 19:13 | |
*** fragatina has quit IRC | 19:14 | |
*** fragatina has joined #openstack-ironic | 19:15 | |
*** Nisha_away has quit IRC | 19:18 | |
*** jvgrant_ has joined #openstack-ironic | 19:20 | |
*** jvgrant_ has quit IRC | 19:25 | |
*** piet has quit IRC | 19:25 | |
*** piet has joined #openstack-ironic | 19:26 | |
*** harlowja has joined #openstack-ironic | 19:32 | |
*** MattMan has quit IRC | 19:32 | |
*** MattMan has joined #openstack-ironic | 19:32 | |
NobodyCam | jlvillal: thank you for the email | 19:33 |
*** ijw has joined #openstack-ironic | 19:35 | |
*** joprovost has quit IRC | 19:36 | |
*** ijw has quit IRC | 19:40 | |
*** rajinir has quit IRC | 19:44 | |
*** joprovost has joined #openstack-ironic | 19:49 | |
*** mbound has joined #openstack-ironic | 19:51 | |
*** xhku has quit IRC | 19:52 | |
*** e0ne has quit IRC | 19:53 | |
*** ijw has joined #openstack-ironic | 19:55 | |
*** mbound has quit IRC | 19:56 | |
openstackgerrit | Merged openstack/ironic: Centralize config options - [cisco_ucs] https://review.openstack.org/301382 | 19:57 |
*** raildo is now known as raildo-afk | 19:57 | |
*** rama_y has quit IRC | 19:59 | |
*** piet has quit IRC | 20:00 | |
*** xavierr is now known as xavierr-afk | 20:04 | |
*** rama_y has joined #openstack-ironic | 20:07 | |
openstackgerrit | Chris Krelle proposed openstack/ironic: Add support for the audit middleware https://review.openstack.org/272658 | 20:13 |
*** piet has joined #openstack-ironic | 20:13 | |
jlvillal | NobodyCam: Thanks and you're welcome :) | 20:20 |
NobodyCam | :) | 20:21 |
rama_y | Hello NobodyCam | 20:40 |
NobodyCam | hey there rama_y | 20:41 |
rama_y | NobodyCam: Hi. I was just wondering about your patch https://review.openstack.org/#/c/272658/ getting rebased | 20:41 |
rama_y | NobodyCam: a bunch of my config options patches are currently in review and they are getting merged one by one | 20:42 |
rama_y | NobodyCam: they are going to create merge conflicts for #272658 | 20:43 |
NobodyCam | yea I just rebased it | 20:43 |
rama_y | NobodyCam: I thought you might want to hold off on rebasing because there may be a few more merging soon | 20:43 |
NobodyCam | but I'll hold off prob until Monday then pick up again | 20:44 |
NobodyCam | THnak you for the heads up! | 20:45 |
NobodyCam | Thank you even :) | 20:45 |
rama_y | NobodyCam: sure, no problem. | 20:45 |
*** fragatina has quit IRC | 20:47 | |
*** fragatina has joined #openstack-ironic | 20:53 | |
*** wajdi_ has joined #openstack-ironic | 20:53 | |
*** trown is now known as trown|outtypewww | 20:54 | |
NobodyCam | TheJulia: should I look at rebasing some of the merge conflicted patches.. like https://review.openstack.org/254245 | 20:56 |
*** wajdi has quit IRC | 20:57 | |
*** fragatina has quit IRC | 20:57 | |
*** wajdi_ has quit IRC | 20:58 | |
*** livelace has quit IRC | 20:59 | |
*** joprovost has quit IRC | 20:59 | |
*** davidlenwell has quit IRC | 21:02 | |
*** daemontool has joined #openstack-ironic | 21:12 | |
*** davidlenwell has joined #openstack-ironic | 21:13 | |
*** ametts has quit IRC | 21:13 | |
*** piet has quit IRC | 21:15 | |
*** ijw has quit IRC | 21:23 | |
*** ijw has joined #openstack-ironic | 21:23 | |
*** lascii is now known as alaski | 21:28 | |
*** dmk0202 has quit IRC | 21:33 | |
*** dmk0202 has joined #openstack-ironic | 21:34 | |
*** dprince has quit IRC | 21:36 | |
*** PollyZ has quit IRC | 21:45 | |
*** mbound has joined #openstack-ironic | 21:49 | |
*** dmk0202 has quit IRC | 21:50 | |
*** daemontool has quit IRC | 21:53 | |
*** jcoufal has quit IRC | 21:59 | |
*** ijw has quit IRC | 22:07 | |
*** baoli has quit IRC | 22:13 | |
*** ijw has joined #openstack-ironic | 22:13 | |
*** fragatina has joined #openstack-ironic | 22:14 | |
*** fragatin_ has joined #openstack-ironic | 22:15 | |
*** fragatina has quit IRC | 22:19 | |
*** jcoufal has joined #openstack-ironic | 22:30 | |
*** ppiela has quit IRC | 22:35 | |
*** mbound has quit IRC | 22:36 | |
*** praneshp has quit IRC | 22:36 | |
*** sdake has quit IRC | 22:38 | |
*** sdake has joined #openstack-ironic | 22:41 | |
*** SiRiuS__ has quit IRC | 22:51 | |
*** SiRiuS__ has joined #openstack-ironic | 22:54 | |
*** jcoufal has quit IRC | 23:03 | |
*** rama_y has quit IRC | 23:07 | |
*** rama_y has joined #openstack-ironic | 23:08 | |
*** sacharya_ has quit IRC | 23:13 | |
openstackgerrit | Stephanie Miller proposed openstack/ironic-specs: Boot from Volume - Reference Drivers https://review.openstack.org/294995 | 23:17 |
*** fragatin_ has quit IRC | 23:21 | |
*** fragatina has joined #openstack-ironic | 23:25 | |
*** fragatina has quit IRC | 23:26 | |
*** fragatina has joined #openstack-ironic | 23:26 | |
*** mbound has joined #openstack-ironic | 23:36 | |
*** mbound has quit IRC | 23:42 | |
*** rbudden has quit IRC | 23:51 | |
*** praneshp has joined #openstack-ironic | 23:53 | |
*** wajdi has joined #openstack-ironic | 23:54 | |
*** fragatina has quit IRC | 23:55 | |
*** Sukhdev has quit IRC | 23:55 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!