| *** marcoemorais has quit IRC | 00:00 | |
| devananda | i objected on r52 | 00:00 |
|---|---|---|
| devananda | it was ignored | 00:00 |
| *** bradjones has quit IRC | 00:01 | |
| *** bradjones has joined #openstack-ironic | 00:03 | |
| rloo | devananda, I'm guessing you guys are discussing the 'inspect' api? I was wondering what 'provisioning' meant. | 00:03 |
| *** marcoemorais has joined #openstack-ironic | 00:03 | |
| NobodyCam | it changes between rev 46 and 47 up to 47 it was going to be the ManagementInterface on rev 47 it changes to IntrospectionInterface almost with out notice :( | 00:04 |
| rloo | devananda: does verifying, inspecting, (all those states) fall under 'provisioning'? | 00:04 |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic-specs: iRMC Virtual Media Deploy Driver for Ironic https://review.openstack.org/134865 | 00:05 |
| devananda | NobodyCam: i do recall a discussion about /that/ on IRC, but couldn't find it. I'm not thrilled but dont object either to the separate interface | 00:05 |
| devananda | the REST API semnantic bothers me | 00:05 |
| devananda | rloo: yoes | 00:06 |
| devananda | yes | 00:06 |
| devananda | rloo: right now, we expose only two types of states. power and probvision | 00:06 |
| * naohirot NobodyCam, I'll out of office today, I quickly updated the spec, thanks. | 00:06 | |
| rloo | devananda: ok. I was thinking that if we got 'away' from provisioning and used some new API, it might make it easier to transition to the new states. | 00:06 |
| devananda | rloo: such as? | 00:06 |
| devananda | rloo: while the name "provision" is a bit of legacy here, the model still fits, at least in my mind. everything in the state machine spec describes a provisioning state to me | 00:07 |
| rloo | devananda: i dunno. that's as far as I thought. we have put /v1/nodes/../states/provision. I thought if we added some other put .../states/hmm?, and had node.hmm_state and node.hmm_target_state, we could leave the target-provision-state stuff as legacy/old state names. | 00:09 |
| *** marcoemorais has quit IRC | 00:09 | |
| *** ChuckC has joined #openstack-ironic | 00:11 | |
| devananda | rloo: compatibility aside, what is meaningfully different about the new states? | 00:11 |
| devananda | aren't "cleaning" and "inspecting" also provision states? | 00:11 |
| devananda | ditto for "validating" and "zapping" | 00:11 |
| rloo | devananda: dunno. I've been learning baremetal terminology from you guys ! | 00:12 |
| devananda | rloo: fair. so, back to the spec's correctness | 00:12 |
| devananda | rloo: that I must GET /node/state to see the change, which I made by PUT /node/inspect -- this doesn't "0pass the smell test" for me | 00:13 |
| rloo | devananda: i agree with you then. there's no need for a PUT /node/inspect. | 00:15 |
| *** cinerama has quit IRC | 00:15 | |
| *** cinerama has joined #openstack-ironic | 00:17 | |
| openstackgerrit | Naohiro Tamura proposed openstack/ironic-specs: iRMC Management Driver for Ironic https://review.openstack.org/136020 | 00:17 |
| jroll | rloo: huh, we've been making up baremetal terminology :) | 00:25 |
| rloo | jroll: in that case... ;) | 00:26 |
| rloo | jroll, devananda: my other thought of the week is whether the node need's a provision_state AND target_provision_state if (I'm not totally sure this is true yet, haven't convinced myself) the fsm knows what the target_provision-state is for any given provision state. | 00:29 |
| jroll | I think it does | 00:30 |
| jroll | there may be multiple future states for a given state | 00:30 |
| jroll | see: MANAGED | 00:30 |
| jroll | may go to INSPECT, ZAP, or AVAILABLE | 00:30 |
| devananda | jroll: the target is only set for an -ING or -FAIL state | 00:30 |
| devananda | not a static one like MANAGE | 00:30 |
| *** chuckC_ has quit IRC | 00:31 | |
| jroll | oh, right | 00:31 |
| devananda | rloo: i'm going to have to think on that | 00:32 |
| rloo | anyway, just a thought I have. esp wrt looking at the fsm code. and wondering if we can get rid of the fsm._target_state if we can do what we need, without it. | 00:32 |
| * naohirot meeting | 00:32 | |
| jroll | yeah, it might be possible | 00:33 |
| devananda | rloo: i think we need to track that within the fsm, but i'm not sure we need to expose it in the API any longer | 00:33 |
| jroll | don't know how valuable removing that is, though | 00:33 |
| jroll | it is nice in the API for monitoring | 00:33 |
| jroll | maybe | 00:33 |
| * jroll checks if we're looking at that | 00:33 | |
| rloo | I was gungho on making sure we have unit tests for all the fsm transitions, but I'm petering out ;) Gotta focus... | 00:33 |
| jroll | we almost certainly are | 00:33 |
| devananda | jroll: on when they match? or what? | 00:33 |
| rloo | devananda: yeah, agreed, we need to track it. we could probably still expose it, but get the info from the fsm, not saving it with the node. | 00:33 |
| devananda | rloo: right | 00:34 |
| jroll | devananda: we have things that look at nodes to tell us what's going on | 00:34 |
| rloo | devananda: expose the target_provision_state I mean :-) | 00:34 |
| rloo | but first... get the fsm to work... | 00:34 |
| devananda | jroll: look at node's current state? | 00:34 |
| JayF | jroll: we are | 00:34 |
| jroll | turns out we don't look at target_provision_state, with the exception to say a node is 'deploying', but we could just as easily say provision_state in ('wait call-back', 'deploying') | 00:34 |
| devananda | jroll: or target state? | 00:34 |
| devananda | right | 00:35 |
| jroll | we might display it though, trying to remember if that's been useful | 00:35 |
| devananda | so one thing that's different in the proposed model than the current code | 00:35 |
| devananda | i mean, among the things that are different ... | 00:35 |
| devananda | when deploying, target state is deploydone | 00:36 |
| devananda | not activbe | 00:36 |
| rloo | jroll: we could easily be backwards compatible. just make node.target_provision_state a getter method. | 00:36 |
| devananda | the new model says target should be active | 00:36 |
| devananda | ditto for deleting/deleted -> deleting/available | 00:36 |
| jroll | hmmm | 00:36 |
| devananda | that was intentional. i hope everyone already thought about that difference :) | 00:36 |
| jroll | yeah, I don't see a problem with that | 00:37 |
| devananda | but if you're UX is keying off of target state, it might cause some issue | 00:37 |
| devananda | cool | 00:37 |
| devananda | rloo: that's going to change the fsm.py code | 00:37 |
| * jroll should probably change that now | 00:37 | |
| devananda | jroll: :) | 00:37 |
| devananda | rloo: the more unit tests you write now, the more i"ll have to work with, i mean replace, over the holidays | 00:38 |
| rloo | devananda: the difference between the existing and new states? yeah. i added a comment about that but haven't yet pushed up another revision. | 00:38 |
| devananda | i imagine i'll write the bulk of the new fsm stuff during the break | 00:38 |
| rloo | devananda: i thought i'd get it done today. You'll get something tomorrow, cuz I don't think I can do this beyond tomorrow. Need to do something different ;) | 00:40 |
| devananda | rloo: cheers | 00:40 |
| rloo | devananda: btw, i looked at the other stuff for kilo-1, and i don't think the code for the two bps are/will be ready this week. my opinion of course ;) | 00:41 |
| devananda | nope. i agree completely | 00:42 |
| devananda | going to bump them both tonight | 00:42 |
| devananda | thanks for looking | 00:42 |
| *** Masahiro has joined #openstack-ironic | 00:43 | |
| *** ChuckC has quit IRC | 00:44 | |
| *** r-daneel has quit IRC | 00:46 | |
| *** Masahiro has quit IRC | 00:47 | |
| *** derekh has joined #openstack-ironic | 00:52 | |
| *** Masahiro has joined #openstack-ironic | 00:54 | |
| * devananda wanders off for the evening | 01:04 | |
| *** jerryz has joined #openstack-ironic | 01:04 | |
| clif_h | http://logs.openstack.org/04/142204/1/gate//gate-python-ironicclient-python33/3c9e0af/console.html | 01:12 |
| *** ChuckC has joined #openstack-ironic | 01:13 | |
| clif_h | gate failed my review, but it looks like it was a setup issue and didn't even get to testing | 01:13 |
| *** chuckC_ has joined #openstack-ironic | 01:13 | |
| clif_h | https://review.openstack.org/#/c/142204/ | 01:13 |
| clif_h | has anyone seen this happen before? | 01:13 |
| *** killer_prince is now known as lazy_prince | 01:14 | |
| JayF | I think all py33 tests are broken right now but IMBW | 01:14 |
| JayF | asking ^ that exact question in #openstack-infra might get you more results | 01:14 |
| jroll | oh right | 01:15 |
| jroll | I think shrews mentioned this | 01:15 |
| *** derekh has quit IRC | 01:16 | |
| rloo | clif_h, jroll, JayF: yeah, I think py33 tests are broken for all clients | 01:18 |
| clif_h | oh nice | 01:18 |
| jroll | "nice" | 01:18 |
| rloo | should probably mention it in our etherpad. but to do that, i'd have to find out more about it | 01:19 |
| JayF | clif_h: so in this case, if you find the actual bug number, just keep it in your notes to recheck later when it's fixed | 01:19 |
| rloo | looks like bug is 1403557, no 1403510 | 01:20 |
| rloo | there's email about it in openstack-dev, look for [python-*client] py33 jobs seem to be failing | 01:21 |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/142635 | 01:22 |
| *** chenglch has joined #openstack-ironic | 01:23 | |
| rloo | clif_h, jroll, JayF: doesn't help you, but I added a note in our IronicWhiteBoard about it with the bug number, and added that it affects our client too. | 01:25 |
| lazy_prince | Hi.. is there a way to run two nova-computes on a single host..? say one for KVM and another for Ironic.. | 01:26 |
| JayF | I'm not sure if it's possible, but I'm certain it's not desirable. | 01:26 |
| jroll | lazy_prince: I have no idea, seems weird, you might ask nova folks | 01:26 |
| lazy_prince | thanks.. | 01:26 |
| JayF | I'd rather run the second nova compute in a VM running on that one nova-compute host | 01:26 |
| lazy_prince | Actually, This is already a VM where I want to configure two nova-computes.. | 01:28 |
| lazy_prince | nova folks say its not possible.. so I will go ahead and deploy another VM with ironic service.. | 01:30 |
| *** bradjones has quit IRC | 01:32 | |
| Haomeng|2 | lazy_prince: one nova.conf can support one single hypervisor configuration I think, if possible, we have to different nova.conf and run nova compute with differeent nova.conf in same machine, but maybe more configuration are required for different nova compute process | 01:39 |
| Haomeng|2 | lazy_prince: but it will be confus, so no need to do such configuration I think, just run with more vms | 01:40 |
| openstackgerrit | Josh Gachnang proposed openstack/ironic-specs: Implement Cleaning States https://review.openstack.org/102685 | 01:41 |
| *** kes has joined #openstack-ironic | 01:45 | |
| *** Marga_ has quit IRC | 01:46 | |
| *** Marga_ has joined #openstack-ironic | 01:46 | |
| *** Marga_ has quit IRC | 01:51 | |
| *** mitz has joined #openstack-ironic | 01:54 | |
| *** mitz_ has joined #openstack-ironic | 02:04 | |
| *** david-lyle is now known as david-lyle_afk | 02:10 | |
| lintan | Hi rloo | 02:10 |
| rloo | hi lintan | 02:10 |
| lintan | do we get some agreement on amt_boot_device? | 02:11 |
| rloo | lintan: sorry, I don't think so. Take a look in irc; deva had two issues. I think he agreed not to use driver_info, and he was concerned about your use of wsman library/CLI. | 02:12 |
| lintan | thanks | 02:13 |
| *** lucas-dinner has joined #openstack-ironic | 02:14 | |
| rloo | lintan: better to look back in irc because I may have missed something (was working on something else). Sorry. I don't think your code will make it for kilo-1 this week, but once people agree, it shouldn't take long to get your patch in. (I'm really trying to get the finite-state-machine stuff done cuz that will hold up a lot of stuff it is isn't done soon.) | 02:14 |
| lazy_prince | Haomeng|2: I decided to go with an extra VM.. | 02:15 |
| Haomeng|2 | lazy_prince: yes, that is the way:) | 02:19 |
| lazy_prince | by the way, is there any heat template available to provision ironic in overcloud..? | 02:20 |
| *** ryanpetrello has joined #openstack-ironic | 02:20 | |
| lintan | OK, I understand, btw do we have a link to see the history like meeting? | 02:20 |
| jiangfei | lazy_prince: hi, these two nova-compute can work simultaneously in the same nova Service? | 02:21 |
| lazy_prince | I wanted them to work.. but seems like its not possible.. | 02:22 |
| lazy_prince | nova folks denied the possibilities.. | 02:22 |
| Haomeng|2 | lazy_prince: heat template works with ironic, because our iornic provision is start from nova boot | 02:22 |
| jiangfei | lazy_prince: thank you | 02:23 |
| Haomeng|2 | lazy_prince: so make sure the heat template include the correct ironic flavor, then nova schedule will check ironic hypervisor out | 02:23 |
| lazy_prince | umm... what I want it heat to provision ironic in overcloud.. so that I could manage BMs in overcloud.. | 02:23 |
| Haomeng|2 | lazy_prince: you can take tripleo as reference | 02:23 |
| Haomeng|2 | lazy_prince: there are some heat template used to deploy openstack itself | 02:24 |
| lazy_prince | aha.. thank you... i will have a look... | 02:24 |
| lintan | hi devananda: | 02:24 |
| Haomeng|2 | lazy_prince: https://git.openstack.org/cgit/openstack/tripleo-heat-templates | 02:25 |
| Haomeng|2 | lintan: I think deva is in dream now:) | 02:25 |
| *** ryanpetrello has quit IRC | 02:27 | |
| lintan | Haomeng: OK, which time zone he is in? | 02:27 |
| jiangfei | lazy_prince: nova have any good suggestions? nova you have any good suggestions, we need to deploy two nova service? like seed/undercloud in tripleo | 02:29 |
| lazy_prince | Well.. usually, seed and undercloud are two separate VMs.. | 02:31 |
| lazy_prince | I have not seen them on a single VM or machine | 02:31 |
| lintan | Haomeng|2: can we check the chat history for ironic? I miss some part | 02:31 |
| Haomeng|2 | lintan: tip is that you run irc client and keep to open to receive all the chat, and no irc log exclude the meetings | 02:32 |
| Haomeng|2 | lintan: and if you want, I can send to you my history | 02:33 |
| lazy_prince | or you could look at http://eavesdrop.openstack.org/irclogs/ | 02:34 |
| lintan | Thansk lazy_prince Haomeng|2 | 02:35 |
| Haomeng|2 | lazy_prince: cool, I dont know such log:) | 02:36 |
| lazy_prince | wc.. | 02:37 |
| *** lucas-dinner has quit IRC | 02:37 | |
| lazy_prince | ofcourse, they would not log any PMs.. | 02:37 |
| Haomeng|2 | lazy_prince: yes, can not find nova, maybe nova irc is hurge:( | 02:38 |
| Haomeng|2 | lazy_prince: thanks for you info | 02:38 |
| lazy_prince | thats strange... | 02:39 |
| Haomeng|2 | lazy_prince: :) | 02:42 |
| *** igordcard has joined #openstack-ironic | 02:44 | |
| *** cuihaozhi has quit IRC | 02:50 | |
| *** cuihaozhi has joined #openstack-ironic | 02:51 | |
| *** igordcard has quit IRC | 02:53 | |
| *** ramineni has joined #openstack-ironic | 02:56 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Issue with FSM's _target_state https://review.openstack.org/142303 | 03:06 |
| *** naohirot has quit IRC | 03:08 | |
| *** penick has quit IRC | 03:22 | |
| *** rloo has quit IRC | 03:23 | |
| openstackgerrit | jiangfei proposed openstack/ironic: ironic-api vaild the input driver https://review.openstack.org/141703 | 03:31 |
| jiangfei | Haomeng: hi, is there ? | 03:44 |
| *** teju has joined #openstack-ironic | 03:46 | |
| Haomeng|2 | jiangfei: yes | 03:47 |
| Haomeng|2 | jiangfei: hi | 03:47 |
| jiangfei | Haomeng: hi, take a look at https://review.openstack.org/141703. | 03:48 |
| Haomeng|2 | jiangfei: ok | 03:48 |
| jiangfei | Haomeng: just add the warning log when load failed, and change the error message for ironic node-create | 03:49 |
| Haomeng|2 | jiangfei: thanks for your patch, will review | 03:49 |
| Haomeng|2 | jiangfei: ok, great work | 03:49 |
| Haomeng|2 | jiangfei: looks fine this time:) | 03:49 |
| jiangfei | Haomeng: thank you :) | 03:50 |
| Haomeng|2 | jiangfei: yw:) | 03:50 |
| *** lazy_prince is now known as killer_prince | 03:51 | |
| jiangfei | Haomeng: ^^ | 03:51 |
| Haomeng|2 | jiangfei: :) | 03:56 |
| openstackgerrit | jiangfei proposed openstack/python-ironicclient: ironicclient handle faultstring when using SessionClient https://review.openstack.org/142021 | 04:00 |
| *** Marga_ has joined #openstack-ironic | 04:13 | |
| *** harlowja is now known as harlowja_away | 04:19 | |
| *** killer_prince is now known as lazy_prince | 04:20 | |
| *** Masahiro has quit IRC | 04:24 | |
| *** rameshg87 has joined #openstack-ironic | 04:29 | |
| *** Masahiro has joined #openstack-ironic | 04:31 | |
| *** Marga_ has quit IRC | 04:35 | |
| *** Marga_ has joined #openstack-ironic | 04:35 | |
| *** Marga_ has quit IRC | 05:05 | |
| *** nosnos has joined #openstack-ironic | 05:10 | |
| *** andreykurilin_ has joined #openstack-ironic | 05:25 | |
| *** Marga_ has joined #openstack-ironic | 05:31 | |
| *** jerryz has quit IRC | 05:36 | |
| *** lazy_prince has quit IRC | 05:48 | |
| *** jerryz has joined #openstack-ironic | 05:48 | |
| *** killer_prince has joined #openstack-ironic | 05:49 | |
| *** killer_prince is now known as lazy_prince | 05:49 | |
| *** jerryz_ has joined #openstack-ironic | 05:55 | |
| *** Masahiro has quit IRC | 05:55 | |
| *** jerryz has quit IRC | 05:58 | |
| *** k4n0 has joined #openstack-ironic | 06:10 | |
| *** andreykurilin_ has quit IRC | 06:11 | |
| openstackgerrit | jiangfei proposed openstack/python-ironicclient: ironicclient handle faultstring when using SessionClient https://review.openstack.org/142021 | 06:19 |
| *** achanda has joined #openstack-ironic | 06:22 | |
| *** Masahiro has joined #openstack-ironic | 06:29 | |
| *** MattMan has quit IRC | 06:31 | |
| *** MattMan has joined #openstack-ironic | 06:32 | |
| openstackgerrit | Tan Lin proposed openstack/ironic-specs: Update amt-pxe-driver spec https://review.openstack.org/141269 | 06:40 |
| *** pcrews has joined #openstack-ironic | 06:43 | |
| *** pcrews has quit IRC | 06:50 | |
| *** yuanying_ has joined #openstack-ironic | 07:01 | |
| *** yuanying has quit IRC | 07:03 | |
| *** kevinbenton has quit IRC | 07:06 | |
| *** kevinbenton has joined #openstack-ironic | 07:08 | |
| *** subscope has joined #openstack-ironic | 07:11 | |
| *** PaulCzar has quit IRC | 07:16 | |
| *** jcoufal has joined #openstack-ironic | 07:18 | |
| *** achanda has quit IRC | 07:31 | |
| *** alexpilotti has joined #openstack-ironic | 07:35 | |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic-specs: Enable boot images sharing for iLO driver(s) https://review.openstack.org/137291 | 07:43 |
| *** Marga_ has quit IRC | 07:46 | |
| *** dlpartain has joined #openstack-ironic | 07:52 | |
| *** pcaruana has joined #openstack-ironic | 07:56 | |
| *** yuanying has joined #openstack-ironic | 07:59 | |
| *** ndipanov has joined #openstack-ironic | 08:00 | |
| *** ujuc has joined #openstack-ironic | 08:00 | |
| *** yuanying_ has quit IRC | 08:02 | |
| *** JohnMarkByers has joined #openstack-ironic | 08:04 | |
| JohnMarkByers | Is Barack Obama loyal to the United States of America? | 08:04 |
| JohnMarkByers | I think it bears investigation. | 08:04 |
| *** dlpartain has left #openstack-ironic | 08:06 | |
| JohnMarkByers | Because Freemasons, you see, promote communist revolution wherever they seep into power. Obama, as you may know, has past communist ties and was mentored by communist Frank Marshall Davis, a close friend of his father. He is also a 32nd Degree Freemason of the Prince Hall Lodge and has, on one occasion, posed with a Masonic ring for a shot of his fist that appeared on a magazine cover. | 08:07 |
| JohnMarkByers | Virtually all of the communist revolutionaries in Russia and Germany have been Freemasons. Nearly every revolution engineered in the Americas has been instigated and led by Freemasons. Our nation's founding fathers, ironically, were Masons themselves, but were revolting against the tyranny of the British Crown, which forms the nucleas of Freemasonry. Thus, the American Revolution was the good Masons revolting against | 08:09 |
| JohnMarkByers | Obama is a communist and a foot-soldier of Prince Hall Freemasonry, which is involved, like every other part of the Masonic system, in promoting communist revolution across the world. | 08:10 |
| JohnMarkByers | This is why Obama is supplying U.S. military and financial aid to the Muslim Brotherhood in various countries, as well as the various mercenary groups, including "al Qaeda." The Muslim Brotherhood was formed by a Freemason. al Qaeda grew out of the anti-Soviet rebel groups in Afghanistan, whch were armed, trained and funded by the CIA under the orders of President Jimmy Carter and his Polish communist National Securit | 08:13 |
| JohnMarkByers | Carter was a well-known communist, and Brzezinski emigrated to the U.S. from a Soviet satellite. He was Obama's mentor at Columbia University, where he was, naturally enough, head of the Russian Department. | 08:14 |
| JohnMarkByers | Brzezinski co-founded the Trilateral Commission with David Rockefeller, and was a member of the Council on Foreign Relations, which was set up by the Rockefeller and Morgan families, along with other Wall Street luminaries. These men were agents of the American branch of the Round Table, and these groups were fronts for this London-based group. | 08:16 |
| *** achanda has joined #openstack-ironic | 08:19 | |
| JohnMarkByers | The Round Table itself was merely the 'outer core' of a secret society called the Society of the Elect, founded in the late nineteenth century by Cecil Rhodes, an English Freemason and "revolutionary" (communist) who wanted to establish a one-world government. He was a diamond tycoon in South Africa, operating largely as an agent for the Rothschild family. This makes enough sense, as Rhodes facilitated the expansion o | 08:19 |
| JohnMarkByers | Rhodes had also set up a Jesuit mission, giving a generous amount of money to a Jesuit priest. | 08:19 |
| JohnMarkByers | The Jesuits were the true forefathers of modern communism. Their Paraguay Reductions in the seventeenth century formed the basis for the absolutist, collectivist dictatorship of the communist state. | 08:20 |
| JohnMarkByers | Karl Marx was, by some reports, tutored by Jesuits for no less than *thirty years* at the British Museum. Nevertheless, his "manifesto" was fed to him by a mysterious, possibly Masonic group known as "the League of Just Men." Many communists identify with the Bavarian Illuminati, a rather infamous Masonic group, and they celebrate May Day on May 1 because this is the date on which the Illuminati was formed at Ingolsta | 08:22 |
| JohnMarkByers | The man who founded it, Adam Weishaupt, was the son of a professor at this Jesuit college, and he was raised by a godfather who was also a professor there. He was educated by Jesuits from age seven on. He was called a "Jesuitical character" and "a Jesuit in disguise" by some of his own followers. | 08:23 |
| *** rameshg87 has quit IRC | 08:24 | |
| JohnMarkByers | He taught canon law at Ingolstadt. In the past, this had *exclusively* been taught by Jesuits. The Illuminati formed in 1776, only three years after the Jesuits had been abolished and suppressed by a pope who would soon be poisoned to death and who would blame the Jesuits until taking his last breath. | 08:24 |
| JohnMarkByers | The Illuminati orchestrated the French Revolution and other revolutions, operating as a front for the Jesuits. The same royal family that had cooperated in suppressing the Jesuits was systematically exterminated, along with much of the Dominican Order. The Dominicans were a longtime rival of the Jesuits and had taken control of the Holy Office of the Inquisition after they were abolished. This was a special post to wh | 08:26 |
| JohnMarkByers | Virtually all of the instigators of the revolution were Masons, members of the Illuminati, and graduates of Jesuit colleges. It is unlikely all three affiliations would appear so consistently by accident. | 08:27 |
| JohnMarkByers | In the aftermath, Emmanuel Joseph Sieyes, who had wrote "The Third Estate," which was touted as the 'manifesto' of the revolution, organized a coup that led to the ascension of Emperor Napoleon Bonaparte I, a Freemason like his five brothers. Bonaparte was also a devout Catholic but was firmly under the control of Sieyes, who was a rogue member of the clergy who had been tutored by Jesuits and ordained in 1773, the ye | 08:29 |
| JohnMarkByers | The Jesuits had taken control of France and were now using its military force to destroy strategic Catholic targets and a few Protestant nations as well. The island of Malta was raided and the lavish fortresses of the Sovereign Military Order of Malta, one of the Vatican's chief intelligence arms, were seized. | 08:30 |
| JohnMarkByers | The Vatican itself was eventually captured, and the Pope was imprisoned until he agreed to reestablish the Jesuits, which he did. Bonaparte, however, would eventually be cut loose and defeated. He lived out the rest of his live in exile, bitter about the betrayal by his Jesuit masters. | 08:32 |
| JohnMarkByers | The Catholic Church formally reinstated the Jesuits in 1814. In 1832, the Rothschild family became their official financial agents. This is confirmed by the Jewish Encyclopedia, which describes them as "guardians of the papal treasure." They are also the force behind the Morgans, Rockefellers, Warburgs, Schiffs, and virtually every major Wall Street banking house. | 08:33 |
| JohnMarkByers | They are behind the communist revolution in Russia. They are behind the national socialist takeover of Germany and the holocaust that resulted. | 08:33 |
| JohnMarkByers | They are behind all of the wars in modern history. These families are JESUIT bloodlines | 08:34 |
| *** JohnMarkByers has quit IRC | 08:35 | |
| kes | JohnMarkByers was catched by CHENG GUAN. | 08:36 |
| *** ifarkas has joined #openstack-ironic | 08:46 | |
| *** alexpilotti has quit IRC | 08:56 | |
| *** ujuc has quit IRC | 09:00 | |
| *** romcheg has joined #openstack-ironic | 09:02 | |
| *** lazy_prince has quit IRC | 09:08 | |
| *** athomas has joined #openstack-ironic | 09:10 | |
| *** lazy_prince has joined #openstack-ironic | 09:11 | |
| *** lazy_prince has quit IRC | 09:11 | |
| *** lazy_prince has joined #openstack-ironic | 09:11 | |
| *** derekh has joined #openstack-ironic | 09:14 | |
| *** viktors|afk is now known as viktors | 09:14 | |
| *** subscope has quit IRC | 09:20 | |
| *** vdrok has quit IRC | 09:27 | |
| openstackgerrit | Gopi Krishna S proposed openstack/ironic-specs: Add pxe_ucs driver to support Cisco UCS B/C/M-series servers https://review.openstack.org/139517 | 09:33 |
| *** igordcard has joined #openstack-ironic | 09:39 | |
| openstackgerrit | Tan Lin proposed openstack/ironic: Add AMT-PXE-Driver to deploy cloud on PC https://review.openstack.org/135184 | 09:41 |
| openstackgerrit | Merged openstack/ironic: Improve Agent deploy driver validation https://review.openstack.org/141766 | 09:47 |
| *** rameshg87 has joined #openstack-ironic | 09:53 | |
| *** dtantsur|afk is now known as dtantsur | 10:12 | |
| *** erwan_taf has joined #openstack-ironic | 10:14 | |
| openstackgerrit | OpenStack Proposal Bot proposed openstack/ironic: Updated from global requirements https://review.openstack.org/142635 | 10:14 |
| *** achanda has quit IRC | 10:26 | |
| *** achanda has joined #openstack-ironic | 10:27 | |
| *** chenglch has quit IRC | 10:30 | |
| *** achanda has quit IRC | 10:31 | |
| *** achanda has joined #openstack-ironic | 10:32 | |
| *** alexpilotti has joined #openstack-ironic | 10:35 | |
| rameshg87 | dtantsur, hi | 10:40 |
| dtantsur | rameshg87, o/ | 10:40 |
| rameshg87 | dtantsur, one question on https://review.openstack.org/#/c/137567/4/specs/kilo/ilo-virtualmedia-drivers-dhcpless-deploy.rst | 10:40 |
| rameshg87 | dtantsur, regarding the comment on format of "<MAC1>,<IP1>,<NETMASK1>,<GATEWAY1>;<MAC2>,<IP2>,<NETMASK2>,<GATEWAY2>;..." | 10:41 |
| rameshg87 | dtantsur, what do you mean by something standard here ? | 10:41 |
| rameshg87 | dtantsur, i wanted to send the network info in one argument. so had to use some delimiter to delimit the individual elements | 10:42 |
| rameshg87 | dtantsur, are you having something specific in mind ? | 10:42 |
| dtantsur | rameshg87, don't know, maybe JSON? | 10:42 |
| dtantsur | just don't want to increase number of home-grown formats | 10:43 |
| rameshg87 | dtantsur, the value passed is parsed by a dib ramdisk | 10:43 |
| *** athomas has quit IRC | 10:43 | |
| *** achanda has quit IRC | 10:45 | |
| *** achanda has joined #openstack-ironic | 10:46 | |
| dtantsur | rameshg87, anyway, if you at least make it like MAC=$MAC1,IP=$IP1 etc you will make both parsing and future expansion somewhat easier | 10:47 |
| rameshg87 | dtantsur, the pxe dib ramdisk is a shell script, so wanted to make it simpler too | 10:47 |
| dtantsur | primitive data formats not necessary make life simpler + we want to go away from bash ramdisk with time | 10:48 |
| rameshg87 | dtantsur, so do you mean multiple instances MAC=$MAC1,IP=$IP1 MAC=$MAC2,IP=$IP2 like that ? | 10:49 |
| rameshg87 | dtantsur, because i wanted to send network information of multiple nics | 10:50 |
| *** achanda has quit IRC | 10:50 | |
| dtantsur | rameshg87, that's why I suggested a structured format in the first place :) well, I understand that parsing JSON in bash is not funny. maybe there's something existing for it? | 10:51 |
| *** athomas has joined #openstack-ironic | 10:51 | |
| dtantsur | rameshg87, well yeah https://github.com/dominictarr/JSON.sh | 10:51 |
| dtantsur | even APACHE-licensed :) | 10:52 |
| rameshg87 | dtantsur, so just send the json data to the ramdisk and let it decode :) | 10:52 |
| rameshg87 | dtantsur, ipa can do it easily, but dib ramdisk will find it not that easy | 10:53 |
| rameshg87 | dtantsur, okay, let me check, will update the spec | 10:53 |
| rameshg87 | dtantsur, thanks | 10:53 |
| dtantsur | rameshg87, looking into the future, I think base64-encoded JSON is a bette way to go... actually even parsing your self-inventing format is bash doesn't look that easy... | 10:53 |
| dtantsur | rameshg87, you're welcome | 10:54 |
| rameshg87 | dtantsur, yeah but the format was close enough to what was already done in dib ramdisk | 10:54 |
| rameshg87 | dtantsur, sort of close enough | 10:54 |
| rameshg87 | dtantsur, https://github.com/openstack/diskimage-builder/blob/master/elements/ramdisk-base/init.d/20-init-variables#L17-L27 | 10:54 |
| rameshg87 | dtantsur, pxelinux adds it in ip:boot_server_ip:gateway:netmask | 10:55 |
| rameshg87 | dtantsur, since i had to pass mac also, and mac could contain ':', i just chose to delimit with , | 10:55 |
| rameshg87 | dtantsur, but parsing using simple delimiters is actually easy :) | 10:56 |
| *** subscope has joined #openstack-ironic | 10:56 | |
| dtantsur | well, we can't do anything about pxelinux :) | 10:57 |
| dtantsur | you can ask the others too, it may be just my personal opinion | 10:57 |
| rameshg87 | dtantsur, yeah i agree | 10:57 |
| rameshg87 | dtantsur, let me just look around ... will ping you back again | 10:58 |
| dtantsur | ack | 10:58 |
| *** ramineni has quit IRC | 10:58 | |
| *** rakesh_hs has joined #openstack-ironic | 10:59 | |
| *** viktors has quit IRC | 11:01 | |
| *** yuriyz has quit IRC | 11:04 | |
| *** rameshg87 has quit IRC | 11:11 | |
| *** lucasagomes has joined #openstack-ironic | 11:15 | |
| *** Haomeng|2 has quit IRC | 11:22 | |
| lintan | Hi lucasagomes | 11:24 |
| *** ndipanov has quit IRC | 11:25 | |
| *** nosnos has quit IRC | 11:26 | |
| *** nosnos has joined #openstack-ironic | 11:26 | |
| *** jiangfei has quit IRC | 11:27 | |
| *** jiangfei has joined #openstack-ironic | 11:27 | |
| lucasagomes | lintan, morning sorry I haven't finished the review. Did you guys decide anything new re which attribute to use yesterday? | 11:28 |
| *** nosnos has quit IRC | 11:30 | |
| lintan | lucasagomes: hmmmm, I miss the discussion. From the log, it seems that extra is a better choice, but not quite sure, I should double check with devananda | 11:32 |
| lucasagomes | ack | 11:32 |
| *** Masahiro has quit IRC | 11:34 | |
| *** nosnos has joined #openstack-ironic | 11:39 | |
| lucasagomes | dtantsur, https://review.openstack.org/#/c/137762/ when u get some time | 11:43 |
| dtantsur | ack | 11:43 |
| lucasagomes | dtantsur, this is related as well https://review.openstack.org/#/c/142492/ (the CLI part) | 11:44 |
| dtantsur | lucasagomes, +2, bit I'd prefer to wait for devananda, as he had concerns about naming | 11:47 |
| lucasagomes | dtantsur, sure | 11:49 |
| *** athomas has quit IRC | 11:53 | |
| *** athomas has joined #openstack-ironic | 11:59 | |
| *** slagle_ is now known as slagle | 12:07 | |
| *** erwan_taf has quit IRC | 12:14 | |
| *** alexpilotti has quit IRC | 12:26 | |
| *** yuriyz has joined #openstack-ironic | 12:30 | |
| *** viktors1 has joined #openstack-ironic | 12:32 | |
| *** yuriyz has quit IRC | 12:34 | |
| *** Masahiro has joined #openstack-ironic | 12:34 | |
| *** afaranha has left #openstack-ironic | 12:35 | |
| *** afaranha has quit IRC | 12:35 | |
| *** teju has quit IRC | 12:36 | |
| *** Masahiro has quit IRC | 12:39 | |
| *** lazy_prince is now known as killer_prince | 12:47 | |
| *** subscope has quit IRC | 12:53 | |
| TheJulia | good morning | 13:00 |
| *** yuriyz has joined #openstack-ironic | 13:03 | |
| *** nosnos has quit IRC | 13:05 | |
| *** nosnos has joined #openstack-ironic | 13:06 | |
| *** athomas has quit IRC | 13:08 | |
| *** nosnos has quit IRC | 13:10 | |
| *** dprince has joined #openstack-ironic | 13:14 | |
| *** athomas has joined #openstack-ironic | 13:15 | |
| *** stendulker has joined #openstack-ironic | 13:18 | |
| dtantsur | TheJulia, morning | 13:26 |
| *** Masahiro has joined #openstack-ironic | 13:35 | |
| openstackgerrit | Matthew Gilliard proposed openstack/ironic: Check glance for new kernel and ramdisk on rebuild https://review.openstack.org/142782 | 13:37 |
| *** ryanpetrello has joined #openstack-ironic | 13:39 | |
| *** jcoufal_ has joined #openstack-ironic | 13:40 | |
| *** Masahiro has quit IRC | 13:40 | |
| *** jcoufal has quit IRC | 13:43 | |
| *** rloo has joined #openstack-ironic | 13:43 | |
| openstackgerrit | Merged openstack/ironic: Updated from global requirements https://review.openstack.org/142635 | 13:47 |
| *** rameshg87 has joined #openstack-ironic | 13:49 | |
| *** derekh has quit IRC | 13:52 | |
| *** derekh has joined #openstack-ironic | 13:57 | |
| lucasagomes | JayF, jroll https://bugs.launchpad.net/ironic/+bug/1403887 have you guys seem it with IPA? since you guys also use parted for partitioning | 13:58 |
| lucasagomes | I thought parted would automatically tell the kernel to re-read the partition table, but apparently not | 13:59 |
| *** rakesh_hs has quit IRC | 13:59 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic-specs: UEFI Secure Boot support for iLO drivers https://review.openstack.org/135228 | 14:12 |
| ekarlso- | does ironci require some nova stuff to be usable ? | 14:15 |
| lucasagomes | ekarlso-, it can be used in a standalone way but it's not straight forward nowadays | 14:16 |
| lucasagomes | we are doing some work on that | 14:16 |
| lucasagomes | TheJulia, morning | 14:16 |
| stendulker | dtantsur: lucasgomes: jroll: Hi | 14:17 |
| dtantsur | stendulker, hi | 14:17 |
| openstackgerrit | Merged openstack/python-ironicclient: Fix to properly issue an Unauthorized exception https://review.openstack.org/142204 | 14:18 |
| stendulker | dtantsur: lucasgomes: jroll: Can you please review these specs related to UEFI secure boot Ironic Management Interfaces to support UEFI Secure Boot https://review.openstack.org/#/c/135845 | 14:18 |
| stendulker | dtantsur: lucasgomes: jroll: UEFI Secure Boot support for iLO drivers https://review.openstack.org/#/c/135228/ | 14:18 |
| dtantsur | I'll try to find some time, but can't promise anything, sorry | 14:18 |
| stendulker | dtantsur: Sure. Thank you. | 14:19 |
| *** afaranha has joined #openstack-ironic | 14:20 | |
| *** Marga_ has joined #openstack-ironic | 14:25 | |
| *** k4n0 has quit IRC | 14:29 | |
| *** afaranha has quit IRC | 14:31 | |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic-specs: Enable boot images sharing for iLO driver(s) https://review.openstack.org/137291 | 14:31 |
| rameshg87 | ifarkas, i guess you can only set it as WIP - https://review.openstack.org/#/c/135899/ | 14:33 |
| rameshg87 | ifarkas, because the review was started by you | 14:33 |
| rameshg87 | ifarkas, may be you can remove your -1 and set it to WIP | 14:33 |
| ifarkas | rameshg87, ahh, OK | 14:33 |
| ifarkas | rameshg87, I will do that, thanks! | 14:33 |
| *** Marga_ has quit IRC | 14:35 | |
| *** Marga_ has joined #openstack-ironic | 14:36 | |
| *** Marga_ has quit IRC | 14:41 | |
| *** lintan_ has joined #openstack-ironic | 14:51 | |
| *** erwan_taf has joined #openstack-ironic | 14:52 | |
| *** lucasagomes is now known as lucas-hungry | 14:52 | |
| NobodyCam | Good Morning Ironic | 14:55 |
| dtantsur | NobodyCam, morning | 14:59 |
| lintan_ | Morning NobodyCam | 15:00 |
| NobodyCam | morning dtantsur & lintan_ | 15:01 |
| NobodyCam | :) | 15:01 |
| openstackgerrit | Yuriy Zveryanskyy proposed openstack/ironic-specs: Add a new driver for Fuel Agent https://review.openstack.org/138115 | 15:02 |
| NobodyCam | yuriyz: Thank you :) | 15:03 |
| *** viktors1 has quit IRC | 15:04 | |
| *** david-lyle_afk has quit IRC | 15:07 | |
| openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Enable functional testing with local ramdisk source https://review.openstack.org/142811 | 15:08 |
| *** bradjones has joined #openstack-ironic | 15:16 | |
| jroll | morning all :) | 15:18 |
| NobodyCam | good mornig jrist | 15:18 |
| jroll | lucas-hungry: never seen that, that I'm aware of | 15:18 |
| NobodyCam | gah | 15:18 |
| jroll | D: | 15:18 |
| NobodyCam | tab fail | 15:18 |
| NobodyCam | morning jroll | 15:18 |
| jroll | hehe :) | 15:18 |
| jrist | :) | 15:23 |
| jrist | what, I dont' get a good morning? | 15:23 |
| NobodyCam | oh so sorry jrist was a total tab fail on my part | 15:23 |
| jrist | NobodyCam: :) | 15:23 |
| NobodyCam | but Good Morning :) | 15:23 |
| jrist | oh yay | 15:23 |
| jrist | good morning to you too NobodyCam | 15:23 |
| jroll | heya jrist :) | 15:24 |
| *** Masahiro has joined #openstack-ironic | 15:24 | |
| jrist | hey jroll | 15:24 |
| *** chuckC_ has quit IRC | 15:25 | |
| *** andreykurilin has joined #openstack-ironic | 15:25 | |
| dtantsur | jroll, morning | 15:25 |
| jrist | dtantsur: have a nice birthday? | 15:25 |
| jroll | lucas-hungry: thanks for the network plugin review thing, btw, we should talk more about the upstream bits in neutron | 15:26 |
| jroll | hey dtantsur :) | 15:26 |
| dtantsur | jrist, it was pretty awesome, thanks! | 15:27 |
| jrist | good! | 15:27 |
| *** viktors1 has joined #openstack-ironic | 15:28 | |
| *** Masahiro has quit IRC | 15:28 | |
| NobodyCam | brb | 15:37 |
| *** lucas-hungry is now known as lucasagomes | 15:41 | |
| lucasagomes | jroll, sure :) | 15:41 |
| lucasagomes | jroll, morning | 15:41 |
| lucasagomes | jroll, btw, you got a minute to talk about the configdrive? | 15:41 |
| *** r-daneel has joined #openstack-ironic | 15:44 | |
| NobodyCam | morning lucasagomes | 15:45 |
| lucasagomes | NobodyCam, yo morning | 15:45 |
| NobodyCam | :) | 15:45 |
| lucasagomes | NobodyCam, btw I asnwer the comments on root-devic-hint | 15:45 |
| NobodyCam | I shall take a look :) | 15:45 |
| lucasagomes | NobodyCam, we decided to keep size | 15:46 |
| lucasagomes | cool no problemo | 15:46 |
| NobodyCam | oh | 15:46 |
| NobodyCam | :) | 15:46 |
| lucasagomes | rameshg87, and ifarkas were talking about using size for the RAID stuff as a fallback if I remeber well | 15:46 |
| rameshg87 | lucasagomes, that was only a last alternative :) | 15:47 |
| lucasagomes | yup | 15:47 |
| lucasagomes | yeah as a fallback if nothing else can be used | 15:47 |
| jroll | lucasagomes: yeah, what's up? | 15:47 |
| lucasagomes | jroll, how does it work from nova pov? For e.g, for ephemeral we pass ephemeral_gb and ephemeral_size to ironic | 15:48 |
| lucasagomes | so it knows the size and format of the partition | 15:48 |
| jroll | lucasagomes: it just passes a url, it has a max size of 64MB | 15:48 |
| lucasagomes | jroll, is the config drive the same? It seems that nova will only pass a "configdrive" with a URL | 15:48 |
| lucasagomes | and it's up to us to decide the size/format etc | 15:49 |
| jroll | it's always an ISO, max size of 64MB | 15:49 |
| lucasagomes | gotcha | 15:49 |
| lucasagomes | oh it's an ISO!? should it was a json blob | 15:49 |
| jroll | the URL will be to an ISO image that is base64'd and gzipped | 15:49 |
| lucasagomes | but makes sense for a vm to be an ISO | 15:49 |
| jroll | so we just download and write | 15:49 |
| jroll | yeah | 15:49 |
| lucasagomes | gotcha | 15:50 |
| lucasagomes | jroll, so, there's any changes needed on the nova ironic driver to complete the work as well? | 15:50 |
| lucasagomes | (apart from the work on specific drivers) | 15:51 |
| jroll | references for how we write it out in the agent: https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/extensions/standby.py#L84 | 15:52 |
| jroll | and https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/shell/copy_configdrive_to_disk.sh | 15:52 |
| jroll | lucasagomes: yeah, see https://review.openstack.org/#/c/98930/8/specs/kilo/approved/use-configdrive-with-ironic.rst | 15:52 |
| jroll | basically need to write code to put it in swift and pass ironic the url | 15:52 |
| lucasagomes | I see | 15:53 |
| jroll | we have the generation code downstream, need to get it up :| | 15:53 |
| jroll | and need to write the swift bits still | 15:53 |
| lucasagomes | ack I will take a look | 15:55 |
| openstackgerrit | Dmitry Tantsur proposed stackforge/ironic-discoverd: Functional test for setting IPMI credentials https://review.openstack.org/142823 | 15:58 |
| lucasagomes | jroll, 1 more thing... so the content of the ISO is pretty much a json blob ? | 16:01 |
| jroll | lucasagomes: it's a bunch of files containing json | 16:01 |
| NobodyCam | lucasagomes: voted | 16:01 |
| * jroll checks if he has a server free with one | 16:01 | |
| openstackgerrit | Merged stackforge/ironic-discoverd: Enable functional testing with local ramdisk source https://review.openstack.org/142811 | 16:02 |
| lucasagomes | NobodyCam, thanks! yeah if needed a new review I will add that info | 16:02 |
| lucasagomes | ta much! | 16:02 |
| jroll | lucasagomes: give me an ssh key :) | 16:02 |
| NobodyCam | are we going to try and get a bug scrub in this morning? | 16:02 |
| *** david-lyle has joined #openstack-ironic | 16:02 | |
| lucasagomes | jroll, pm'ed | 16:02 |
| lucasagomes | NobodyCam, yeah we should do some | 16:03 |
| NobodyCam | I was looking for a email from devananda about it.. but I don't see it.. I suspect he got distracted by other (more importent) things :) | 16:05 |
| *** stendulker has quit IRC | 16:08 | |
| lucasagomes | there was an email about it? | 16:09 |
| NobodyCam | lucasagomes: there was? | 16:10 |
| NobodyCam | or is that a question? | 16:10 |
| lucasagomes | not sure, haven't seem it either | 16:10 |
| NobodyCam | ya I suspect he got pulled away | 16:11 |
| NobodyCam | :) | 16:11 |
| BadCub_ | morning Ironic | 16:11 |
| lucasagomes | I see yeah | 16:12 |
| NobodyCam | moring BadCub_ | 16:12 |
| NobodyCam | morning even | 16:12 |
| *** andreykurilin has quit IRC | 16:15 | |
| lucasagomes | BadCub_, morning | 16:16 |
| *** pcrews has joined #openstack-ironic | 16:18 | |
| NobodyCam | rloo: happen to be around? | 16:20 |
| rloo | hi, morning NobodyCam | 16:20 |
| NobodyCam | morning rloo :) | 16:20 |
| NobodyCam | I just had a question on a comment you have on 100951 | 16:21 |
| rloo | NobodyCam: ? | 16:22 |
| rameshg87 | hello all | 16:22 |
| rameshg87 | in the new ironic provisioner state machine | 16:22 |
| rameshg87 | https://github.com/openstack/ironic-specs/blob/master/specs/kilo/new-ironic-state-machine.rst | 16:22 |
| NobodyCam | wrt your question about property values being over written: do line2 186 - 188 answer that question? | 16:22 |
| NobodyCam | rloo: ^^^ | 16:22 |
| NobodyCam | mornig rameshg87 | 16:22 |
| rameshg87 | is there no way abort an ongoing request ? | 16:22 |
| rameshg87 | morning NobodyCam :) | 16:23 |
| jroll | rameshg87: there should be, just like there should be today | 16:23 |
| jroll | not sure if that's true | 16:23 |
| jroll | err | 16:23 |
| rameshg87 | jroll, i guess there isn't | 16:23 |
| rameshg87 | jroll, can't see from diagram | 16:23 |
| rameshg87 | jroll, and it's not in fsm either | 16:23 |
| jroll | oh, I see what you mean | 16:23 |
| rloo | NobodyCam: yes. but I don't think (honestly) I should have to look in the REST API Impact section to find that out. | 16:23 |
| rameshg87 | jroll, https://github.com/openstack/ironic/blob/master/ironic/common/states.py | 16:24 |
| rameshg87 | jroll, there is not DEPLOYING -> NOSTATE | 16:24 |
| jroll | rameshg87: right, but I don't think we can interrupt a deploy or whatever today either | 16:24 |
| jroll | (in general, there are some cases where you can) | 16:24 |
| NobodyCam | ok .. and that is valid. I was just checking if that was the information you were questioning | 16:24 |
| NobodyCam | TY :) | 16:24 |
| *** Marga_ has joined #openstack-ironic | 16:25 | |
| rameshg87 | jroll, previously we could just issue a delete i guess | 16:25 |
| rameshg87 | jroll, iirc delete used to abort an ongoing deployment | 16:25 |
| rloo | NobodyCam: I didn't -1 it cuz I still don't know what is a nit or whatever. I did give feedback though cuz I think it is important that people can understand what they're reading ;) | 16:25 |
| jroll | rameshg87: right now, if a node is deploying and you call 'nova delete', it doesn't delete cleanly | 16:25 |
| jroll | rameshg87: it did not afaik | 16:25 |
| NobodyCam | rloo: yeppers I agree :) | 16:25 |
| rameshg87 | jroll, :) | 16:25 |
| rameshg87 | jroll, i remember i have aborted ongoing deployment. i majorly run deploys without nova | 16:26 |
| rameshg87 | jroll, but i might be wrong. anyone else remember if we could delete ongoing deployments ? | 16:26 |
| *** PaulCzar has joined #openstack-ironic | 16:26 | |
| jroll | rameshg87: https://github.com/openstack/ironic/blob/master/ironic/api/controllers/v1/node.py#L330-332 | 16:26 |
| jroll | rameshg87: if a node is deploying, target state will be 'deployed' here | 16:27 |
| *** Marga_ has quit IRC | 16:28 | |
| *** bradjones has quit IRC | 16:28 | |
| *** Marga_ has joined #openstack-ironic | 16:28 | |
| rameshg87 | jroll, oh we could delete when it is in DEPLOYWAIT | 16:28 |
| rameshg87 | jroll, that's what i was meaning actually | 16:29 |
| jroll | yes, but not DEPLOYING | 16:29 |
| jroll | I tend to forget about DEPLOYWAIT since we leave our ramdisks running, sorry :) | 16:29 |
| *** stendulker has joined #openstack-ironic | 16:30 | |
| NobodyCam | :-p | 16:30 |
| rameshg87 | jroll, yeah i got it, thanks | 16:30 |
| rameshg87 | jroll, it's working as expected, my mistake :) | 16:30 |
| jroll | ha, no worries | 16:31 |
| * rameshg87 wonders what i was thinking | 16:31 | |
| jroll | ah, I see the confusion | 16:31 |
| jroll | that spec doesn't talk about the wait flag yet | 16:31 |
| *** bradjones has joined #openstack-ironic | 16:32 | |
| jroll | I would think we should be able to abort things with the wait flag set | 16:32 |
| *** pcaruana has quit IRC | 16:33 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: PXE: Pass root device hints via kernel cmdline https://review.openstack.org/142109 | 16:35 |
| rameshg87 | jroll, yeah .. but previous behaviour is intact | 16:36 |
| rameshg87 | jroll, but when i checked spec after i did something wrong, i wondered "oh i can't find it" | 16:36 |
| NobodyCam | dtantsur: are you around? | 16:36 |
| jroll | rameshg87: right :) | 16:37 |
| dtantsur | NobodyCam, yep | 16:37 |
| *** Marga_ has quit IRC | 16:38 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: PXE/ISCSI: Call partprobe after partioning the disk https://review.openstack.org/142838 | 16:39 |
| lucasagomes | dtantsur, toure bug ^ | 16:39 |
| lucasagomes | need test tho | 16:39 |
| NobodyCam | dtantsur: do you happen to recall why the ManagementInterface on rev 46 changes to IntrospectionInterface on rev 47? | 16:40 |
| NobodyCam | on https://review.openstack.org/#/c/100951 | 16:40 |
| dtantsur | NobodyCam, it was my request and we agreed on meeting. mostly because it's not management, but also because people want to use in-band introspection with e.g. ilo driver | 16:41 |
| dtantsur | NobodyCam, I'm pretty resistant to treat ManagementInterface as "everything we don't know where to put" :) | 16:41 |
| jroll | dtantsur: remember which meeting by chance? | 16:42 |
| NobodyCam | dtantsur: we were looking lastnight for the conversation releated to it | 16:42 |
| jroll | aha | 16:42 |
| jroll | AGREED: add new IntrospectionInterface (devananda, 17:34:12) | 16:42 |
| jroll | http://eavesdrop.openstack.org/meetings/ironic/2014/ironic.2014-12-01-17.02.html | 16:42 |
| dtantsur | also there was my ML post, but nearly nobody answered :) | 16:42 |
| dtantsur | yep | 16:42 |
| jroll | welp! | 16:43 |
| NobodyCam | d'oh | 16:43 |
| NobodyCam | Thnank you | 16:43 |
| jroll | RIP driver matrix | 16:44 |
| NobodyCam | lol | 16:45 |
| NobodyCam | its at good to know that we were not going crazy.. | 16:46 |
| jroll | I mean... | 16:46 |
| *** rameshg87 has quit IRC | 16:46 | |
| NobodyCam | s/at good/at least good/ | 16:46 |
| jroll | a bunch of people were in that meeting and don't remember it :P | 16:46 |
| NobodyCam | lol | 16:46 |
| * jroll afk for a bit | 16:46 | |
| dtantsur | IIRC people were distracted by something more interesting and just wanted to get rid of this one ASAP :) | 16:46 |
| NobodyCam | ack... quick brb (smoke) for me | 16:47 |
| *** lintan_ has quit IRC | 16:48 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: PXE/ISCSI: Call partprobe after partioning the disk https://review.openstack.org/142838 | 16:48 |
| *** Marga_ has joined #openstack-ironic | 16:51 | |
| devananda | morning, all | 16:52 |
| lucasagomes | devananda, morning | 16:52 |
| NobodyCam | good morning devananda | 16:52 |
| NobodyCam | devananda: jut fyi dtantsur recalled the meeting for the IntrospectionInterface stuff above ^^^^ | 16:53 |
| devananda | NobodyCam: yep, see that in scrollback. | 16:53 |
| *** stendulker has quit IRC | 16:54 | |
| NobodyCam | :) | 16:54 |
| devananda | my objection was about the REST API changse | 16:54 |
| devananda | wtf? where is my -1? | 16:55 |
| devananda | oh. gah. need more coffee.... gertty hasn't synced yet | 16:56 |
| lucasagomes | devananda, sorry for pinging you right in the morning, but ur opnion is being requested here https://review.openstack.org/#/c/137762/ | 16:56 |
| *** dlaube has joined #openstack-ironic | 16:58 | |
| * NobodyCam hands a large cup of coffee to devananda! | 16:59 | |
| devananda | lucasagomes: so I still don't see why you're not just changing the MultiType class itself | 17:02 |
| lucasagomes | devananda, because MultiType is generic | 17:03 |
| lucasagomes | where ApiuMultiType is a MultiType class with some pre-defined types | 17:03 |
| lucasagomes | because all the json-like fields in the API should support the same values | 17:03 |
| devananda | lucasagomes: but MultiType is not actually used anywhere else | 17:03 |
| lucasagomes | but MultiType could be used for other attibutes | 17:03 |
| devananda | is there a plan to use it for some other field? | 17:04 |
| devananda | what other sorts of attributes could use? | 17:04 |
| lucasagomes | not from the top of my head, I just didn't want to turn it into a less generic thing | 17:04 |
| *** jerryz_ has quit IRC | 17:04 | |
| * lucasagomes thinks | 17:05 | |
| devananda | IIUC, you're adding more strict type checking to the existing JSON fields -- which is fine | 17:05 |
| devananda | you're not adding a _new_ type of field | 17:05 |
| devananda | in what case would we prefer less strict type checking (eg, the current MultiType) over the new type? | 17:05 |
| lucasagomes | yeah looks at the attributes I can't see any other field where we would need multi type for now | 17:07 |
| *** EmilienM has quit IRC | 17:08 | |
| lucasagomes | alright I will make MultiType more strict then | 17:08 |
| *** viktors1 has quit IRC | 17:09 | |
| *** EmilienM has joined #openstack-ironic | 17:09 | |
| * lucasagomes maybe should propose MultiType to wsme, so it can serve as a base class for other types | 17:09 | |
| devananda | lucasagomes: that sounds good. also, wsme now as a decorator to remove XML support | 17:10 |
| devananda | I believe mrda had started working on that bug a while back, but his work was abandoned / superceded by those changes in wsme | 17:10 |
| devananda | not sure if you saw them, but I was reminded recently | 17:10 |
| lucasagomes | yeah i will submit a patch with the multitype one after fixng it in ironic | 17:11 |
| lucasagomes | so once it's merged we may use it instead | 17:11 |
| *** Masahiro has joined #openstack-ironic | 17:13 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Issue with FSM's _target_state https://review.openstack.org/142303 | 17:15 |
| *** Masahiro has quit IRC | 17:17 | |
| *** dtantsur is now known as dtantsur|afk | 17:19 | |
| *** Marga_ has quit IRC | 17:21 | |
| *** romcheg has quit IRC | 17:23 | |
| openstackgerrit | Merged openstack/ironic: Update command options in the Installation Guide https://review.openstack.org/142107 | 17:23 |
| *** romcheg has joined #openstack-ironic | 17:23 | |
| *** romcheg has quit IRC | 17:24 | |
| JayF | lucasagomes: we actually already have had and solved that bug before agent got upstreamed iirc | 17:24 |
| JayF | lucasagomes: 1s and I'll link you | 17:24 |
| lucasagomes | JayF, oh, right | 17:24 |
| lucasagomes | JayF, this is the fix I proposed https://review.openstack.org/142838 | 17:24 |
| JayF | lucasagomes: https://github.com/openstack/ironic-python-agent/blob/master/ironic_python_agent/shell/copy_configdrive_to_disk.sh#L46 | 17:25 |
| lucasagomes | +1 sounds good | 17:25 |
| lucasagomes | yeah i do call partprobe on the device | 17:25 |
| lucasagomes | and then try to list the partitions | 17:25 |
| JayF | yeah; exact;y | 17:25 |
| lucasagomes | if both succeed I return | 17:25 |
| lucasagomes | yeah idk how we didn't hit that before | 17:25 |
| *** achanda has joined #openstack-ironic | 17:28 | |
| NobodyCam | i have a general question: its comming from a update to a landed spec. https://review.openstack.org/#/c/141269 .. there are two comments about possiably adding a new field for driver's data. I kinda feel its not with in the scope of the spec. but would like other opinions on that. | 17:34 |
| *** sambetts has quit IRC | 17:40 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Extend API multivalue fields https://review.openstack.org/137762 | 17:44 |
| NobodyCam | also any reson to keep the "Current" state machine stuff on the agenda? | 17:45 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Extend API multivalue fields https://review.openstack.org/137762 | 17:46 |
| *** Marga_ has joined #openstack-ironic | 17:50 | |
| JayF | NobodyCam: I agree wholehardedly, additionally, I don't think any of our current power drivers takes previous boot device into consideration | 17:57 |
| JayF | NobodyCam: i.e. I'm pretty sure we force boot from pxe/disk every reboot as is needed in IPA | 17:57 |
| devananda | JayF: ++ for being explicit within the driver, and letting each driver decide how best to accomplish the behavior that driver expects | 17:57 |
| NobodyCam | :) | 17:58 |
| *** derekh has quit IRC | 17:58 | |
| *** Marga_ has quit IRC | 18:00 | |
| *** Marga_ has joined #openstack-ironic | 18:02 | |
| JayF | lucasagomes: the "why it worked before" is that the kernel does actually pay attention and update part tables on its own sometimes | 18:03 |
| lucasagomes | JayF, yeah, it doesn't fail all the time | 18:03 |
| lucasagomes | but quite often, like 20% of the time | 18:04 |
| JayF | lucasagomes: we actually didn't see it until the change where we were looking for the config-2 label preexisting in the image | 18:04 |
| lucasagomes | oh | 18:04 |
| JayF | but even that wasn't failing often | 18:04 |
| JayF | before that we blindly appended a partition to the end of the drive (what we still do if there's no existing config-2 ox) | 18:04 |
| lucasagomes | gotcha | 18:05 |
| lucasagomes | I thought that parted, after partitioning a disk, triggered the kernel to re-read the partition tables | 18:06 |
| lucasagomes | but seems parted can read it anyway even if it's not showing up on the host | 18:06 |
| *** Nisha has joined #openstack-ironic | 18:06 | |
| JayF | oh absolutely | 18:06 |
| JayF | parted is just reading bits off the disk | 18:06 |
| JayF | and does its own thing | 18:07 |
| lucasagomes | yeah | 18:07 |
| lucasagomes | testing it now, seems partprobe does the trick | 18:08 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: PXE/ISCSI: Call partprobe after partioning the disk https://review.openstack.org/142838 | 18:10 |
| NobodyCam | brb | 18:17 |
| *** harlowja_away is now known as harlowja | 18:23 | |
| Nisha | NobodyCam, devananda hi | 18:26 |
| Nisha | NobodyCam, devananda this is regarding introspection comments | 18:26 |
| Nisha | jroll, NobodyCam devananda , if the url shall be PUT /v1/nodes/<uuid>/states/provision {"target": "inspected"} , then the cli to be used shall be node-set-provision-state ? or the new CLI node-inspect? | 18:30 |
| devananda | Nisha: JayF: I noticed in the cleaning and zapping specs that new API endpoints are being introduced for those as well | 18:31 |
| devananda | JayF: wondering if you can shed some light on the reasoning for that, which I may have missed in the spec | 18:31 |
| Nisha | devananda, i am fine to change the url to as suggested, but then i think in that case even the cli shall be the node-set-provision-state. | 18:33 |
| NobodyCam | Nisha: yep that would need to change as well | 18:34 |
| NobodyCam | and good morning Nisha | 18:34 |
| Nisha | Ok. Good Morning NobodyCam | 18:35 |
| Nisha | I will just the post the patches now after modification as suggested | 18:36 |
| Nisha | I am on leave from tomorrow(19th), it will help if i can get comments now so that i can resolve the comments if any as sson as possible | 18:36 |
| Nisha | rloo, are you around> | 18:37 |
| Nisha | ? | 18:37 |
| rloo | hi Nisha. I am :-) | 18:38 |
| Nisha | rloo, i saw your comments too | 18:38 |
| Nisha | on introspection spec | 18:38 |
| rloo | Nisha: I forgot what they were. Did you have a question about them? | 18:38 |
| Nisha | i wanted to ask does it require me to implement introspection states as well along with introspection? | 18:39 |
| Nisha | if yes, then i will add it as a work item | 18:39 |
| *** ifarkas has quit IRC | 18:40 | |
| Nisha | rloo, this question is from work around ..."But there is a series of patches that add a finite-state-machine for the existing (not new) states, and it isn't as simple as adding variables for the states, you'd need to add them to the finite-state-machine too ;)" | 18:40 |
| rloo | Nisha: oh, the part about hooking this into the new FSM. We don't have a mechanism for the new FSM finalized yet, so hard to say. I'd word it something like 'it needs to be added to the new FSM'. | 18:40 |
| rloo | Nisha: I seriously doubt we'd approve any code changes to that spec, until after we have the plumbing working to handle the new fsm. | 18:41 |
| *** jcoufal_ has quit IRC | 18:41 | |
| Nisha | rloo, 'that spec' means? | 18:42 |
| rloo | Nisha: so the spec can be approved, but the code won't be merged until after we have new fsm plumbing in code. | 18:42 |
| rloo | 'that spec' is the spec you're talking about. | 18:42 |
| Nisha | yes, i understand that | 18:42 |
| Nisha | rloo, its not expected from introspection to add any code for state model... am i correct? | 18:43 |
| *** viktors1 has joined #openstack-ironic | 18:44 | |
| *** igordcard has quit IRC | 18:44 | |
| rloo | Nisha: no. The only thing (I think) is that you might have to add transitions for the INSPECT* states, into the fsm. But actually, I don't even think you'll need to do that. That should be done already. So no, you shouldn't have to do anything there. | 18:45 |
| Nisha | rloo, thanks. | 18:45 |
| rloo | Nisha: just looking at your spec. I'd remove the part under Dependencies, 'It can be worked around...' | 18:46 |
| rloo | Nisha: and maybe where it says 'Requires http://...', mention 'Requires implementation of http://...' | 18:47 |
| rloo | Nisha, I'll add a comment about that to your spec in case someone wonders later why you made those spec changes. | 18:48 |
| *** athomas has quit IRC | 18:49 | |
| Nisha | rloo, ok. i added as per review comments. and because my code was working fine by just adding states into the states.py file. Since it is not implemented in new state model, i understand that just adding in states.py may not be correct to do. | 18:49 |
| *** bradjones has quit IRC | 18:49 | |
| Nisha | I will remove as suggested | 18:49 |
| *** bradjones has joined #openstack-ironic | 18:51 | |
| rloo | thx Nisha | 18:52 |
| *** afaranha has joined #openstack-ironic | 18:57 | |
| *** Masahiro has joined #openstack-ironic | 19:02 | |
| JayF | JoshNang: ^ | 19:04 |
| JoshNang | devananda: cleaning adds an endpoint to list the steps that would be run on a node. i think that's handy for operators to get some visibility, but not completely necessary. i would like to see the provision state api return the current clean_step as well, again for visibility. | 19:06 |
| *** Masahiro has quit IRC | 19:07 | |
| *** penick has joined #openstack-ironic | 19:08 | |
| *** Marga_ has quit IRC | 19:09 | |
| JoshNang | devananda: for zapping, if we want to use the provision state api with an arg in the PUT body (what zap step or list of steps to run), i'm ok with that. zapping needs to be able to list the zap steps so you can choose what to execute, so i think that should be a new api endpoint | 19:10 |
| JoshNang | (or zapping and cleaning can share an endpoint for cleaning/zapping steps, as i see zapping as a superset of cleaning) | 19:10 |
| *** Marga_ has joined #openstack-ironic | 19:12 | |
| *** viktors1 has quit IRC | 19:15 | |
| devananda | JoshNang: "get some visibility" - so it is read-only, and there is nothing a user can do with the information provided? | 19:17 |
| devananda | JoshNang: can you explain in what situation a user would need to pick and choose the zapping (or cleaning) steps to run | 19:18 |
| JoshNang | devananda: as of yet, yes, it's read-only. i can imagine someone's going to ask for per-node configs for cleaning, and some of this is left over from zapping and cleaning being just zapping | 19:19 |
| JoshNang | devananda: for zapping, i imagine something like "cache an image" would be a zap step, but not something you may want for all nodes. or you want half your fleet configured with raid10 for compute nodes, and raid6 for storage nodes | 19:20 |
| devananda | hmm | 19:23 |
| devananda | so I see that, while a node is in MANAGE state, a user would want to queue up a set of operatoions to perform during ZAPPING (or perhaps move the node to ZAPPING then request each one individually?) | 19:24 |
| devananda | but cleaning seems less user-driver and more "the driver should know what is there and how to clean it" | 19:25 |
| JoshNang | agreed on both. i can drop the clean_step api. it will likely make everything cleaner on the API side | 19:26 |
| devananda | thinking more on exposing possible zap-related operations, I think that has to be visible for nodes in MANAGE state | 19:27 |
| NobodyCam | yuriyz: are you still around? | 19:27 |
| JayF | We have to expose what clean_step is happening somewhere in the API | 19:28 |
| JoshNang | right, which is what the api was originally designed for. discoverable api's are nice :) | 19:28 |
| JayF | preferably in node details | 19:28 |
| JoshNang | JayF: agreed, as part of provision state, but there was another api that just listed which steps were going to be executed in cleaning | 19:28 |
| devananda | eg, while node is in MANAGE state, I could GET a list of supported actions, then PUT /node/NNN/state/provision {'target': 'zap', 'actions': ['list', 'of', 'actions']} | 19:29 |
| devananda | and then the node moves to ZAPPING | 19:29 |
| devananda | and proceeds through that list linearly | 19:29 |
| jroll | JoshNang: wasn't the intent of that API endpoint for the agent to be able to fetch its steps from ironic? | 19:29 |
| devananda | I agree that current activity, in both cleaning and zapping, needs to be visible | 19:29 |
| devananda | while a user should be able to instruct ironic to perform specific actions durign zapping (and thus must be able to discover the list of /possible/ actions before initiating ZAPPING) | 19:30 |
| JoshNang | i'd be much happier to use the existing provision API than creating a new one | 19:30 |
| devananda | I do not think the same applies for cleaning | 19:30 |
| devananda | JoshNang: ++ | 19:30 |
| devananda | for state transitions, I think we need to use the current API | 19:30 |
| JoshNang | jroll: nope. the agent knows what steps it can execute (harware managers). it tell sthe conductor what it can do. the conductor makes the list of steps to run, and when its an agent step, tells the agent to execute that step | 19:31 |
| devananda | and thus, also, to see what's happening with respect to the provision state (which is the whole gamut from ENROLL to ACTIVE to CLEAN and so on) | 19:31 |
| jroll | JoshNang: ok, it keeps going back and forth in my head where that info comes from | 19:31 |
| jroll | thanks | 19:31 |
| JoshNang | jroll: no worries! | 19:32 |
| devananda | heh, so that's different than, say, what iLO will do | 19:32 |
| JoshNang | yup. yay in + out of band! | 19:32 |
| devananda | which IIUC is: check hardware support (version, license, etc) and then the driver immediately knows what that node supports | 19:32 |
| devananda | *check ... by talking to BMC | 19:32 |
| JoshNang | seems reasonable | 19:32 |
| JoshNang | i'm gonna go grab lunch before lines get too long. i'll update the specs this afternoon (zapping needs some love) | 19:33 |
| openstackgerrit | Joshua Harlow proposed openstack/ironic: Add a fsm state -> dot diagram generator https://review.openstack.org/142887 | 19:33 |
| JoshNang | devananda: thanks for the help clarifying | 19:33 |
| devananda | so, for inspect, it sounds as though you agree -- that should use the /nodes/NNNN/state/provision API endpoint, rather than create a new one -- yes? | 19:33 |
| devananda | JoshNang: np | 19:34 |
| NobodyCam | devananda: you have talked me in to it | 19:34 |
| NobodyCam | :) | 19:34 |
| *** viktors1 has joined #openstack-ironic | 19:35 | |
| *** dprince has quit IRC | 19:35 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic-specs: Discover node properties using new CLI node-inspect https://review.openstack.org/100951 | 19:36 |
| *** penick has quit IRC | 19:36 | |
| *** bradjones has quit IRC | 19:36 | |
| Nisha | devananda, NobodyCam jroll rloo ^^^^ | 19:37 |
| Nisha | updated the introspection spec as comments | 19:37 |
| NobodyCam | Nisha: line 121 is not correct, the node-set-provision-state command | 19:38 |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic-specs: Discover node properties using new CLI node-inspect https://review.openstack.org/100951 | 19:39 |
| Nisha | NobodyCam, thanks. missed it | 19:39 |
| Nisha | updated again | 19:39 |
| Nisha | ^^^ | 19:39 |
| *** bradjones has joined #openstack-ironic | 19:40 | |
| NobodyCam | :-p | 19:41 |
| Nisha | i would request review for this today, as i am on leave for next two weeks starting 19th Dec till 4th Jan | 19:45 |
| Nisha | i will rebase the dependent spec as soon as this is in approval | 19:48 |
| Nisha | :) | 19:48 |
| *** penick has joined #openstack-ironic | 19:51 | |
| NobodyCam | Nisha: will look over in a bit.. | 20:02 |
| Nisha | ok NobodyCam | 20:03 |
| *** Marga_ has quit IRC | 20:05 | |
| *** Marga_ has joined #openstack-ironic | 20:06 | |
| *** rlr0329 has quit IRC | 20:07 | |
| *** penick has quit IRC | 20:09 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: PXE/ISCSI: Call partprobe after partioning the disk https://review.openstack.org/142838 | 20:18 |
| lucasagomes | I will call it a day, have a good night everyone | 20:18 |
| *** bradjones has quit IRC | 20:18 | |
| *** Marga_ has quit IRC | 20:19 | |
| *** Marga_ has joined #openstack-ironic | 20:20 | |
| rloo | night lucasagomes | 20:22 |
| *** andreykurilin has joined #openstack-ironic | 20:22 | |
| NobodyCam | night lucasagomes | 20:23 |
| *** Marga_ has quit IRC | 20:25 | |
| *** viktors1 has quit IRC | 20:32 | |
| devananda | rloo: based on your feedback to 139217, I'm refactoring that considerably more | 20:34 |
| *** viktors1 has joined #openstack-ironic | 20:35 | |
| rloo | devananda: thx. | 20:36 |
| *** viktors1 has quit IRC | 20:40 | |
| NobodyCam | Nisha: still around? | 20:43 |
| *** harlowja has quit IRC | 20:45 | |
| *** harlowja_ has joined #openstack-ironic | 20:45 | |
| *** Marga_ has joined #openstack-ironic | 20:50 | |
| *** Masahiro has joined #openstack-ironic | 20:50 | |
| NobodyCam | devananda: question: from MANAGE state I "node-set-provision-state inspect" will the target provision state be INSPECTED or back MANAGE(D) | 20:51 |
| *** Nisha_away has joined #openstack-ironic | 20:52 | |
| NobodyCam | *back to MANAGE(D) | 20:52 |
| *** Nisha has quit IRC | 20:53 | |
| *** Masahiro has quit IRC | 20:55 | |
| *** Marga_ has quit IRC | 20:55 | |
| *** Nisha_brb has joined #openstack-ironic | 20:56 | |
| *** Nisha_away has quit IRC | 20:58 | |
| *** dprince has joined #openstack-ironic | 21:02 | |
| Nisha_brb | NobodyCam, yes | 21:04 |
| devananda | rloo: oy. found a bug in ManagerDoSyncPowerStateTestCase -- it's not testing any cases where the power driver raises an exception | 21:06 |
| *** Nisha_brb is now known as Nisha | 21:06 | |
| devananda | rloo: it sets up self.node.power.set_power_state to raise an exception, but is mocking the function that would call that function :( | 21:07 |
| rloo | devananda: ugh. these unit tests can be tricky :-( | 21:07 |
| devananda | rloo: if I wasn't deep into refactoring this code already, I'd want that to be a separate patch | 21:09 |
| rloo | devananda: yeah, I was going to ask you if you were going to fix in a separate patch and decided to shut up ;) | 21:10 |
| devananda | :p | 21:10 |
| *** Nisha_away has joined #openstack-ironic | 21:10 | |
| rloo | devananda: you are making changes to the power stuff in that patch, so it does make sense to fix the test there, so it can test your changes properly | 21:10 |
| *** afaranha has quit IRC | 21:11 | |
| *** Nisha has quit IRC | 21:13 | |
| Nisha_away | devananda, NobodyCam , rloo , jroll could i request review for introspection spec? | 21:13 |
| jroll | Nisha_away: you can request, but I don't have time right now | 21:14 |
| rloo | hi Nisha_away, you can always request. the problem is responding. | 21:14 |
| jroll | (sorry) | 21:14 |
| NobodyCam | Nisha_away: line 126 is out of date :( | 21:14 |
| NobodyCam | also I have not gone throu the entire rest API section | 21:15 |
| rloo | Nisha_away: I'm in the middle of something. I'll try later. Although, if you're going to be away anyway, it doesn't seem that urgent to look at your spec now. It might be more important to get stuff done to unblock others so they can work next week. | 21:15 |
| Nisha_away | ok | 21:16 |
| *** Nisha_away is now known as Nisha | 21:16 | |
| rloo | Nisha_away: I don't mean to be hard-nosed about it. If I were superhuman and could deal with everyone's patches, I'd love to do that. | 21:16 |
| Nisha | rloo, its fine | 21:16 |
| Nisha | i just asked... | 21:16 |
| rloo | Nisha: yeah, it is ok to ask. I thought it might help if I gave you my reasoning ;) | 21:17 |
| *** lucasagomes has quit IRC | 21:18 | |
| openstackgerrit | Nisha Agarwal proposed openstack/ironic-specs: Discover node properties using new CLI node-inspect https://review.openstack.org/100951 | 21:20 |
| Nisha | NobodyCam, ^^^ updated as per request | 21:20 |
| NobodyCam | :) | 21:20 |
| *** mjturek has quit IRC | 21:30 | |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Issue with FSM's _target_state https://review.openstack.org/142303 | 21:40 |
| rloo | devananda: I think (hope) I'm done with going through the unit tests etc for the fsm ^^^ I hope it helps. | 21:41 |
| openstackgerrit | Alex Weeks proposed openstack/ironic-specs: Add pluggable metrics backend for Ironic https://review.openstack.org/137171 | 21:42 |
| openstackgerrit | Alex Weeks proposed openstack/ironic-specs: Add pluggable metrics backend for Ironic https://review.openstack.org/137171 | 21:44 |
| *** erwan_taf has quit IRC | 21:47 | |
| *** Marga_ has joined #openstack-ironic | 21:52 | |
| devananda | rloo: cheers | 21:52 |
| openstackgerrit | Alex Weeks proposed openstack/ironic-specs: Add pluggable metrics backend for Ironic https://review.openstack.org/137171 | 21:53 |
| aweeks | jroll: JayF: devananda: https://review.openstack.org/#/c/137171 if you have a moment :) | 21:56 |
| *** Marga_ has quit IRC | 21:56 | |
| *** Marga_ has joined #openstack-ironic | 22:00 | |
| *** dprince has quit IRC | 22:08 | |
| *** Nisha has quit IRC | 22:21 | |
| *** romcheg has joined #openstack-ironic | 22:25 | |
| NobodyCam | so I shouldn't try and make you laugh while the call is in progress... So ... A guy with dyslexia walks into a bra. | 22:35 |
| NobodyCam | gah | 22:36 |
| NobodyCam | ww | 22:36 |
| NobodyCam | :-p | 22:36 |
| JayF | NobodyCam: What does an insomniac agnostic dyslexic person do? | 22:36 |
| JayF | NobodyCam: Stays up all night wondering if there really is a dog | 22:36 |
| NobodyCam | lol | 22:37 |
| NobodyCam | there is | 22:37 |
| NobodyCam | there is | 22:37 |
| devananda | NobodyCam: there might even be more than one dog :) | 22:37 |
| NobodyCam | lol | 22:37 |
| JayF | Revised: Stays up all night wondering if there really is one or more sdog | 22:37 |
| NobodyCam | :) | 22:38 |
| *** Masahiro has joined #openstack-ironic | 22:39 | |
| *** Masahiro has quit IRC | 22:44 | |
| *** david-ly_ has joined #openstack-ironic | 22:58 | |
| *** david-lyle has quit IRC | 23:03 | |
| *** mitz has quit IRC | 23:03 | |
| *** andreykurilin has quit IRC | 23:03 | |
| *** romcheg has quit IRC | 23:03 | |
| *** mitz has joined #openstack-ironic | 23:06 | |
| *** achanda has quit IRC | 23:15 | |
| *** EmilienM is now known as EmilienM|pto | 23:21 | |
| *** ryanpetrello has quit IRC | 23:25 | |
| NobodyCam | devananda: just check'n do you think you'll have a new rev of 139217 up today? | 23:28 |
| devananda | working on it | 23:30 |
| devananda | between meetings | 23:30 |
| NobodyCam | :) | 23:30 |
| NobodyCam | was just checkin not pushing | 23:30 |
| NobodyCam | :) | 23:31 |
| *** viktors1 has joined #openstack-ironic | 23:31 | |
| *** viktors1 has quit IRC | 23:32 | |
| *** viktors1 has joined #openstack-ironic | 23:33 | |
| NobodyCam | anyone ever figure out what up with https://review.openstack.org/#/c/128965 | 23:40 |
| *** r-daneel has quit IRC | 23:44 | |
| *** Marga_ has quit IRC | 23:52 | |
| *** Marga_ has joined #openstack-ironic | 23:53 | |
| *** viktors1 has quit IRC | 23:54 | |
| *** Marga__ has joined #openstack-ironic | 23:56 | |
| *** romcheg has joined #openstack-ironic | 23:57 | |
| *** Marga_ has quit IRC | 23:57 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic: Refactor async helper methods in conductor/manager.py https://review.openstack.org/139217 | 23:58 |
| devananda | all t3h refactoring | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!