| *** smoriya has joined #openstack-ironic | 00:19 | |
| *** zhenguo has joined #openstack-ironic | 00:19 | |
| *** yuanying_ has joined #openstack-ironic | 00:29 | |
| *** naohirot has joined #openstack-ironic | 00:29 | |
| *** yuanying has quit IRC | 00:31 | |
| *** jjohnson2 has joined #openstack-ironic | 00:39 | |
| openstackgerrit | Yuiko Takada proposed openstack/ironic-inspector: Migrate to oslo_db https://review.openstack.org/181905 | 00:52 |
|---|---|---|
| *** jjohnson2 has quit IRC | 00:55 | |
| openstackgerrit | Yuiko Takada proposed openstack/ironic-inspector: Pass environment variables of proxy to tox https://review.openstack.org/191598 | 00:55 |
| openstackgerrit | Yuiko Takada proposed openstack/ironic: Pass environment variables of proxy to tox https://review.openstack.org/191599 | 01:02 |
| *** jjohnson2 has joined #openstack-ironic | 01:03 | |
| *** kan_ has joined #openstack-ironic | 01:59 | |
| *** davideagnello has joined #openstack-ironic | 02:23 | |
| openstackgerrit | Yuiko Takada proposed openstack/ironic-inspector: Load authenticate token from HTTP header https://review.openstack.org/174231 | 02:38 |
| *** yuikotakada has joined #openstack-ironic | 02:55 | |
| *** oomichi has joined #openstack-ironic | 03:02 | |
| openstackgerrit | Yuiko Takada proposed openstack/ironic-inspector: Migrate to oslo_db https://review.openstack.org/181905 | 03:11 |
| *** coolsvap|afk is now known as coolsvap | 03:32 | |
| *** logan2 has quit IRC | 03:35 | |
| *** saripurigopi has joined #openstack-ironic | 03:55 | |
| *** amotoki has joined #openstack-ironic | 03:56 | |
| *** zz_natorious is now known as natorious | 04:02 | |
| *** amotoki has quit IRC | 04:04 | |
| *** achanda has joined #openstack-ironic | 04:07 | |
| *** amotoki has joined #openstack-ironic | 04:08 | |
| *** natorious is now known as zz_natorious | 04:14 | |
| *** achanda has quit IRC | 04:17 | |
| *** amotoki has quit IRC | 04:31 | |
| *** yuanying has joined #openstack-ironic | 04:39 | |
| *** amotoki has joined #openstack-ironic | 04:41 | |
| *** yuanying_ has quit IRC | 04:42 | |
| *** ramineni has joined #openstack-ironic | 04:59 | |
| *** davideagnello has quit IRC | 05:00 | |
| *** pradipta has joined #openstack-ironic | 05:00 | |
| *** jjohnson2 has quit IRC | 05:01 | |
| mrda | devananda: Have we finalised midcycle dates yet? | 05:09 |
| *** lazy_prince has joined #openstack-ironic | 05:37 | |
| *** rameshg87 has joined #openstack-ironic | 05:37 | |
| rameshg87 | good morning ironic | 05:38 |
| lazy_prince | o/ | 05:38 |
| rameshg87 | o/ | 05:38 |
| yuikotakada | rameshg87, lazy_prince: o/ | 05:38 |
| rameshg87 | yuikotakada: o/ | 05:38 |
| *** davideagnello has joined #openstack-ironic | 05:42 | |
| *** davideagnello has quit IRC | 05:45 | |
| *** bradjones has quit IRC | 05:46 | |
| *** bradjones has joined #openstack-ironic | 05:49 | |
| *** bradjones has quit IRC | 05:49 | |
| *** bradjones has joined #openstack-ironic | 05:49 | |
| *** davideagnello has joined #openstack-ironic | 05:51 | |
| openstackgerrit | Gopi Krishna S proposed openstack/ironic: Address follow-up comments on ucs drivers https://review.openstack.org/190913 | 05:54 |
| saripurigopi | good morning all | 05:54 |
| openstackgerrit | Merged openstack/ironic: IPA: Do a soft power off at the end of deployment https://review.openstack.org/185667 | 05:57 |
| *** tteggel has quit IRC | 05:59 | |
| *** tteggel has joined #openstack-ironic | 06:01 | |
| openstackgerrit | SHIGEMATSU Mitsuhiro proposed openstack/ironic-specs: iPXE to use Swift Temporary URLs https://review.openstack.org/181920 | 06:03 |
| *** ukalifon has joined #openstack-ironic | 06:07 | |
| *** yuanying_ has joined #openstack-ironic | 06:07 | |
| *** ijw has joined #openstack-ironic | 06:08 | |
| *** radek__ has joined #openstack-ironic | 06:09 | |
| *** yuanying has quit IRC | 06:10 | |
| *** yuanying_ has quit IRC | 06:11 | |
| openstackgerrit | Michael Davies proposed openstack/python-ironicclient: Cache negotiated api microversion for server https://review.openstack.org/173674 | 06:13 |
| openstackgerrit | Tan Lin proposed openstack/ironic: Add a new boot section 'trusted_boot' for PXE https://review.openstack.org/191661 | 06:14 |
| *** yuanying has joined #openstack-ironic | 06:15 | |
| *** davideagnello has quit IRC | 06:25 | |
| *** ifarkas has joined #openstack-ironic | 06:33 | |
| *** itamarl has joined #openstack-ironic | 06:37 | |
| *** dtantsur|afk is now known as dtantsur | 06:41 | |
| dtantsur | Morning Ironic! | 06:41 |
| zhenguo | morning dtantsur | 06:46 |
| dtantsur | zhenguo, o/ | 06:46 |
| zhenguo | dtantsur, I proposed a new project ironic-dashboard to openstack, https://github.com/niuzhenguo/ironic-dashboard and https://review.openstack.org/191131, want to see if you are interested about this. | 06:49 |
| dtantsur | possibly :) did not consider it yet, though sounds promising | 06:50 |
| zhenguo | and this will be one agenda of today's meeting. | 06:50 |
| dtantsur | cool! | 06:51 |
| openstackgerrit | SHIGEMATSU Mitsuhiro proposed openstack/bifrost: WIP: Initial changes for testing VM creation for CentOS https://review.openstack.org/191504 | 06:51 |
| zhenguo | dtantsur :) | 06:52 |
| openstackgerrit | SHIGEMATSU Mitsuhiro proposed openstack/bifrost: Vagrantfile and vagrant.yml for testing NUC https://review.openstack.org/191110 | 06:54 |
| openstackgerrit | SHIGEMATSU Mitsuhiro proposed openstack/bifrost: Add os_ironic_facts module https://review.openstack.org/191093 | 06:57 |
| yuikotakada | dtantsur: o/ | 07:01 |
| dtantsur | yuikotakada, o/ | 07:01 |
| dtantsur | we got some rename done \o/ | 07:02 |
| yuikotakada | yeah! dtantsur, thank you for a lot of paper works! | 07:02 |
| dtantsur | :) | 07:02 |
| dtantsur | the last paper-work thingy should be https://review.openstack.org/#/c/188798 I'll revive it today | 07:03 |
| openstackgerrit | Tan Lin proposed openstack/ironic: Improve the ability to resolve capability value https://review.openstack.org/190900 | 07:06 |
| yuikotakada | dtantsur: Oops, I forgot it sorry, you changed just gate jobs from python34 to python3-jobs, right? when you will have been finished rebase, I'll +1 | 07:08 |
| dtantsur | yeah, I think it's the only change | 07:09 |
| dtantsur | yuikotakada, I tried pypy, but our dependencies do not install on it :( | 07:09 |
| dtantsur | so no pypy-jobs for now | 07:09 |
| yuikotakada | dtantsur: dependency? between discoverd and inspector? | 07:10 |
| dtantsur | yuikotakada, no, I mean stuff in requirements.txt. nevermind anyway, that's not important. | 07:11 |
| *** yog__ has joined #openstack-ironic | 07:11 | |
| *** romainh has joined #openstack-ironic | 07:12 | |
| *** Haomeng|2 has joined #openstack-ironic | 07:15 | |
| yuikotakada | dtantsur, ok ;) | 07:18 |
| *** Haomeng has quit IRC | 07:18 | |
| *** dlpartain has joined #openstack-ironic | 07:28 | |
| rameshg87 | saripurigopi: hi | 07:29 |
| saripurigopi | rameshg87: hi | 07:29 |
| rameshg87 | saripurigopi: I think we can close the other defect as invalid as it's not related to any of openstack or stackforge projects | 07:29 |
| saripurigopi | rameshg87: okay | 07:30 |
| rameshg87 | saripurigopi: if you are tracking the issues that you fix in github issue tracker, you can point to the github issue | 07:30 |
| rameshg87 | saripurigopi: or you can reference the commit in github which fixes the mentioned one | 07:30 |
| rameshg87 | saripurigopi: or you can start using launchpad to track UcsSdk issues :) | 07:30 |
| saripurigopi | rameshg87: as of now there is no tracking method we are following for UcsSdk. | 07:31 |
| rameshg87 | saripurigopi: but unless you can point the bug to a valid launchpad project, you can close it as invalid | 07:31 |
| saripurigopi | rameshg87: sure, will mark it as invalid. | 07:32 |
| saripurigopi | rameshg87: will discuss with the team about making it part of stack forge projects, only the utils part alone. UcsSdk is very big. | 07:32 |
| rameshg87 | saripurigopi: okay | 07:33 |
| *** chlong has quit IRC | 07:34 | |
| *** subscope has joined #openstack-ironic | 07:34 | |
| *** dlpartain has left #openstack-ironic | 07:35 | |
| *** dmitchell has joined #openstack-ironic | 07:35 | |
| saripurigopi | rameshg87: thank you | 07:38 |
| *** stendulker has joined #openstack-ironic | 07:38 | |
| saripurigopi | rameshg87: is RAIDInterface. update_raid_info common across all the drivers? Will there be driver specific implementation, I guess no right? | 07:41 |
| *** ndipanov has joined #openstack-ironic | 07:42 | |
| rameshg87 | saripurigopi: a driver can implement if it wants | 07:42 |
| rameshg87 | saripurigopi: but the common one should be applicable for all the drivers | 07:42 |
| openstackgerrit | Merged openstack/ironic-inspector: Update devstack plugin for the recent authentication change https://review.openstack.org/191111 | 07:43 |
| rameshg87 | saripurigopi: I raised review for it on Friday | 07:43 |
| rameshg87 | saripurigopi: https://review.openstack.org/#/c/155230/23/ironic/common/raid.py | 07:43 |
| rameshg87 | it has that method | 07:43 |
| saripurigopi | rameshg87: okay, update_config makes sense, or its always delete and create in case of any changes? | 07:43 |
| *** Haomeng|2 is now known as Haomeng | 07:43 | |
| saripurigopi | rameshg87: thanks for the link. | 07:43 |
| rameshg87 | saripurigopi: update_config ? | 07:44 |
| rameshg87 | saripurigopi: brb | 07:45 |
| *** rameshg87 is now known as rameshg87-away | 07:45 | |
| saripurigopi | rameshg87: update_raid_config on the node, I'm not sure ,if that makes sense at all.. | 07:46 |
| saripurigopi | rameshg87: oksy | 07:46 |
| *** v92 has joined #openstack-ironic | 07:47 | |
| *** dmitchell__ has joined #openstack-ironic | 07:52 | |
| *** athomas has joined #openstack-ironic | 07:53 | |
| *** dmitchell has quit IRC | 07:55 | |
| openstackgerrit | Zhai, Edwin proposed openstack/ironic-specs: New meter for Node Manager https://review.openstack.org/176109 | 07:57 |
| *** rameshg87-away has quit IRC | 08:01 | |
| *** dmitchell has joined #openstack-ironic | 08:04 | |
| *** athomas has quit IRC | 08:05 | |
| *** lucasagomes has joined #openstack-ironic | 08:06 | |
| *** dmitchell__ has quit IRC | 08:07 | |
| dtantsur | lucasagomes, hey morning! what do you think about https://bugs.launchpad.net/ironic/+bug/1465153 ? | 08:07 |
| openstack | Launchpad bug 1465153 in Ironic "BMC failure can lead to undeletable nodes (and Nova instances)" [Medium,Confirmed] | 08:07 |
| *** yuanying has quit IRC | 08:09 | |
| lucasagomes | dtantsur, morning | 08:09 |
| * lucasagomes looks | 08:09 | |
| *** jcoufal has joined #openstack-ironic | 08:12 | |
| *** davideagnello has joined #openstack-ironic | 08:13 | |
| *** Nisha has joined #openstack-ironic | 08:16 | |
| *** davideagnello has quit IRC | 08:18 | |
| *** bethelwell has joined #openstack-ironic | 08:19 | |
| *** rameshg87 has joined #openstack-ironic | 08:27 | |
| *** bethelwe_ has joined #openstack-ironic | 08:29 | |
| naohirot | rameshg87: hello, I just have a technical question regarding standalone mode. | 08:32 |
| stendulker | lucasagomes, good morning! | 08:32 |
| stendulker | lucasagomes: Have received a comment from Yuriy for https://review.openstack.org/#/c/174785/14 wrt your comment mentioning not to use tolower() for get_node_capability() in review https://review.openstack.org/#/c/190900/ | 08:32 |
| *** bethelwell has quit IRC | 08:32 | |
| stendulker | lucasagomes: In review https://review.openstack.org/#/c/174785/14 capability values from instance_info and properties are changed to lower to check if they are one of the valid values for that capability | 08:34 |
| lucasagomes | stendulker, hi there | 08:35 |
| * lucasagomes looks | 08:35 | |
| stendulker | lucasagomes: I think it should be fair to change the capabilities value to lower in validation method as these values are not returned. | 08:35 |
| lucasagomes | stendulker, but is not case sensitive? | 08:35 |
| lucasagomes | if yes, MyCapability != mycapability | 08:36 |
| lucasagomes | and we should conisder both | 08:36 |
| lucasagomes | is nova case* | 08:36 |
| stendulker | lucasagomes: in validation method we just check that the given value for a ironic supported capability is a valid one. boot_mode=UEfi would be returned as a valid value | 08:37 |
| *** dguerri` is now known as dguerri | 08:37 | |
| lucasagomes | stendulker, right, but when we pass this capabilities to nova, we don't pass them as lower case right? | 08:38 |
| openstackgerrit | chenglch proposed openstack/ironic-specs: Add console log support https://review.openstack.org/164586 | 08:38 |
| * lucasagomes test it | 08:38 | |
| openstackgerrit | Merged openstack/ironic-inspector: Pass environment variables of proxy to tox https://review.openstack.org/191598 | 08:39 |
| *** athomas has joined #openstack-ironic | 08:39 | |
| stendulker | lucasagomes: Proposed deploy_utils.py:validate_capabilities() do not return any value. it throws exception is the given value is not a valid value. This function is case insensitive | 08:39 |
| stendulker | lucasagomes: https://review.openstack.org/#/c/174785/14/ironic/drivers/modules/deploy_utils.py L1089 | 08:40 |
| lucasagomes | stendulker, I get it... But I'm worried about Nova | 08:42 |
| lucasagomes | if someone creates in the extra spec "capabilities:boot_option=uefi" | 08:42 |
| lucasagomes | and in ironic "boot_option:uEFI" we will pass validation but the scheduler won't match the node | 08:42 |
| lucasagomes | (assuming nova is case sensitive here) | 08:42 |
| *** dguerri has quit IRC | 08:43 | |
| *** dguerri has joined #openstack-ironic | 08:43 | |
| stendulker | lucasagomes: Today, is nova case sensistive? | 08:44 |
| lucasagomes | stendulker, I don't know that's my doubt | 08:44 |
| lucasagomes | lemme test | 08:44 |
| rameshg87 | naohirot: hey | 08:44 |
| rameshg87 | naohirot: yeah please go ahead | 08:44 |
| naohirot | rameshg87: Hi, good afternoon | 08:45 |
| rameshg87 | naohirot: good afternoon | 08:45 |
| naohirot | rameshg87: My question is, can we run multiple conductors in standalone mode? | 08:46 |
| rameshg87 | naohirot: yes | 08:47 |
| rameshg87 | naohirot: we can | 08:47 |
| naohirot | rameshg87: I believe that the answer is no, because conductor doesn't know each other | 08:47 |
| openstackgerrit | SHIGEMATSU Mitsuhiro proposed openstack/ironic: Add boot interface in Ironic https://review.openstack.org/166512 | 08:47 |
| naohirot | rameshg87: really? how does conductor know each other? | 08:47 |
| rameshg87 | naohirot: from the db | 08:48 |
| rameshg87 | naohirot: there is a table called conductors to which every conductor registers when it is initialized | 08:48 |
| naohirot | rameshg87: Is it possible without nova controller? | 08:49 |
| rameshg87 | naohirot: what do you mean by nova controller ? | 08:49 |
| rameshg87 | naohirot: for ironic, nova is just a client which asks it to perform tasks one after the other and queries data from it. | 08:50 |
| naohirot | rameshg87: I just meant that it is not standalone mode. | 08:50 |
| rameshg87 | naohirot: other than that ironic has no dependency on nova. | 08:50 |
| lucasagomes | stendulker, http://paste.openstack.org/show/293425/ nova is case sensitive | 08:50 |
| rameshg87 | naohirot: ironic can perfectly work fine without nova. that's what projects like bifrost are counting on. | 08:50 |
| lucasagomes | stendulker, I think the best thing to do, when document this capabilities is to actually say it at the document | 08:51 |
| naohirot | rameshg87: who provides conductor information into the db? does user have invoke some Ironic CLI? | 08:52 |
| stendulker | lcasagomes: thanks for testing. But how do we validate the capabilities? | 08:52 |
| *** athomas has quit IRC | 08:52 | |
| rameshg87 | naohirot: when ironic-conductor is started it registers in the db | 08:53 |
| rameshg87 | naohirot: https://github.com/openstack/ironic/blob/master/ironic/conductor/manager.py#L232 | 08:53 |
| *** subscope has quit IRC | 08:54 | |
| naohirot | rameshg87: Mybe I asked same question long time ago, is the db some kind of P2P distributed hash table? | 08:54 |
| rameshg87 | naohirot: nope | 08:54 |
| rameshg87 | naohirot: db just has a list of conductor nodes | 08:54 |
| rameshg87 | naohirot: every conductor reads from the table and gets information about other conductor | 08:55 |
| rameshg87 | naohirot: every conductor applies the same hashing technique to arrive at which belongs to whom | 08:55 |
| openstackgerrit | SHIGEMATSU Mitsuhiro proposed openstack/ironic: Refactor node's and driver's vendor passthru to a common place https://review.openstack.org/190121 | 08:56 |
| openstackgerrit | SHIGEMATSU Mitsuhiro proposed openstack/ironic: Allow vendor methods to serve static files https://review.openstack.org/189716 | 08:56 |
| openstackgerrit | SHIGEMATSU Mitsuhiro proposed openstack/ironic: Change return value of [driver_]vendor_passthru to dict https://review.openstack.org/190120 | 08:56 |
| rameshg87 | naohirot: and ends up with the distribution of nodes | 08:56 |
| stendulker | lucasagomes: should validation passed node be always selected by the nova? As validation does more than lexical validation of the capability values. | 08:56 |
| rameshg87 | naohirot: same is done by the ironic-api as well to see which node is managed by which conductor | 08:56 |
| naohirot | rameshg87: does that db reside in erlang rabbit ? | 08:57 |
| rameshg87 | naohirot: no | 08:58 |
| lucasagomes | stendulker, the use of capabilities in Ironic is to allow advanced filtering by nova... Sure there's more to validate, but I don't see the advantage of being case insensitive in Ironic and case sensitive in Nova | 08:58 |
| naohirot | rameshg87: I'm wondering how multiple conductors share a db? | 08:59 |
| lucasagomes | it seems to bring more headache than help | 08:59 |
| rameshg87 | naohirot: currently we have dbapi written in sqlalchemy | 08:59 |
| lucasagomes | there's no advantage in passing validation with boot_option:UEFI if it's going to fail later | 08:59 |
| rameshg87 | naohirot: so any db which can be controlled by sqlalchemy can be used | 08:59 |
| lucasagomes | or we have both case insensitive or none | 08:59 |
| *** dmitchell__ has joined #openstack-ironic | 08:59 | |
| lucasagomes | both == nova and ironic | 08:59 |
| rameshg87 | naohirot: mysql is the most popular one I gues | 08:59 |
| dtantsur | harlowja, hey, around? it's re futurist | 09:00 |
| naohirot | rameshg87: But we support sqlite too right? | 09:00 |
| rameshg87 | naohirot: yes, sqlalchemy supports multiple dbs | 09:00 |
| rameshg87 | sqlalchemy support sqlite too | 09:00 |
| dtantsur | harlowja, wanna know, when you plan on releasing all this periodic stuff. we might start adopting it then :) | 09:00 |
| naohirot | rameshg87: In case of MySQL, it's client-server db, so multiple conductors can share a db. | 09:01 |
| naohirot | rameshg87: but sqlite is just a file. | 09:02 |
| lucasagomes | stendulker, or I'm missing something obvious here? | 09:02 |
| rameshg87 | naohirot: sqlite is just for testing purpose :) | 09:02 |
| *** dmitchell has quit IRC | 09:02 | |
| stendulker | lucasagomes: I understand. But in validate we compare if the nova were to select a node for any of the capabilities, then can Ironic deploy using the same. For that we are comparing the values against hardcoded supported values | 09:02 |
| rameshg87 | naohirot: or may be a single conductor thing | 09:03 |
| naohirot | rameshg87: I see | 09:03 |
| *** oomichi has quit IRC | 09:05 | |
| lucasagomes | stendulker, but the instance_info capabilities will only be present IF nova already pick that node to deploy right? | 09:05 |
| stendulker | lucasagomes: I think driver validate() method would get called after the node selection. | 09:05 |
| lucasagomes | that's nova that adds those capabilties to the node | 09:06 |
| stendulker | lucasagomes: yes. | 09:06 |
| naohirot | rameshg87: Is multiple conductors configuration just redundancy purpose? | 09:06 |
| stendulker | lucasagomes: in validate() we need not worry about the case sensitivity wrt nova. As that part is already completed by nova and node is selected. | 09:07 |
| lucasagomes | stendulker, validate in the nova driver is called before the node is deployed, so it's called after selection | 09:07 |
| stendulker | lucasagomes: At this stage, ironic validates f it can deploy the nova selected node. | 09:07 |
| lucasagomes | stendulker, right, and for that to happen the case was considered | 09:07 |
| stendulker | lucasagomes: yes | 09:07 |
| lucasagomes | otherwise it wouldn't match | 09:07 |
| lucasagomes | so meaning the case in instance_info == properties | 09:08 |
| lucasagomes | why we need to convert the case then? | 09:08 |
| stendulker | lucasagomes: So we are good to use tolower() to compare values in validate() | 09:08 |
| * lucasagomes review the patch | 09:08 | |
| stendulker | lucasagomes: We change the case to check if the value specified in the capabilities is a valid one. Ex: UEFI, Uefi are valid. UEFIS Uefis are invalid | 09:09 |
| *** bethelwe_ has quit IRC | 09:09 | |
| stendulker | lucasagomes: We do these validation by comparing the capabilities values with their correspondig range of valid values. | 09:10 |
| *** bethelwell has joined #openstack-ironic | 09:10 | |
| dtantsur | harlowja, and one more thing: any chances that Futurist will work with periodic tasks decorated by old oslo incubator decorators? | 09:10 |
| rameshg87 | naohirot: yes, and distribution and ha | 09:10 |
| openstackgerrit | Merged openstack/ironic: DRAC: fix set/get boot device for 11g https://review.openstack.org/191177 | 09:11 |
| lucasagomes | stendulker, but what about https://github.com/openstack/ironic/blob/d1b586cbf673af3c05257bd16339aadc0f61d7f4/ironic/drivers/modules/msftocs/management.py#L88-L89 | 09:11 |
| lucasagomes | ? | 09:11 |
| rameshg87 | naohirot: if one conductor goes down, it can be detected within a timeout and the hashring can be rebalanced | 09:11 |
| naohirot | rameshg87: Okay | 09:11 |
| rameshg87 | naohirot: so that the nodes belonging to the dead conductor can be picked up by other conductors after rebalance | 09:11 |
| lucasagomes | in the code we look for some values in lower case, if validate pass with UEFI it might not work... | 09:11 |
| lucasagomes | we don't lower it here https://github.com/openstack/ironic/blob/309707bab00a8a30d473815687515ac906ffbeb2/ironic/drivers/utils.py#L124-L145 | 09:12 |
| lucasagomes | stendulker, https://github.com/openstack/ironic/blob/309707bab00a8a30d473815687515ac906ffbeb2/ironic/common/pxe_utils.py#L206-L209 | 09:12 |
| lucasagomes | https://github.com/openstack/ironic/blob/309707bab00a8a30d473815687515ac906ffbeb2/ironic/common/pxe_utils.py#L273-L274 | 09:12 |
| lucasagomes | there's plenty == 'uefi' (lower case) | 09:12 |
| naohirot | rameshg87: which conductor manages the hashring? Is one of conductors become master? | 09:13 |
| lucasagomes | case matters even for Ironic... there for validation is it's lower case and not converting it makes sense | 09:13 |
| lucasagomes | therefore* | 09:13 |
| stendulker | lucasagomes: This means ironic need it in lowercase | 09:13 |
| rameshg87 | naohirot: there is no master concept | 09:13 |
| lucasagomes | stendulker, yes... so in validate if we lower the case | 09:13 |
| *** pelix has joined #openstack-ironic | 09:13 | |
| lucasagomes | it will fail later | 09:13 |
| lucasagomes | stendulker, unless we replace the value with the lower case value | 09:14 |
| stendulker | lucasagomes: But nova can match based on case, which may not work for ironic later | 09:14 |
| rameshg87 | naohirot: since the same hashring technique is applied by each conductor on all the same nodes, they will result in the same distribution | 09:14 |
| lucasagomes | yes, exactly... so validation if the user input it as lower case already | 09:14 |
| lucasagomes | is correct | 09:14 |
| rameshg87 | if there are 100 nodes | 09:14 |
| lucasagomes | we should consider the case | 09:14 |
| rameshg87 | and 4 conductor a,b,c,d | 09:14 |
| rameshg87 | 25 will go to a, 25 will go to b and so on | 09:14 |
| stendulker | lucasagomes: so check should be taht capabilities muct be in lower case only | 09:14 |
| stendulker | lucasagomes: else fail it. | 09:14 |
| lucasagomes | stendulker, yeah | 09:15 |
| *** romcheg has joined #openstack-ironic | 09:15 | |
| rameshg87 | the same 25 nodes will goto conductor a when applied on all 4 conductors a,b,c,d | 09:15 |
| naohirot | rameshg87: interesting | 09:15 |
| rameshg87 | because it is the same algorithm applied on same input set | 09:15 |
| rameshg87 | naohirot: yeah it is pretty interesting :) | 09:15 |
| naohirot | rameshg87: yeah :) | 09:15 |
| stendulker | lucasagomes: ok. Will add that check in validate to fail if the capability values are not in lower case. | 09:15 |
| rameshg87 | naohirot: https://github.com/openstack/ironic/blob/master/ironic/common/hash_ring.py | 09:15 |
| rameshg87 | naohirot: have a look at this | 09:16 |
| lucasagomes | stendulker, +1 thanks | 09:16 |
| stendulker | lucasagomes: Thank you for making me understand this. :) | 09:16 |
| lucasagomes | stendulker, it's all good :D | 09:16 |
| naohirot | rameshg87: I'll take a look in details later, one more question in case of Conductors with Nova, not standalone. | 09:18 |
| rameshg87 | naohirot: yeah | 09:18 |
| naohirot | rameshg87: can Nova support more than two set of conductors right now? | 09:18 |
| naohirot | rameshg87: one set of conductors share the same ring. | 09:19 |
| lintan | hi lucasagomes, I understand Nova is case sensitive, but what about User add 'boot_mode=BIOS' in flavor and node.properties? | 09:19 |
| lintan | hi lucasagomes, it make no sense to fail | 09:20 |
| naohirot | rameshg87: so my question is , does nova support more than two ironic groups, two ironic-rings? | 09:20 |
| lucasagomes | lintan, if in ironic we are expecting "bios" (lower case) and we have those == "bios" in the code | 09:20 |
| lucasagomes | it will fail | 09:20 |
| lucasagomes | will fail validation I mean | 09:20 |
| lucasagomes | lintan, because in the code we expect it to be case sensitive like the uefi one | 09:21 |
| lucasagomes | lintan, perhaps we could change it... but it's hard | 09:21 |
| lucasagomes | we have to think that capabilties is not only for boot_mode, boot_option etc that we pre determine | 09:21 |
| rameshg87 | naohirot: no | 09:21 |
| lucasagomes | it can be used for anything that the user wants | 09:22 |
| rameshg87 | naohirot: nova-compute running ironic virt driver can be configured to talk to only one ironic-api | 09:22 |
| naohirot | rameshg87: I see, I got it.:) | 09:22 |
| lucasagomes | lintan, that's why it's hard... perhaps the best thing to do IMHO is documenting that the case matters | 09:22 |
| lucasagomes | and we use lower case | 09:23 |
| lucasagomes | at least for the pre determined capabilites such as boot_mode, boot_option etc | 09:23 |
| *** gabriel-bezerra has quit IRC | 09:23 | |
| *** afaranha has quit IRC | 09:24 | |
| naohirot | rameshg87: thanks! | 09:24 |
| lintan | lucasagomes, hmmm, but it's something wired | 09:24 |
| lucasagomes | lintan, yeah, I get that... but we need to fix the rest of the code if we want to fix validation | 09:24 |
| lucasagomes | otherwise we are introducing more problems | 09:24 |
| lucasagomes | passing validation if boot_mode=UEFI... won't fix the code looking for == 'uefi' | 09:25 |
| *** athomas has joined #openstack-ironic | 09:25 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-specs: Switch periodic tasks to the Futurist library https://review.openstack.org/191710 | 09:25 |
| lintan | lucasagomes, I didn't fix validation actually, I fix get_node_capability which will work for things you post above | 09:25 |
| lucasagomes | lintan, yeah but that's a generic method | 09:26 |
| *** mgoddard has joined #openstack-ironic | 09:26 | |
| lucasagomes | say a user adds properties/capabilities=test:TEST | 09:26 |
| lucasagomes | and in nova it's capabilities/test=TEST | 09:26 |
| lucasagomes | if get_node_capability convert it to lower case | 09:27 |
| lucasagomes | we will pass "test:test" to the nova scheduler from ironic | 09:27 |
| lucasagomes | and it will fail to match the test=TEST in the nova flavor (because nova considers the case) | 09:27 |
| lintan | lucasagomes, wait a minute, I disagree on that, I think nova will read node.properties directly | 09:27 |
| * lucasagomes looks | 09:28 | |
| lucasagomes | lintan, yeah it will https://github.com/openstack/nova/blob/master/nova/virt/ironic/driver.py#L256 | 09:29 |
| lucasagomes | hmm | 09:29 |
| lintan | lucasagomes, yes, so this can solve your concern | 09:29 |
| lucasagomes | lintan, yes | 09:30 |
| lucasagomes | but it all sounds like black magic | 09:30 |
| lucasagomes | I mean in nova test != TEST, in ironic not | 09:30 |
| *** e0ne has joined #openstack-ironic | 09:30 | |
| *** Nisha_away has joined #openstack-ironic | 09:31 | |
| lucasagomes | lintan, maybe it's me, I just find it odd on having case sensitive in one edge but not in the other | 09:31 |
| *** Nisha has quit IRC | 09:31 | |
| * lucasagomes would prefer to have everything case sensitive (nova and ironic) or everything case insensitive | 09:32 | |
| rameshg87 | lucasagomes: hi | 09:33 |
| lucasagomes | rameshg87, hi there | 09:33 |
| rameshg87 | lucasagomes: recently there was a mail on suds dependency of ironic | 09:33 |
| rameshg87 | lucasagomes: something in wsme seems to use it | 09:33 |
| rameshg87 | lucasagomes: https://github.com/stackforge/wsme/blob/master/wsmeext/tests/test_soap.py | 09:33 |
| rameshg87 | lucasagomes: not sure if it was mentioned because of that. anything to do with it ? | 09:33 |
| *** foexle has joined #openstack-ironic | 09:34 | |
| rameshg87 | lucasagomes: but requirements.txt doesn't mention though | 09:34 |
| lucasagomes | rameshg87, oh, and wsme doesn't mention that? | 09:34 |
| lucasagomes | :-( | 09:34 |
| rameshg87 | so sort of sure if it isn't the code actually using it | 09:34 |
| rameshg87 | and grep confirms it. code doesn't use suds | 09:34 |
| rameshg87 | it's only the test | 09:34 |
| lucasagomes | I see | 09:35 |
| * rameshg87 might be wrong | 09:35 | |
| lucasagomes | rameshg87, should be in the test-requirements of wsme then | 09:35 |
| sambetts | Morning all :) | 09:35 |
| rameshg87 | lucasagomes: there is no such file :) | 09:35 |
| rameshg87 | sambetts: o/ | 09:35 |
| lucasagomes | sambetts, morning | 09:35 |
| yuikotakada | sambetts: o/ | 09:35 |
| lucasagomes | rameshg87, oh dear wsme *sigh* | 09:35 |
| sambetts | o/ lucasagomes yuikotakada rameshg87 | 09:35 |
| rameshg87 | lucasagomes: wsme was written for openstack, right ? | 09:37 |
| rameshg87 | lucasagomes: anythign in openstack uses soap ? | 09:37 |
| lucasagomes | rameshg87, the email was saying suds is a dependency in Ironic? | 09:37 |
| rameshg87 | yeah | 09:38 |
| lucasagomes | I don't see how it would affect Ironic if it's only a dependency for wsme tests | 09:38 |
| lucasagomes | rameshg87, afaik, no | 09:38 |
| * rameshg87 digs for email | 09:38 | |
| lucasagomes | rameshg87, I mean don't use soap | 09:38 |
| lucasagomes | rameshg87, I'm not sure if wsme was written for openstack... I believe not | 09:38 |
| lucasagomes | rameshg87, people at one point pushed wsme to be the default way to create APIs in Ironic, but it's no longer a valid thing | 09:39 |
| lucasagomes | (that's the reason why Ironic uses WSME I believe) | 09:39 |
| rameshg87 | oh okay | 09:39 |
| rameshg87 | lucasagomes: http://lists.openstack.org/pipermail/openstack-dev/2015-June/066561.html | 09:39 |
| rameshg87 | lucasagomes: I thought it was because wsme dependency on suds, ironic was listed | 09:40 |
| sambetts | lucasagomes: wsme lives in stackforge, and they link to OpenStack https://github.com/stackforge/wsme/ | 09:40 |
| *** e0ne has quit IRC | 09:40 | |
| lucasagomes | rameshg87, I see, yeah it might be | 09:41 |
| rameshg87 | but zigo says it has been removed - http://lists.openstack.org/pipermail/openstack-dev/2015-June/066864.html | 09:41 |
| lucasagomes | rameshg87, in any case, it would be good to remove that dependency on WSME itself | 09:41 |
| lucasagomes | since it's going to be deprecated in debian and probably other distros | 09:41 |
| * lucasagomes opens a bug in wsme for it | 09:41 | |
| rameshg87 | yeah | 09:41 |
| *** rbudden_ has quit IRC | 09:42 | |
| lucasagomes | sambetts, yeah wsme uses the openstack infra for reviews etc | 09:42 |
| *** rbudden has joined #openstack-ironic | 09:42 | |
| lintan | lucasagomes, hmmm, but case insensitive it's more friendly to users and save developer's word. I will propose a patch to nova to see their feedback. | 09:42 |
| lucasagomes | sambetts, but I'm not sure it was like this since the begging | 09:42 |
| lucasagomes | lintan, right, +1 | 09:43 |
| *** Nisha_away has quit IRC | 09:43 | |
| lucasagomes | lintan, but you agree that have case insensitive in only one part is more confusing? | 09:43 |
| lucasagomes | maybe I'm wrong... but that's how it feels to me | 09:43 |
| sambetts | lucasagomes: ah, that makes sense, yeah projects like that I'm don't know where they belong, I guess if it was really part of OS it would be an oslo thing | 09:43 |
| lucasagomes | sambetts, yeah... Tho I'm glad it's not part of oslo | 09:44 |
| lucasagomes | heh | 09:44 |
| lintan | lucasagomes, yes, kind of | 09:44 |
| *** e0ne has joined #openstack-ironic | 09:47 | |
| *** f13o has joined #openstack-ironic | 09:59 | |
| openstackgerrit | Shivanand Tendulker proposed openstack/ironic: Validate capability in properties and instance_info https://review.openstack.org/174785 | 10:02 |
| *** zhenguo has quit IRC | 10:02 | |
| *** davideagnello has joined #openstack-ironic | 10:02 | |
| openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: Add support for certificate based authentication https://review.openstack.org/191722 | 10:02 |
| dtantsur | yuikotakada, I remember you had a patch to enable i18n for discoverd, which was rejected due to "we don't support stackforge", right? Could you revive it now? | 10:03 |
| yuikotakada | dtantsur, yeah, right. it was https://review.openstack.org/#/c/160610/ | 10:04 |
| *** davideagnello has quit IRC | 10:07 | |
| *** yuikotakada is now known as yuikot_dinner | 10:13 | |
| dtantsur | lucasagomes, please lemme know what you think about https://bugs.launchpad.net/ironic/+bug/1465153 | 10:13 |
| openstack | Launchpad bug 1465153 in Ironic "BMC failure can lead to undeletable nodes (and Nova instances)" [Medium,Confirmed] | 10:13 |
| dtantsur | I'm inclined to allow much more stuff in maintenance mode due to this | 10:14 |
| dtantsur | devananda, if you're around by change, please also weigh in ^^ | 10:14 |
| *** yuikot_dinner has quit IRC | 10:14 | |
| *** marzif_ has joined #openstack-ironic | 10:18 | |
| dtantsur | ifarkas, hi! could you please review https://review.openstack.org/#/c/190643 ? | 10:18 |
| lucasagomes | dtantsur, will do, yeah I was looking at it but I don't know the reason yet | 10:19 |
| lucasagomes | why it can't be deleted in nova, perhaps we are not considering the ERROR state | 10:19 |
| lucasagomes | I will check in a min | 10:19 |
| dtantsur | lucasagomes, I suspect that node-update is forbidden, but I didn't check this guess | 10:20 |
| dtantsur | lucasagomes, anyway, being able to delete a completely broken node sounds like a good feature... | 10:21 |
| lucasagomes | +1 | 10:21 |
| dtantsur | lucasagomes, https://github.com/openstack/ironic/blob/master/ironic/api/controllers/v1/node.py#L1012-L1017 | 10:22 |
| *** ukalifon has quit IRC | 10:23 | |
| lucasagomes | dtantsur, x.x | 10:24 |
| *** dlpartain has joined #openstack-ironic | 10:25 | |
| *** ukalifon1 has joined #openstack-ironic | 10:26 | |
| openstackgerrit | Merged openstack/ironic: Pass environment variables of proxy to tox https://review.openstack.org/191599 | 10:26 |
| *** ukalifon1 has quit IRC | 10:26 | |
| *** dlpartain has left #openstack-ironic | 10:27 | |
| *** e0ne is now known as e0ne_ | 10:32 | |
| *** ukalifon1 has joined #openstack-ironic | 10:36 | |
| *** e0ne_ has quit IRC | 10:39 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic: Allow deleting even associated and active node in maintenance mode https://review.openstack.org/191736 | 10:40 |
| dtantsur | lucasagomes, first step ^^ | 10:40 |
| *** marzif_ has quit IRC | 10:41 | |
| *** marzif_ has joined #openstack-ironic | 10:41 | |
| *** e0ne has joined #openstack-ironic | 10:41 | |
| openstackgerrit | Ramakrishnan G proposed stackforge/proliantutils: Add support for certificate based authentication https://review.openstack.org/191722 | 10:47 |
| openstackgerrit | Matt Keenan proposed openstack/ironic: get_supported_boot_devices() returns static device list https://review.openstack.org/188466 | 10:48 |
| rameshg87 | saripurigopi: the other issue is related to logging format string | 10:48 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Vagrantfile and vagrant.yml for testing NUC https://review.openstack.org/191110 | 10:48 |
| rameshg87 | saripurigopi: are you using osx ? | 10:48 |
| * rameshg87 wonders if it makes some difference | 10:49 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Fix Cisco UCS slow tests https://review.openstack.org/191745 | 10:49 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Fix Cisco UCS slow tests https://review.openstack.org/191745 | 10:51 |
| saripurigopi | rameshg87: I'm trying this on ubuntu 14.10. | 10:52 |
| rameshg87 | saripurigopi: okay | 10:52 |
| saripurigopi | rameshg87: accessing it from OSX. | 10:52 |
| rameshg87 | okay | 10:52 |
| lucasagomes | saripurigopi, I tried on 14.04 as the bug mentions and it was broken... Lemme try on other versions see if I can spot something | 10:53 |
| saripurigopi | lucasagomes: have you tried with 0.8.1.7? It is trying to install 0.8.1.6 and that is broken. | 10:54 |
| openstackgerrit | Anusha Ramineni proposed openstack/ironic: Add option to configure passes in erase_devices https://review.openstack.org/191750 | 10:54 |
| lucasagomes | saripurigopi, yeah 0.8.1.7 installs | 10:55 |
| lucasagomes | but once I try to run the Ironic conductor I get that error about the YesOrNo class not being defined | 10:55 |
| openstackgerrit | Anusha Ramineni proposed openstack/ironic-python-agent: Add option to configure passes in erase_devices https://review.openstack.org/191751 | 10:56 |
| saripurigopi | lucasagomes: I've just tried again and pasted the result, tried on ubuntu 14.10. | 10:56 |
| lucasagomes | DriverLoadError: Driver fake could not be loaded. Reason: name 'YesOrNo' is not defined. | 10:57 |
| *** afaranha has joined #openstack-ironic | 10:58 | |
| lucasagomes | saripurigopi, also, http://paste.openstack.org/show/293785/ | 10:59 |
| saripurigopi | lucasagomes: Saw the error in the review log. | 10:59 |
| saripurigopi | lucasagomes: http://paste.openstack.org/show/293786/ | 11:00 |
| saripurigopi | lucasagomes: which environment are you trying this? | 11:00 |
| ifarkas | dtantsur, hi, sure, I will review it | 11:02 |
| *** ramineni has quit IRC | 11:02 | |
| lucasagomes | saripurigopi, on ubuntu 14.04 that we run in the gate | 11:02 |
| lucasagomes | lemme try on fedora too, 1 sec lemme see if I have one env handy | 11:03 |
| * rameshg87 leaves for the day | 11:07 | |
| *** rameshg87 has quit IRC | 11:07 | |
| saripurigopi | lucasagomes: sure | 11:07 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Adding agent_ucs driver to the driver list https://review.openstack.org/191219 | 11:08 |
| saripurigopi | lucasagomes: tried on ubuntu14.04, its failing. | 11:08 |
| lucasagomes | saripurigopi, :-( | 11:08 |
| saripurigopi | lucasagomes: ;-( | 11:09 |
| lucasagomes | 1 sec I will try on fedora | 11:09 |
| saripurigopi | sure | 11:09 |
| lucasagomes | >>> UcsSdk.__version__ | 11:12 |
| lucasagomes | Traceback (most recent call last): | 11:12 |
| lucasagomes | File "<stdin>", line 1, in <module> | 11:12 |
| lucasagomes | AttributeError: 'module' object has no attribute '__version__' | 11:12 |
| lucasagomes | saripurigopi, building a devstack env to see when loading... | 11:12 |
| *** stendulker has quit IRC | 11:14 | |
| saripurigopi | lucasagomes: sure, its same pip version on both ubuntu 14.10 and 14.04. | 11:14 |
| *** dtantsur is now known as dtantsur|brb | 11:14 | |
| openstackgerrit | Ramakrishnan G proposed openstack/ironic: Add boot interface in Ironic https://review.openstack.org/166512 | 11:14 |
| *** bcornec has quit IRC | 11:20 | |
| *** bcornec has joined #openstack-ironic | 11:21 | |
| *** marzif_ has quit IRC | 11:24 | |
| *** marzif_ has joined #openstack-ironic | 11:27 | |
| *** gabriel-bezerra has joined #openstack-ironic | 11:27 | |
| saripurigopi | lucasagomes: figured out the problem. __init__.py is missing the UcsSdk, in ubuntu14.04. | 11:29 |
| lucasagomes | saripurigopi, oh... how come? odd | 11:31 |
| lucasagomes | saripurigopi, are you guys releasing a new version fixing the problem? | 11:31 |
| saripurigopi | lucasagomes: yeah, there is newer version, trying to upload to PyPi. | 11:33 |
| *** e0ne is now known as e0ne_ | 11:36 | |
| lucasagomes | saripurigopi, ack, thanks for looking into it | 11:38 |
| *** e0ne_ is now known as e0ne | 11:39 | |
| *** lucasagomes is now known as lucas-hungry | 11:39 | |
| * lucas-hungry brb will get some lunch | 11:39 | |
| openstackgerrit | Merged openstack/ironic: Remove tools/flakes.py https://review.openstack.org/191061 | 11:43 |
| *** pradipta has quit IRC | 11:50 | |
| *** davideagnello has joined #openstack-ironic | 11:51 | |
| *** davideagnello has quit IRC | 11:56 | |
| *** degorenko has joined #openstack-ironic | 12:03 | |
| *** trown|outttypeww is now known as trown | 12:12 | |
| *** bradjones has quit IRC | 12:13 | |
| *** dtantsur|brb is now known as dtantsur | 12:13 | |
| *** bradjones has joined #openstack-ironic | 12:15 | |
| *** bradjones has quit IRC | 12:15 | |
| *** bradjones has joined #openstack-ironic | 12:15 | |
| *** chlong has joined #openstack-ironic | 12:20 | |
| *** e0ne is now known as e0ne_ | 12:28 | |
| *** e0ne_ is now known as e0ne | 12:28 | |
| openstackgerrit | Devananda van der Veen proposed openstack/ironic-specs: iPXE to use Swift Temporary URLs https://review.openstack.org/181920 | 12:30 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: WIP: Initial changes for testing VM creation for CentOS https://review.openstack.org/191504 | 12:31 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Support RedHat yum installation https://review.openstack.org/185294 | 12:31 |
| *** marzif_ has quit IRC | 12:33 | |
| *** lucas-hungry is now known as lucasagomes | 12:38 | |
| lucasagomes | devananda, thanks for fixing the spec | 12:39 |
| lucasagomes | and good ugt morning TheJulia devananda | 12:40 |
| TheJulia | goodmorning lucasagomes | 12:40 |
| lucasagomes | saripurigopi, FYI, same problem on F21... Ironic conductor won't start if UcsSdk == 0.8.1.7 is installed | 12:41 |
| *** jcoufal has quit IRC | 12:47 | |
| TheJulia | NobodyCam had a lead on what the issue was, but I tried putting what he thought might fix it in locally and it didn't work. :( | 12:49 |
| lucasagomes | TheJulia, apparently, __init__.py is missing | 12:49 |
| lucasagomes | TheJulia, saripurigopi is looking into it | 12:49 |
| *** kan_ has quit IRC | 12:51 | |
| TheJulia | that could do it quite nicely | 12:51 |
| lucasagomes | TheJulia, https://raw.githubusercontent.com/CiscoUcs/UcsPythonSDK/master/src/UcsSdk/__init__.py | 12:52 |
| lucasagomes | if you copy that file onto /usr/lib/python2.7/site-packages/UcsSdk/__init__.py | 12:52 |
| lucasagomes | it works | 12:52 |
| lucasagomes | NobodyCam, FYI ^ | 12:52 |
| TheJulia | awesome | 12:53 |
| lucasagomes | TheJulia, heh more or less.. :-P I mean, it's broken on pypi | 12:53 |
| lucasagomes | a new version is needed anyway | 12:53 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Vagrantfile and vagrant.yml for testing https://review.openstack.org/191110 | 12:53 |
| TheJulia | yeah | 12:54 |
| * lucasagomes should update the bug with the workaround | 12:54 | |
| *** f13o has quit IRC | 12:54 | |
| *** jcoufal has joined #openstack-ironic | 12:55 | |
| *** smoriya has quit IRC | 12:56 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: WIP: Initial changes for testing VM creation for CentOS https://review.openstack.org/191504 | 12:56 |
| *** jjohnson2 has joined #openstack-ironic | 13:01 | |
| *** jcoufal has quit IRC | 13:03 | |
| saripurigopi | lucasagomes: okay | 13:05 |
| *** Marga_ has joined #openstack-ironic | 13:08 | |
| *** MattMan has quit IRC | 13:08 | |
| *** MattMan has joined #openstack-ironic | 13:09 | |
| *** rloo has joined #openstack-ironic | 13:10 | |
| *** Marga_ has quit IRC | 13:11 | |
| *** Marga_ has joined #openstack-ironic | 13:11 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: WIP: Initial changes for testing VM creation for CentOS https://review.openstack.org/191504 | 13:13 |
| *** yuikotakada has joined #openstack-ironic | 13:15 | |
| *** cdearborn has joined #openstack-ironic | 13:15 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: WIP: Initial changes for testing VM creation for CentOS https://review.openstack.org/191504 | 13:16 |
| *** jcoufal has joined #openstack-ironic | 13:20 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Move create_ports to NodeInfo https://review.openstack.org/190643 | 13:21 |
| *** dlpartain has joined #openstack-ironic | 13:33 | |
| *** dlpartain has left #openstack-ironic | 13:33 | |
| openstackgerrit | Dmitry Tantsur proposed openstack/ironic-inspector: Final discoverd -> inspector rename/move pass https://review.openstack.org/191796 | 13:35 |
| trown | dtantsur: congrats on openstack/ironic-inspector | 13:36 |
| dtantsur | trown, \o/ | 13:37 |
| zigo | lucasagomes: Our plan is to remove suds from Stretch, instead you can use pysimplesoap which is actively maintained. | 13:37 |
| zigo | Thanks for the WSME thing. | 13:37 |
| lucasagomes | zigo I opened a bug in WSME to talk about pysimplesoap | 13:38 |
| lucasagomes | if nobody fix it soon I will put a patch up | 13:38 |
| zigo | Cool. | 13:38 |
| lucasagomes | problem is that WSME has very few cores, so if I submit a patch I can't +2 it | 13:38 |
| lucasagomes | which kinda sucks so I prefer other people fixing stuff | 13:38 |
| zigo | lucasagomes: Make sure to add me on the reivew, so that I can backport the patch to WSME for Kilo. | 13:38 |
| lucasagomes | zigo, ack | 13:38 |
| *** davideagnello has joined #openstack-ironic | 13:40 | |
| openstackgerrit | Roman Podoliaka proposed openstack/ironic: WIP: use new enginefacade feature of oslo.db https://review.openstack.org/191801 | 13:43 |
| yuikotakada | dtantsur: thank you for 191796. Is old pypi's URL(https://pypi.python.org/pypi/ironic-discoverd) necessary to be written in somewhere? for people using < 2.0.0 ? WDYT? | 13:44 |
| dtantsur | yuikotakada, I think we should stop people from using it asap :) | 13:44 |
| *** davideagnello has quit IRC | 13:45 | |
| jroll | morning all :) | 13:45 |
| jroll | I have a couple of specs I'd like eyes on: https://review.openstack.org/#/c/187829/ and https://review.openstack.org/#/c/185171/ | 13:45 |
| dtantsur | yuikotakada, oh btw, while you're here: did you do something about Tempest test for inspection? | 13:45 |
| *** zhenguo has joined #openstack-ironic | 13:45 | |
| dtantsur | jroll, morning! | 13:45 |
| yuikotakada | dtantsur: ok ;) | 13:46 |
| jroll | would be great to get some reviews before the meetings this morning | 13:46 |
| jroll | heya dtantsur! and yuikotakada and lucasagomes and trown and TheJulia :) | 13:46 |
| lucasagomes | jroll, yo, morning | 13:46 |
| trown | morning jroll | 13:46 |
| dtantsur | trown, morning! | 13:47 |
| lucasagomes | trown, morning | 13:47 |
| dtantsur | yuikotakada, I think we can start with something really simple just calling node-set-provision-state inspect, and then experiment with Rally/whatever | 13:47 |
| dtantsur | wdyt? | 13:47 |
| yuikotakada | jroll, trown, lucasagomes: g'morning | 13:49 |
| lucasagomes | yuikotakada, hello there, good ugt morning :-) | 13:49 |
| trown | good ugt morning yuikotakada | 13:49 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: WIP: Initial changes for testing VM creation for CentOS https://review.openstack.org/191504 | 13:50 |
| lucasagomes | jroll, the decouple from global requirements sounds a bit too much no? I'm not sure how this will work out with the current infra | 13:50 |
| lucasagomes | is that really needed? | 13:50 |
| yuikotakada | dtantsur: If we remain L405 and L429 in README.rst, we should write old pypi's URL, also. But if we remove L405 and L429, we don't need to write old pypi's URL. I think we need to keep consistency. WDYT? | 13:50 |
| dtantsur | yuikotakada, that's for existing users, which presumably already installed discoverd, and now have to use the new docs. I don't see much point in providing them a link to the old pypi | 13:51 |
| yuikotakada | dtantsur: do you mean that old pypi is only for old users? Hmm, that's right | 13:52 |
| dtantsur | yep | 13:52 |
| jroll | lucasagomes: maybe the wording is bad. my goal is to continue to follow global-reqs. however during the time which 1) global-reqs is frozen, *and* 2) we've already cut our stable release branch for the cycle, I think master should allow requirements updates if and only if there is a corresponding global-reqs patch blocked only by the freeze. does that make sense? | 13:53 |
| dtantsur | and the only reason I didn't delete it, is that ironic-discoverd is still in global-requirements (and maybe similar downstream scripts) | 13:53 |
| dtantsur | jroll, makes sense, but I don't imagine how to implement it... | 13:53 |
| yuikotakada | dtantsur, OK, let's merge it. Except for old pypi's URL, it seems np :) | 13:53 |
| dtantsur | :) | 13:54 |
| dtantsur | yuikotakada, so, any updates on tempest? | 13:54 |
| jroll | dtantsur: it's really just rules around how requirements changes are merged, idk | 13:54 |
| lucasagomes | jroll, right yeah it does... so eventually the global-req is going to get updated too right? | 13:54 |
| lucasagomes | jroll, I wonder if the patch actually doesn't make it to global-req | 13:54 |
| lucasagomes | so what we do with Ironic? | 13:54 |
| *** amotoki has quit IRC | 13:54 | |
| jroll | lucasagomes: right, so I think we would need to have g-r reviewers be like "LGTM but -2 until freeze is over" | 13:55 |
| yuikotakada | dtantsur, You mean Ironic+Ironic-inspector integration test? | 13:55 |
| dtantsur | yep | 13:55 |
| lucasagomes | jroll, yeah something like that | 13:55 |
| lucasagomes | kinda of a contract | 13:55 |
| lucasagomes | so we don't diverge | 13:55 |
| jroll | lucasagomes: yeah, need to improve the wording I guess | 13:56 |
| yuikotakada | dtantsur, as you know, we cannot add tests into tempest, so that tempest-lib looks good. | 13:56 |
| *** absubram has joined #openstack-ironic | 13:56 | |
| dtantsur | yuikotakada, I've seen new Ironic tests added to tempest ~today, so I think we should continue proposing stuff against Tempest, until told to stop | 13:57 |
| *** ukalifon2 has joined #openstack-ironic | 13:57 | |
| *** ukalifon1 has quit IRC | 13:57 | |
| lucasagomes | jroll, yeah... it sounds like we are maintinaing the requirements by our own in the spec | 13:57 |
| dtantsur | yuikotakada, otherwise we get in weird situation, because Ironic is still in tempest | 13:57 |
| yuikotakada | dtantsur: But, I'm just investigating, so I cannot say cleary, but tempest-lib looks not perfect yet. | 13:57 |
| jroll | lucasagomes: yeah, I'll fix that up | 13:57 |
| yuikotakada | dtantsur: Ironic's integration tests will be moved into Ironc's own tree | 13:58 |
| dtantsur | yuikotakada, let's start with adding a small modification to existing Ironic scenario: call node_set_provision_state(uuid, 'inspect') | 13:58 |
| yuikotakada | but I don't know when it will be started | 13:58 |
| dtantsur | yuikotakada, maybe, one day. do you have some time estimates? I don't :) | 13:58 |
| *** ukalifon3 has joined #openstack-ironic | 13:58 | |
| dtantsur | right, we don't know :) | 13:59 |
| openstackgerrit | Zhenguo Niu proposed openstack/ironic: Add disk layout check on re-provisioning https://review.openstack.org/180014 | 13:59 |
| *** athomas has quit IRC | 13:59 | |
| dtantsur | yuikotakada, I just don't see it happening right now, so I assume it might take a lot of time | 13:59 |
| dtantsur | let's move forward in the meanwhile | 13:59 |
| *** mgoddard1 has joined #openstack-ironic | 14:00 | |
| *** ukalifon4 has joined #openstack-ironic | 14:00 | |
| *** mgoddard has quit IRC | 14:01 | |
| yuikotakada | dtantsur, "let's start with adding a small modification to existing Ironic scenario" means adding some codes to Ironic scenario tests in tempest? I'm worrying it will be -2ed | 14:01 |
| *** ukalifon2 has quit IRC | 14:02 | |
| *** saripurigopi has quit IRC | 14:02 | |
| dtantsur | yuikotakada, I can't tell for sure. Unfortunately my ML post didn't get enough attention... | 14:03 |
| jroll | bbiab | 14:03 |
| *** ukalifon3 has quit IRC | 14:03 | |
| *** bethelwe_ has joined #openstack-ironic | 14:03 | |
| dtantsur | we can ask on #openstack-qa... | 14:03 |
| *** athomas has joined #openstack-ironic | 14:04 | |
| yuikotakada | dtantsur: I agree. I'll post again tomorrow(in Japan... so that about 10h later ;) | 14:04 |
| yuikotakada | It will be better discussing in public place | 14:05 |
| dtantsur | yuikotakada, I'm also asking now on :) | 14:05 |
| *** bethelwell has quit IRC | 14:06 | |
| *** bethelwe_ has quit IRC | 14:08 | |
| *** bethelwell has joined #openstack-ironic | 14:09 | |
| dtantsur | yuikotakada, <mtreinish> dtantsur: so this cycle that's the direction we're moving, so hopefully by the release all the ironic tests will be in the ironic repo (or an ironic owned repo) | 14:15 |
| dtantsur | yuikotakada, <mtreinish> dtantsur: so I think for the super short term pushing them to tempest is ok | 14:15 |
| dtantsur | yuikotakada, <mtreinish> but for ironic I htink since we already have things in tree we shouldn't block new tests for right now | 14:15 |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector: Add missing backslash and run genconfig https://review.openstack.org/191818 | 14:15 |
| yuikotakada | dtantsur, are you in #openstack-qa? | 14:16 |
| dtantsur | yuikotakada, yep | 14:16 |
| sambetts | dtantsur: ^ checkout that patch it can probably be rushed throuh | 14:16 |
| dtantsur | ack | 14:16 |
| yuikotakada | dtantsur, oh, can we add into tempest now? nice :) | 14:16 |
| dtantsur | yes, if we do it quickly :) | 14:16 |
| dtantsur | they're still working on related specs | 14:17 |
| yuikotakada | dtantsur, ok, I will start tomorrow :) I will go to bed... it's 11pm | 14:18 |
| openstackgerrit | Thiago Paiva Brito proposed openstack/ironic: OneView Driver for Ironic https://review.openstack.org/191822 | 14:18 |
| dtantsur | yuikotakada, sure, g'night :) | 14:18 |
| jroll | this has 2x +2, anyone else want to take a look? https://review.openstack.org/#/c/181920/ | 14:18 |
| jroll | ipxe/swift tempurl thing | 14:19 |
| yuikotakada | dtantsur: I'll work one more minute... | 14:19 |
| yuikotakada | I'm in qa, too. | 14:19 |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector: Provide hook into process when a node isn't found https://review.openstack.org/190124 | 14:20 |
| NobodyCam | good morning Ironicers | 14:21 |
| sambetts | Morning NobodyCam | 14:21 |
| *** cdearborn has quit IRC | 14:21 | |
| yuikotakada | dtantsur: I don't understand..."(mtreinish) so we don't need to worry about migrating ironic, cielo, heat, etc"??? | 14:22 |
| jroll | morning NobodyCam, sambetts | 14:22 |
| zhenguo | morning all | 14:22 |
| NobodyCam | morning sambetts jroll dtantsur yuikotakada zhenguo | 14:22 |
| yuikotakada | NobodyCam, zhenguo : g'morning | 14:22 |
| NobodyCam | :) | 14:22 |
| zhenguo | :) | 14:22 |
| NobodyCam | on rv parks wifi and its super laggy | 14:23 |
| dtantsur | yuikotakada, don't mind for now, it's about tempest-lib, which we probably won't care about (to be decided) | 14:23 |
| dtantsur | NobodyCam, morning! | 14:23 |
| *** openstackgerrit has quit IRC | 14:24 | |
| yuikotakada | dtantsur, ok. thanks :) | 14:24 |
| *** bnemec has joined #openstack-ironic | 14:24 | |
| *** openstackgerrit has joined #openstack-ironic | 14:24 | |
| *** itamarl has quit IRC | 14:25 | |
| *** achanda has joined #openstack-ironic | 14:27 | |
| NobodyCam | :) | 14:28 |
| BadCub | Mornign folks | 14:30 |
| yuikotakada | BadCub, good morning | 14:31 |
| sambetts | Morning BadCub | 14:31 |
| NobodyCam | mornign BadCub :) | 14:32 |
| lucasagomes | NobodyCam, BadCub good ugt morning | 14:32 |
| BadCub | Morning yuikotakada sambetts NobodyCam :) | 14:32 |
| NobodyCam | morning (UGT)lucasagomes | 14:32 |
| openstackgerrit | Merged openstack/ironic-inspector: Add missing backslash and run genconfig https://review.openstack.org/191818 | 14:32 |
| * NobodyCam keep wanting to write that at good (UGT) morning :-p | 14:33 | |
| trown | dtantsur: sorry about that ^...just noticed it in RDO, where I also missed lots of "inspector"s | 14:34 |
| dtantsur | np) | 14:34 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: WIP: Initial changes for testing VM creation for CentOS https://review.openstack.org/191504 | 14:34 |
| TheJulia | good morning NobodyCam | 14:35 |
| NobodyCam | mornign TheJulia :) | 14:35 |
| * NobodyCam needs coffee | 14:36 | |
| *** coolsvap is now known as coolsvap|afk | 14:37 | |
| BadCub | Morning lucasagomes TheJulia :) | 14:37 |
| *** zz_natorious is now known as natorious | 14:40 | |
| *** achanda has quit IRC | 14:42 | |
| *** absubram has quit IRC | 14:45 | |
| lucasagomes | jroll, re network providers... so each port has a driver? | 14:46 |
| *** yuikotakada has quit IRC | 14:46 | |
| lucasagomes | I mean, so that each port can be managed by a different service? Some by neutron others by idk what | 14:46 |
| openstackgerrit | Merged openstack/ironic-inspector: Final discoverd -> inspector rename/move pass https://review.openstack.org/191796 | 14:49 |
| openstackgerrit | Shilla Saebi proposed openstack/ironic: copy editing of ironic deploy docs https://review.openstack.org/191838 | 14:49 |
| *** ukalifon4 has quit IRC | 14:50 | |
| jroll | lucasagomes: yeah, I debated whether that should be node level or port level | 14:50 |
| *** lazy_prince has quit IRC | 14:51 | |
| jroll | lucasagomes: but I don't think there's any downside to port level | 14:51 |
| lucasagomes | right, yeah it seems to be very fine grained | 14:51 |
| lucasagomes | I wonder if someone would actually have more than 1 network provider like that | 14:52 |
| lucasagomes | like, it's possible but unrealistic | 14:52 |
| *** achanda has joined #openstack-ironic | 14:53 | |
| *** achanda_ has joined #openstack-ironic | 14:58 | |
| *** bethelwell has quit IRC | 14:58 | |
| *** mgoddard1 has quit IRC | 15:00 | |
| *** achanda has quit IRC | 15:00 | |
| *** amotoki has joined #openstack-ironic | 15:01 | |
| *** mgoddard has joined #openstack-ironic | 15:01 | |
| *** dguerri is now known as dguerri` | 15:01 | |
| *** bethelwell has joined #openstack-ironic | 15:02 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Initial changes for testing VM creation for CentOS https://review.openstack.org/191504 | 15:05 |
| TheJulia | cinerama: ^^^ please take a look at that one | 15:05 |
| *** dguerri` is now known as dguerri | 15:05 | |
| *** cdearborn has joined #openstack-ironic | 15:14 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Initial changes for testing VM creation for CentOS https://review.openstack.org/191504 | 15:14 |
| *** e0ne is now known as e0ne_ | 15:22 | |
| NobodyCam | ugh really laggy bandwidth here | 15:25 |
| *** Nisha has joined #openstack-ironic | 15:27 | |
| *** davideagnello has joined #openstack-ironic | 15:29 | |
| dtantsur | hey, anyone willing to have a look at the ENROLL spec https://review.openstack.org/179151 ? 2x +2 already :) | 15:29 |
| Nisha | dtantsur, +1 for the spec... :) looks good | 15:29 |
| dtantsur | thnx! | 15:30 |
| *** jcoufal has quit IRC | 15:30 | |
| Nisha | dtantsur, please review the capabilities specs..i need opinions/advise from people to finalize on approach to be followed for it | 15:31 |
| *** foexle has quit IRC | 15:31 | |
| dtantsur | Nisha, link? | 15:31 |
| Nisha | #link https://review.openstack.org/182601, https://review.openstack.org/182934 | 15:32 |
| *** e0ne_ has quit IRC | 15:32 | |
| *** davideagnello has quit IRC | 15:33 | |
| jroll | Nisha: just fyi you only need to use #link during meetings :) | 15:34 |
| Nisha | jroll, :) | 15:34 |
| Nisha | jroll, ^^^^ | 15:35 |
| *** e0ne has joined #openstack-ironic | 15:35 | |
| Nisha | i need opinion from people to finalize an approach to be followed for capabilities | 15:35 |
| jroll | Nisha: I'll try to get to it soon, probably not today :/ | 15:36 |
| Nisha | ok | 15:36 |
| Nisha | :( | 15:36 |
| jroll | sorry, busy day. | 15:36 |
| NobodyCam | just ping folks for sub-team status updates on the white board | 15:37 |
| * dtantsur didn't forget this time =^_^= | 15:37 | |
| NobodyCam | s/ping/pinging/ | 15:37 |
| * jroll will update neutron subteam thing after the neutron/ironic meeting | 15:37 | |
| NobodyCam | :) | 15:37 |
| dtantsur | jroll, do you need me at the ironic-neutron meeting? | 15:37 |
| NobodyCam | :) | 15:38 |
| NobodyCam | awesome thank you :) | 15:38 |
| jroll | dtantsur: I don't think so, unless you have opinions on the specs that are up :) | 15:38 |
| * TheJulia ponders the enroll spec | 15:38 | |
| *** spandhe has joined #openstack-ironic | 15:38 | |
| dtantsur | jroll, not now :) just making sure you're not discussing something inspector-related for now | 15:39 |
| jroll | dtantsur: nah, I don't think inspector things will get contentious | 15:39 |
| *** yog__ has quit IRC | 15:39 | |
| *** absubram has joined #openstack-ironic | 15:39 | |
| dtantsur | good | 15:39 |
| jroll | we just need to add lldp discovery at some point | 15:39 |
| dtantsur | that's on my radar, but moving to IPA is of bigger priority for me wrt the ramdisk | 15:39 |
| jroll | yeah | 15:40 |
| jroll | no big rush | 15:40 |
| * NobodyCam will have to watch the Lego movie in full some time | 15:40 | |
| jroll | oh man | 15:40 |
| jroll | NobodyCam: it's awesome | 15:40 |
| * dtantsur is afk until the meeting | 15:41 | |
| dtantsur | is meeting in 1hr 20 mins? | 15:41 |
| NobodyCam | :) yep | 15:41 |
| dtantsur | cool! | 15:41 |
| * NobodyCam has call in 20 min | 15:41 | |
| *** dtantsur is now known as dtantsur|brb | 15:41 | |
| openstackgerrit | Merged openstack/ironic-specs: iPXE to use Swift Temporary URLs https://review.openstack.org/181920 | 15:43 |
| TheJulia | dtantsur|brb: so if I'm interpretting the enroll state stuff correctly, once the new client lands, we will explicitly have to trigger the transition to manage state? | 15:44 |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector: Provide hook into process when a node isn't found https://review.openstack.org/190124 | 15:51 |
| *** Sukhdev has joined #openstack-ironic | 15:51 | |
| lucasagomes | folks I will take a break and be back for the meeting later on | 15:52 |
| *** lucasagomes is now known as lucas-brb | 15:52 | |
| *** lazy_prince has joined #openstack-ironic | 15:56 | |
| *** mitchjameson has joined #openstack-ironic | 15:59 | |
| openstackgerrit | Roman Podoliaka proposed openstack/ironic: WIP: use new enginefacade feature of oslo.db https://review.openstack.org/191801 | 16:02 |
| *** mgoddard1 has joined #openstack-ironic | 16:03 | |
| *** mgoddard has quit IRC | 16:05 | |
| *** Nisha has quit IRC | 16:09 | |
| *** romcheg has quit IRC | 16:09 | |
| *** romcheg has joined #openstack-ironic | 16:10 | |
| *** Nisha has joined #openstack-ironic | 16:10 | |
| *** romcheg has quit IRC | 16:15 | |
| jlvillal | Good morning Ironic. | 16:19 |
| jlvillal | +1 on Lego movie :) | 16:19 |
| NobodyCam | morning jlvillal | 16:21 |
| NobodyCam | :) | 16:21 |
| *** Nisha has quit IRC | 16:21 | |
| jlvillal | NobodyCam: Thanks | 16:21 |
| *** Nisha has joined #openstack-ironic | 16:21 | |
| *** zz_jgrimm is now known as jgrimm | 16:22 | |
| *** notq has quit IRC | 16:30 | |
| *** romainh has quit IRC | 16:33 | |
| *** dprince has joined #openstack-ironic | 16:34 | |
| *** spandhe_ has joined #openstack-ironic | 16:35 | |
| *** spandhe has quit IRC | 16:37 | |
| *** spandhe_ is now known as spandhe | 16:37 | |
| *** Nisha_away has joined #openstack-ironic | 16:40 | |
| *** Nisha has quit IRC | 16:40 | |
| *** dtantsur|brb is now known as dtantsur | 16:42 | |
| *** trown is now known as trown|lunch | 16:47 | |
| *** lucas-brb is now known as lucasagomes | 16:48 | |
| NobodyCam | if I drop in the meeting please continue with out /me... I will be attempting to rejoin! | 16:51 |
| TheJulia | 10-4 | 16:52 |
| NobodyCam | :) | 16:52 |
| *** dguerri is now known as dguerri` | 16:53 | |
| *** praneshp has joined #openstack-ironic | 16:53 | |
| *** Nisha_away has quit IRC | 16:58 | |
| *** natorious is now known as zz_natorious | 16:59 | |
| NobodyCam | other meeting just ending | 17:01 |
| *** e0ne has quit IRC | 17:02 | |
| *** bethelwell has quit IRC | 17:02 | |
| *** davideagnello has joined #openstack-ironic | 17:06 | |
| *** ifarkas has quit IRC | 17:08 | |
| NobodyCam | mornign rloo :-p | 17:09 |
| rloo | NobodyCam: :) | 17:10 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Refactor node's and driver's vendor passthru to a common place https://review.openstack.org/190121 | 17:10 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Allow vendor methods to serve static files https://review.openstack.org/189716 | 17:10 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Change return value of [driver_]vendor_passthru to dict https://review.openstack.org/190120 | 17:10 |
| *** lazy_prince has quit IRC | 17:11 | |
| openstackgerrit | Shilla Saebi proposed openstack/ironic: changes and edit of the ironic install guide https://review.openstack.org/191900 | 17:12 |
| *** romcheg has joined #openstack-ironic | 17:13 | |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Refactor node's and driver's vendor passthru to a common place https://review.openstack.org/190121 | 17:13 |
| openstackgerrit | Lucas Alvares Gomes proposed openstack/ironic: Allow vendor methods to serve static files https://review.openstack.org/189716 | 17:13 |
| openstackgerrit | Sam Betts proposed openstack/ironic-inspector: Provide hook into process when a node isn't found https://review.openstack.org/190124 | 17:15 |
| *** zz_natorious is now known as natorious | 17:18 | |
| jroll | naohirot: cores each have different priorities, and the project as a whole also has priorities | 17:24 |
| harlowja | dtantsur yt | 17:40 |
| harlowja | as for futurist release, soon? | 17:40 |
| dtantsur | that's good :) | 17:40 |
| harlowja | and as for compat with old stuff, idk, might be easy to write a wrapper that makes the old junk work | 17:40 |
| * harlowja hasn't tried | 17:40 | |
| dtantsur | ok, I'll figure out | 17:40 |
| dtantsur | chances are high, it will just work | 17:41 |
| harlowja | ya, some of the concepts don't apply though | 17:41 |
| harlowja | like 'ticks_between_runs' | 17:41 |
| *** ijw has quit IRC | 17:41 | |
| harlowja | 'external_process_ok' | 17:41 |
| harlowja | https://github.com/openstack/oslo.service/blob/master/oslo_service/periodic_task.py#L40 | 17:41 |
| dtantsur | harlowja, I don't think we used such stuff for anything in-tree | 17:41 |
| harlowja | k | 17:41 |
| harlowja | the way those periodic things all get registered is different, the futurist stuff doesn't use a metaclass blah blah | 17:42 |
| openstackgerrit | Michael Krotscheck proposed openstack/ironic: Added documentation to Vagrantfile https://review.openstack.org/191912 | 17:42 |
| harlowja | *imho for the better* | 17:42 |
| *** Marga_ has quit IRC | 17:42 | |
| harlowja | futurist is imho much more easy to use, it doesn't require metaclasses, eventlet stuff... | 17:42 |
| harlowja | *but i'm biased, lol | 17:42 |
| harlowja | and its more easy to create different work objects, instead of just getting stuck with 1 ... | 17:43 |
| harlowja | *different worker objects | 17:43 |
| harlowja | ya da ya da, lol | 17:43 |
| harlowja | dtantsur https://github.com/openstack/oslo.service/blob/master/oslo_service/periodic_task.py#L150 might be something useful to adapt though | 17:43 |
| harlowja | dtantsur https://review.openstack.org/#/c/191163/ is along these lines (but not exactly, allows for some customization there) | 17:44 |
| * harlowja afaik the oslo.service primary maintainer is also onboard with the futurist periodic stuff, so dtantsur if u have a way to make an 'adaptor' or something, that'd be cool | 17:45 | |
| harlowja | i'm sure he/she (the maintainer) would like that | 17:45 |
| dtantsur | hmm, will consider | 17:45 |
| harlowja | k | 17:45 |
| harlowja | dtantsur http://bugs.python.org/issue24451 may also interest u, getting better metrics out of those objects would be nice/useful | 17:46 |
| dtantsur | nice, IIRC jroll was especially interested in metrics | 17:47 |
| jroll | <3 knowing how my software is running | 17:47 |
| harlowja | ya, futurist gathers some, but its not still optimal imho | 17:47 |
| harlowja | aka these, http://docs.openstack.org/developer/futurist/api.html#futurist.ExecutorStatistics | 17:47 |
| jroll | dtantsur: a large portion of our team spun out of the "cloud monitoring/metrics" team :P | 17:47 |
| harlowja | it'd be more accurate with native metrics 'hooks' | 17:47 |
| dtantsur | rloo, it's up to us eventually whether to adopt guidelines or not... | 17:50 |
| *** Sukhdev has quit IRC | 17:51 | |
| dtantsur | and if we can't agree, it's up to the PTL to break the tie | 17:51 |
| rloo | dtantsur: i realize that, which is why i wanted to know what it meant to 'adopt guidelines' and is there room to deviate from it | 17:51 |
| dtantsur | rloo, I believe, if we deviate in "hiding" features, it's not critical at all | 17:51 |
| harlowja | rloo dear leader rloo ! | 17:52 |
| *** Nisha has joined #openstack-ironic | 17:52 | |
| harlowja | :) | 17:52 |
| * harlowja runs away | 17:52 | |
| *** trown|lunch is now known as trown | 17:52 | |
| dtantsur | lol | 17:52 |
| *** zhenguo has quit IRC | 17:53 | |
| *** pcrews has quit IRC | 17:54 | |
| harlowja | jroll dtantsur comment on http://bugs.python.org/issue24451 if u get sometime, maybe saying 'yes i would find this useful' would be enough :) | 17:55 |
| rloo | dtantsur: not critical, but i wanted to know/understand what people's understanding/expectations were, wrt guidelines | 17:55 |
| harlowja | guido commented on it, so don't want him to kill it, lol | 17:55 |
| harlowja | *killed before birth by BDFL | 17:55 |
| dtantsur | harlowja, ack | 17:55 |
| jroll | JoshNang: I may do that :P | 17:56 |
| JoshNang | jroll: ? | 17:56 |
| jroll | oops | 17:57 |
| jroll | wrong josh. | 17:57 |
| jroll | harlowja: ^^ | 17:57 |
| harlowja | :) | 17:57 |
| JoshNang | heh | 17:57 |
| *** pelix has quit IRC | 17:57 | |
| harlowja | one of those damn josh(s) | 17:57 |
| harlowja | lol | 17:57 |
| jroll | yeah, they're all crazy | 17:57 |
| harlowja | ;0 | 17:57 |
| harlowja | ;) i mean | 17:57 |
| JoshNang | :D | 17:57 |
| harlowja | if http://bugs.python.org/issue24451 goes in, then i can help get it backported (hopefully) into https://github.com/agronholm/pythonfutures and all will be merry | 17:58 |
| *** mgoddard1 has quit IRC | 18:00 | |
| NobodyCam | sorry I got choppy at the end :-p of course | 18:01 |
| NobodyCam | :-p | 18:01 |
| *** ijw has joined #openstack-ironic | 18:01 | |
| dtantsur | so, before I got caught in one more versioning holy war, I'll be going :) | 18:02 |
| NobodyCam | b | 18:02 |
| NobodyCam | r | 18:02 |
| NobodyCam | b | 18:02 |
| NobodyCam | lo | 18:02 |
| NobodyCam | l | 18:02 |
| dtantsur | l | 18:02 |
| NobodyCam | grrr | 18:02 |
| NobodyCam | hehehehe | 18:02 |
| dtantsur | haha | 18:02 |
| dtantsur | g'night folks :) | 18:02 |
| NobodyCam | night dtantsur | 18:02 |
| *** dtantsur is now known as dtantsur|afk | 18:02 | |
| NobodyCam | night lucasagomes | 18:02 |
| NobodyCam | :) | 18:02 |
| sambetts | I'm also heading off, see yall tomorrow :) o/ | 18:02 |
| naohirot | dtantsur: thanks for reviewing the irmc code | 18:02 |
| lucasagomes | NobodyCam, hey | 18:02 |
| lucasagomes | thank you have a good nigth everyone! | 18:03 |
| naohirot | lucasagomes: are you going leave now? | 18:03 |
| lucasagomes | naohirot, I will update some of the spreadsheet first | 18:03 |
| lucasagomes | naohirot, https://docs.google.com/spreadsheets/d/1Hxyfy60hN_Fit0b-plsPzK6yW3ePQC5IfwuzJwltlbo/edit?pli=1#gid=1338037272 | 18:03 |
| naohirot | lucasagomes: are we creating a priorty list? that's good, that is what I'd like to see | 18:04 |
| rloo | lucasagomes: are you ok if i change the tab from 'Current cycle Status' to 'Liberty cycle Status'? | 18:04 |
| lucasagomes | rloo, sure go ahead | 18:05 |
| lucasagomes | naohirot, yes... well with some stuff I have in mind now | 18:05 |
| lucasagomes | we can start add more stuff as we go | 18:05 |
| lucasagomes | so people can look at the spreadsheet and see what's important to review | 18:06 |
| jlvillal | lucasagomes: I think only certain people can edit it. Or I don't understand Google Sheets. As it appears to be read-only for me. | 18:06 |
| rloo | lucasagomes: it doesn't show the priorties. are we going to only list the high priority ones? | 18:06 |
| lucasagomes | rloo, maybe I should add a column for priorities? | 18:06 |
| rloo | lucasagomes: yeah, i think that would be good. | 18:07 |
| lucasagomes | jlvillal, hmm I'm not sure lemme see | 18:07 |
| jlvillal | lucasagomes: Not necessarily a bad thing if only certain people can update the spread sheet. | 18:07 |
| rloo | jlvillal: i just found out that i can edit that spreadsheet. maybe cores got access?? | 18:07 |
| * jlvillal is fine if only Core's can update the priority spreadsheet | 18:07 | |
| lucasagomes | jlvillal, apparently only a few group of people has access to editing it | 18:07 |
| jlvillal | lucasagomes: Sounds good to me. Thanks. | 18:07 |
| lucasagomes | rloo, adding a column | 18:07 |
| NobodyCam | night sambetts | 18:08 |
| lucasagomes | jlvillal, it's just the first stab at it, we can change this stuff later | 18:08 |
| rloo | thx lucasagomes | 18:08 |
| rloo | jlvillal: what did you want changed on that? | 18:08 |
| naohirot | rloo: I attended irc meeting this week which is not convenient time for Asia. | 18:09 |
| rloo | lucasagomes: can I suggest this ordering: Topic first | 18:09 |
| lucasagomes | rloo, sure... few free to edit it | 18:09 |
| lucasagomes | I'm actually bad organizing data | 18:09 |
| rloo | lucasagomes: topic, category, priority, status, lead, spec | 18:09 |
| naohirot | rloo: you mentioned about no core member irc meeting last week in the openstack-dev. | 18:09 |
| lucasagomes | so I'm just put the content | 18:09 |
| lucasagomes | putting* | 18:10 |
| jlvillal | rloo: Nothing. Well I was going to re-arrange the sheets to be in reverse chronological order. But not important. | 18:10 |
| rloo | lucasagomes: am just trying to think of what i would be looking for/at. i don't really care about the lead as the first thing :) | 18:10 |
| *** natorious is now known as zz_natorious | 18:10 | |
| lucasagomes | rloo, heh done | 18:10 |
| rloo | jlvillal: already reversed the order :) | 18:10 |
| rloo | thx lucasagomes | 18:10 |
| jlvillal | rloo: Thanks | 18:11 |
| naohirot | rloo: will the next irc meeting also be no core member available? | 18:11 |
| rloo | i like the 'notes' and 'ADDITIONAL NOTES' columns :) | 18:11 |
| rloo | naohirot: I have no idea. Depends on if any cores attend. I never attend that one. | 18:11 |
| *** spandhe_ has joined #openstack-ironic | 18:12 | |
| *** Nisha has quit IRC | 18:12 | |
| lucasagomes | naohirot, the other meeting time is really bad for me | 18:12 |
| lucasagomes | naohirot, so I have never attended that :-( | 18:12 |
| *** spandhe has quit IRC | 18:12 | |
| *** spandhe_ is now known as spandhe | 18:12 | |
| TheJulia | naohirot: some cores attempt to make it to that meeting, but attendence is often less due to the timing. | 18:12 |
| *** Nisha has joined #openstack-ironic | 18:12 | |
| lucasagomes | rloo, perhaps additional notes is redudant? | 18:12 |
| * TheJulia thinks she made it to one and was told to go to bed | 18:12 | |
| lucasagomes | I think we can remove it right? | 18:12 |
| rloo | naohirot: I mentioned that because I wondered whether/if it was worth holding meetings if no cores attended but I didn't see any feedback saying that it wasn't useful. | 18:13 |
| naohirot | rloo: why does core team assign someone to Asia convenient time meeting? | 18:13 |
| lucasagomes | rloo, not sure you were being sarcastic or not heh | 18:13 |
| rloo | lucasagomes: both. i couldn't figure out why they were there but it seemed amusing too | 18:13 |
| lucasagomes | rloo, lol I copy and pasted from the Kilo status | 18:13 |
| *** Nisha has quit IRC | 18:13 | |
| lucasagomes | anyway removed | 18:13 |
| *** Nisha has joined #openstack-ironic | 18:14 | |
| rloo | naohirot: why don't you suggest it then? I don't know. When the meeting times were picked, I am sure there was coverage/some cores that said they could attend it. (I cannot/won't). | 18:14 |
| naohirot | rloo: I believe if no core member continue attends the meeting, other contributer is getting not attending. | 18:14 |
| rloo | naohirot: but everyone cannot promise to be there all the time. | 18:14 |
| harlowja | rloo stop sleeping so much, lol, jeez | 18:15 |
| lucasagomes | naohirot, we have Haomeng in China | 18:15 |
| rloo | naohirot: i think that if you have some suggestions it would be good to voice them in email. | 18:15 |
| *** Nisha has quit IRC | 18:15 | |
| *** Nisha has joined #openstack-ironic | 18:15 | |
| *** dguerri` is now known as dguerri | 18:15 | |
| naohirot | rloo: lucasagomes : my question is | 18:15 |
| lucasagomes | and ramesh in India (idk how is the TZ there) | 18:15 |
| rloo | naohirot: again, i don't attend those meetings but others do, so for the ones that do, if they find them useful, who am i to object to them. | 18:16 |
| naohirot | rloo: lucasagomes : is some coordination made among core member? | 18:16 |
| lucasagomes | naohirot, about who attend what? Not really | 18:17 |
| lucasagomes | I mean I try to attend the meeting at 1700 UTC cause it's an OK time for me | 18:17 |
| lucasagomes | the other one is really bad time so I really can't attend | 18:17 |
| lucasagomes | we probably can suggest Haomeng and ramesh to try to attend the other meeting | 18:17 |
| lucasagomes | but we can't mandate any of this | 18:17 |
| lucasagomes | naohirot, we also could try change the time if that is not working | 18:18 |
| lucasagomes | naohirot, I would suggest to put it in the ML, see what people think about changing the time of that meeting. Or call some attention about the potential new attendees | 18:19 |
| *** ukalifon has joined #openstack-ironic | 18:19 | |
| *** Marga_ has joined #openstack-ironic | 18:19 | |
| naohirot | lucasagomes: mandate is strong word, but some coordination should be made so that core member attend. | 18:19 |
| lucasagomes | naohirot, right, yeah we could try to suggest them to attend yes | 18:19 |
| lucasagomes | or anyone interested in the core group really | 18:20 |
| naohirot | lucasagomes: and also I'd like to say one thing regarding priority. | 18:20 |
| naohirot | lucasagomes: priority I'm talking about is not promise, and not demand | 18:21 |
| *** dguerri is now known as dguerri` | 18:21 | |
| naohirot | lucasagomes: priority is just plan at this moment | 18:21 |
| naohirot | lucasagomes: if situation changed, then priority should be changed | 18:22 |
| naohirot | lucasagomes: this kind of priority is what I'm talking about. | 18:22 |
| lucasagomes | naohirot, +1 totally | 18:22 |
| lucasagomes | I think that's the only one we can count on | 18:23 |
| lucasagomes | since we all work for different companies | 18:23 |
| naohirot | lucasagomes: Yeah :) | 18:23 |
| lucasagomes | right, and with that I will call it a day | 18:24 |
| lucasagomes | it's late I will get some dinner | 18:24 |
| naohirot | lucasagomes: without priority or plan, we will become day laborer :) | 18:24 |
| lucasagomes | I will put more things in the spreadsheet tomorrow | 18:24 |
| lucasagomes | naohirot, heh yeah it can get a bit chaotic | 18:24 |
| lucasagomes | FOSS is a bit chaotic in general, somehow we manage to work together | 18:25 |
| lucasagomes | I don't know how tho :-) | 18:25 |
| lucasagomes | that's why it's amazing | 18:25 |
| naohirot | lucasagomes: maybe still I didn't get what open source development is :) | 18:25 |
| lucasagomes | naohirot, neither do I | 18:25 |
| lucasagomes | hard to know someone that really got it... each community is different as far as I can tell | 18:25 |
| * lucasagomes remembers mandriva | 18:26 | |
| naohirot | lucasagomes: Okay, I'll go to bed | 18:26 |
| lucasagomes | anyhoo... naohirot thanks for the suggestion. Have a good night | 18:26 |
| lucasagomes | have a good night everyone, I see y'all tomorrow | 18:26 |
| naohirot | lucasagomes: rloo : all: thanks!! | 18:26 |
| *** naohirot has quit IRC | 18:26 | |
| *** lucasagomes is now known as lucas-dinner | 18:26 | |
| *** notq has joined #openstack-ironic | 18:27 | |
| *** pcrews has joined #openstack-ironic | 18:28 | |
| *** albertoffb has joined #openstack-ironic | 18:31 | |
| *** spandhe_ has joined #openstack-ironic | 18:33 | |
| *** Marga_ has quit IRC | 18:35 | |
| *** spandhe has quit IRC | 18:36 | |
| *** spandhe_ is now known as spandhe | 18:36 | |
| *** Marga_ has joined #openstack-ironic | 18:36 | |
| jlvillal | jroll: Can you look at my comment on https://review.openstack.org/#/c/179151/ ? I want to make sure I'm not confused. | 18:37 |
| jroll | yesh | 18:37 |
| *** bethelwell has joined #openstack-ironic | 18:37 | |
| jlvillal | jroll: Thanks! | 18:37 |
| jroll | jlvillal: oh | 18:38 |
| jroll | so verifying is a transient state | 18:38 |
| jroll | so "manage" sends it from enroll -> verifying -> manageable | 18:38 |
| *** bethelwell has quit IRC | 18:38 | |
| jroll | just like we say "deploy" goes from available -> active | 18:38 |
| jroll | even when it really goes available -> deploywait -> deploying -> active | 18:38 |
| jlvillal | jroll: Okay. Do you think the spec should explicitly say that? Or should the reader know it? | 18:39 |
| jroll | jlvillal: idk. basically line 34 is from the user perspective and line 86 is from the state machine perspective | 18:39 |
| *** achanda_ has quit IRC | 18:40 | |
| jroll | jlvillal: if it isn't immediately clear it might be worth changing the words there, idk. | 18:40 |
| jlvillal | jroll, Thanks. It makes sense now. | 18:41 |
| jroll | cool :) | 18:41 |
| *** erelwell has joined #openstack-ironic | 18:41 | |
| jlvillal | jroll, My main workstation just shut down due to over-heating :( I'll change my vote. | 18:41 |
| jlvillal | Two unrelated events :) | 18:42 |
| jroll | jlvillal: heh | 18:42 |
| *** erelwell is now known as bethelwell | 18:45 | |
| *** e0ne has joined #openstack-ironic | 18:51 | |
| TheJulia | jroll: but a user or process still needs to tell ironic about it/enroll, then send it to manage state? | 18:51 |
| *** Marga_ has quit IRC | 18:54 | |
| *** Marga_ has joined #openstack-ironic | 18:54 | |
| *** absubram has quit IRC | 19:01 | |
| rloo | hey, googledoc experts or anyone competent (not me). I accidentally deleted a google doc. How can i get back what I see in revision history? | 19:04 |
| rloo | https://docs.google.com/spreadsheets/d/1Hxyfy60hN_Fit0b-plsPzK6yW3ePQC5IfwuzJwltlbo/edit?pli=1#gid=1338037272 | 19:04 |
| *** pcrews has quit IRC | 19:08 | |
| openstackgerrit | Caio Oliveira proposed openstack/ironic: OneView Driver for Ironic https://review.openstack.org/191822 | 19:11 |
| rloo | false alarm (I hope). I can see stuff there :) | 19:11 |
| *** dguerri` is now known as dguerri | 19:19 | |
| *** ukalifon has quit IRC | 19:21 | |
| *** mtanino has joined #openstack-ironic | 19:28 | |
| *** Marga_ has quit IRC | 19:30 | |
| *** Marga_ has joined #openstack-ironic | 19:30 | |
| *** amotoki has quit IRC | 19:33 | |
| *** pcrews has joined #openstack-ironic | 19:35 | |
| *** Nisha has quit IRC | 19:36 | |
| *** dguerri is now known as dguerri` | 19:37 | |
| *** dguerri` is now known as dguerri | 19:38 | |
| *** dprince has quit IRC | 19:39 | |
| *** ijw_ has joined #openstack-ironic | 19:43 | |
| *** jgrimm is now known as zz_jgrimm | 19:44 | |
| *** ijw has quit IRC | 19:46 | |
| *** alexpilotti has quit IRC | 19:48 | |
| *** achanda has joined #openstack-ironic | 19:50 | |
| *** boris-42 has quit IRC | 19:52 | |
| *** romcheg has quit IRC | 19:52 | |
| *** bethelwell has quit IRC | 19:58 | |
| jroll | TheJulia: yes | 20:00 |
| *** ijw has joined #openstack-ironic | 20:02 | |
| TheJulia | jroll: thats what I thought :( | 20:02 |
| jroll | TheJulia: what's wrong with that? | 20:03 |
| *** ijw__ has joined #openstack-ironic | 20:04 | |
| *** ijw_ has quit IRC | 20:05 | |
| *** ijw__ has quit IRC | 20:06 | |
| TheJulia | jroll: just bifrost will need to know how to trigger that before the ironicclient library microversion gets bumped | 20:06 |
| *** ijw_ has joined #openstack-ironic | 20:07 | |
| jroll | TheJulia: just use the old api version until you have code in place to trigger it? | 20:07 |
| TheJulia | I guess I could see about locking shade to the old api version | 20:07 |
| *** ijw has quit IRC | 20:07 | |
| * TheJulia will need to ponder it... or just get gate-bifrost in python-ironicclient's job list | 20:08 | |
| jroll | yeah, that's the whole point of this version thing | 20:08 |
| *** romcheg has joined #openstack-ironic | 20:09 | |
| TheJulia | off the top of your head, do you know if the client library can be pinned by the user when loading it? | 20:09 |
| jroll | yes | 20:10 |
| jroll | --ironic-api-version iirc | 20:10 |
| openstackgerrit | Caio Oliveira proposed openstack/ironic: OneView Driver for Ironic https://review.openstack.org/191822 | 20:13 |
| openstackgerrit | Ruby Loo proposed openstack/ironic: Don't raise exception from set_failed_state() https://review.openstack.org/188188 | 20:19 |
| *** ijw_ has quit IRC | 20:24 | |
| *** Marga_ has quit IRC | 20:24 | |
| *** Marga_ has joined #openstack-ironic | 20:24 | |
| *** ijw has joined #openstack-ironic | 20:24 | |
| *** praneshp has quit IRC | 20:31 | |
| *** achanda has quit IRC | 20:34 | |
| *** achanda_ has joined #openstack-ironic | 20:34 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-specs: Add "enroll" state to the state machine https://review.openstack.org/179151 | 20:38 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-specs: Minor grammar cleanup of enroll-node-state https://review.openstack.org/191967 | 20:38 |
| jlvillal | Argh! | 20:39 |
| * jlvillal breathes a sigh of relief as his patch did not remove any votes to the parent patch... | 20:40 | |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-specs: Minor grammar cleanup of enroll-node-state https://review.openstack.org/191967 | 20:42 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic-specs: Minor grammar cleanup of enroll-node-state https://review.openstack.org/191967 | 20:43 |
| *** achanda_ has quit IRC | 20:46 | |
| *** rbudden has quit IRC | 20:46 | |
| *** rbudden has joined #openstack-ironic | 20:46 | |
| *** albertoffb has quit IRC | 20:52 | |
| *** Marga_ has quit IRC | 20:55 | |
| *** Marga_ has joined #openstack-ironic | 20:55 | |
| *** jjohnson2 has quit IRC | 21:04 | |
| *** ijw_ has joined #openstack-ironic | 21:08 | |
| *** gabriel-bezerra has quit IRC | 21:10 | |
| *** afaranha has quit IRC | 21:11 | |
| *** ijw has quit IRC | 21:11 | |
| *** penick has joined #openstack-ironic | 21:21 | |
| *** praneshp has joined #openstack-ironic | 21:22 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Fix when logic for boolean handling https://review.openstack.org/191979 | 21:22 |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Allow a user to define the OS for the Disk Image https://review.openstack.org/191980 | 21:23 |
| mrda | Morning Ironic | 21:24 |
| TheJulia | Good morning mrda | 21:24 |
| mrda | o/ | 21:24 |
| cinerama | oh mrda hi | 21:26 |
| TheJulia | cinerama: I'm thinking network info for the config drive might be the next thing. Just.... I think I might need to build centos and try simple-init to see if that works out better | 21:27 |
| cinerama | hmmm | 21:28 |
| cinerama | right now i'm smacking the side of the TV to see if that makes it work better, metaphorically speaking | 21:29 |
| *** Nisha has joined #openstack-ironic | 21:29 | |
| *** trown is now known as trown|outttypeww | 21:30 | |
| * TheJulia uses all the magical bifrost settings | 21:31 | |
| TheJulia | cinerama: if this works with simple-init, I'm just going to put it in the doc to build a centos end machine | 21:33 |
| cinerama | hee hee upstart | 21:34 |
| *** e0ne has quit IRC | 21:36 | |
| TheJulia | yeah... we're very ubuntu centric. This is going to be lots of very small changes | 21:36 |
| *** afaranha has joined #openstack-ironic | 21:39 | |
| *** gabriel-bezerra has joined #openstack-ironic | 21:40 | |
| *** Nisha has quit IRC | 21:45 | |
| *** Nisha has joined #openstack-ironic | 21:46 | |
| *** praneshp has quit IRC | 21:47 | |
| jlvillal | mrda: Good morning | 21:49 |
| mrda | hey jlvillal | 21:49 |
| *** dmitchell__ has quit IRC | 21:50 | |
| *** romcheg has quit IRC | 21:53 | |
| TheJulia | cinerama: I almost feel like the image stuff just needs to be totally split from the install | 21:54 |
| openstackgerrit | Joshua Harlow proposed openstack/ironic: Transition state machine to use automaton oslo lib. https://review.openstack.org/191985 | 21:57 |
| *** chlong has quit IRC | 21:57 | |
| cinerama | TheJulia: i'm happy with that | 21:57 |
| TheJulia | not a today thing, but it would make things a lot cleaner | 21:57 |
| cinerama | no i agree, that is why i'm happy with that idea | 21:59 |
| jroll | TheJulia: what's this about network info for the config drive? :) | 21:59 |
| openstackgerrit | John L. Villalovos proposed openstack/ironic: Transition state machine to use automaton oslo lib. https://review.openstack.org/191985 | 21:59 |
| TheJulia | jroll: So I don't entirely get device naming in centos, or re-naming, haven't digged to much, so I'm kind of just hoping the simple-init element makes me happy | 22:00 |
| TheJulia | "Here, take my network configuration info, and magically make networking!" | 22:01 |
| TheJulia | s/to/too/ | 22:01 |
| jroll | TheJulia: so there's some work around this already | 22:01 |
| TheJulia | jroll: yah, I'm calling what morded did | 22:01 |
| jroll | TheJulia: https://review.openstack.org/#/c/102649/ | 22:01 |
| jroll | ah | 22:01 |
| *** boris-42 has joined #openstack-ironic | 22:01 | |
| jroll | ok | 22:01 |
| *** zz_natorious is now known as natorious | 22:01 | |
| TheJulia | jroll: Ohhhhh thanks! | 22:02 |
| *** Nisha has quit IRC | 22:02 | |
| * TheJulia can use this to validate what bifrost writes out | 22:02 | |
| jroll | :) | 22:02 |
| jroll | what mordred is using now is a downstream implementation of that | 22:03 |
| jroll | should be pretty similar | 22:03 |
| jroll | we're also going to make cloud-init work with that, for users that like cloud-init | 22:03 |
| TheJulia | Yeah, I'd prefer to drop cloud-init from the images bifrost deploys for testing... just not something I've gotten to | 22:04 |
| openstackgerrit | Stephanie Miller proposed openstack/bifrost: Add RedHat family dependency installation https://review.openstack.org/191988 | 22:12 |
| *** cdearborn has quit IRC | 22:13 | |
| *** Marga_ has quit IRC | 22:18 | |
| *** Marga_ has joined #openstack-ironic | 22:19 | |
| *** lucas-dinner has quit IRC | 22:20 | |
| *** david-lyle has quit IRC | 22:31 | |
| *** praneshp has joined #openstack-ironic | 22:44 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Add a dynamic prepare for test role https://review.openstack.org/192001 | 22:46 |
| *** Haomeng|2 has joined #openstack-ironic | 22:50 | |
| *** Haomeng has quit IRC | 22:53 | |
| jlvillal | mrda, Any feedback on https://review.openstack.org/173681 ? | 22:55 |
| jlvillal | mrda, It looks good to me. I was thinking about doing a +1 | 22:56 |
| *** alexpilotti has joined #openstack-ironic | 23:01 | |
| mrda | jlvillal: as you can see jroll has some opinions, but I'll take a CLOSER LOOK TOO | 23:01 |
| jroll | heh | 23:01 |
| jroll | sorry I meant to let you two know I just -1'd that | 23:01 |
| jroll | I might be wrong there | 23:01 |
| jlvillal | mrda, jroll: Glad I asked :) | 23:02 |
| * mrda is glad jroll shares his opinions | 23:02 | |
| *** alexpilotti has quit IRC | 23:03 | |
| *** penick has quit IRC | 23:04 | |
| *** Marga_ has quit IRC | 23:05 | |
| *** Marga_ has joined #openstack-ironic | 23:06 | |
| *** david-lyle has joined #openstack-ironic | 23:11 | |
| openstackgerrit | Julia Kreger proposed openstack/bifrost: Fix variable reference in dynamic node deployment https://review.openstack.org/192013 | 23:15 |
| *** penick has joined #openstack-ironic | 23:17 | |
| *** yuanying has joined #openstack-ironic | 23:17 | |
| *** EmilienM|afk is now known as EmilienM | 23:24 | |
| *** ijw_ has quit IRC | 23:28 | |
| *** ijw has joined #openstack-ironic | 23:29 | |
| *** penick has quit IRC | 23:34 | |
| *** penick has joined #openstack-ironic | 23:42 | |
| *** ijw has quit IRC | 23:45 | |
| *** chlong has joined #openstack-ironic | 23:47 | |
| *** Marga_ has quit IRC | 23:49 | |
| *** Marga_ has joined #openstack-ironic | 23:49 | |
| *** penick has quit IRC | 23:56 | |
| *** praneshp has quit IRC | 23:57 | |
| *** Sukhdev has joined #openstack-ironic | 23:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!