Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-02-28
15:04:00 sfinucan mriedem: That run_tests patch you requested is here https://review.openstack.org/#/c/437947
15:05:13 johnthetubaguy mdbooth: maybe the bit I am not getting is why you have to reset the state when you loose connection to libvirt?
15:14:28 mdbooth johnthetubaguy: That was originally danpb's suggestion. It's a robustness measure.
15:14:49 mdbooth johnthetubaguy: We need to tie initialisation to host_up anyway, because initialisation uses the libvirt connection.
15:15:01 johnthetubaguy yeah, host up makes sense
15:15:07 mdbooth So re-initialising it after host_down is no great complication.
15:15:08 johnthetubaguy just seems odd to drop it
15:15:28 mdbooth johnthetubaguy: Well, if libvirt went down we don't really know what happened.
15:16:06 mdbooth That doesn't really complicate the code all that much, anyway, as it's separate in the manager
15:17:24 bauzas jaypipes: around ?
15:17:43 bauzas jaypipes: I have a question for you before +Wign https://review.openstack.org/#/c/376198/11
15:18:24 mdbooth The code is especially paranoid as it's reference counting. We want to do our best to be correct, even if operators are doing unexpected things underneath us, or things are crashing.
15:18:41 bauzas jaypipes: ie. we set the collation to latin1 encoding, I guess it's for memory reasons, right?
15:19:19 bauzas I just wonder if that's really worth it in case we would need to add another column needing utf8 encoding, like a description TEXT field
15:20:34 jaypipes bauzas: gimme a bit.. back to back meetings
15:20:43 bauzas heh, no worries
15:21:10 mriedem bauzas: before you go approving that code,
15:21:11 mriedem https://blueprints.launchpad.net/openstack/?searchtext=resource-provider-traits
15:21:15 mriedem can we get the bp actually approved?
15:21:34 bauzas oh very good point
15:21:49 bauzas was there a spec reproposed ?
15:22:09 mriedem https://review.openstack.org/#/c/345138/
15:22:12 bauzas there it is https://review.openstack.org/#/c/345138/
15:22:14 bauzas yeah
15:23:00 bauzas here is the thing, given it would take more time for that as we haven't previously approved the spec
15:23:17 bauzas mriedem: I would then wonder if it would be worth decoupling that spec into 2 pieces
15:23:30 mriedem i haven't read the spec,
15:23:31 spotz Hey nova, anyone about who could review the doc we're working on for the placement API and cells and give feedback? We can't build instances so we're missing something https://review.openstack.org/#/c/438328/
15:23:33 bauzas mriedem: one would be the the basement piece we agreed as the top prio
15:23:38 mriedem if it's the full hog of traits, then yeah probably
15:23:42 bauzas mriedem: the other would be exposing it thru the API
15:23:48 mriedem since we just need the piece to fix can_host
15:24:14 mriedem spotz: have you seen the docs in the nova devref?
15:24:28 mriedem https://docs.openstack.org/developer/nova/cells.html
15:24:33 mriedem https://docs.openstack.org/developer/nova/placement.html
15:24:59 dansmith spotz: also, kinda need to know the error you're hitting to know what you're missing
15:27:05 spotz dansmith: I'm getting the very unhelpful No valid hosts, I'm not sure what Brian got last night though
15:27:24 dansmith spotz: got host_mappings in the apidb?
15:28:41 spotz dansmith. Let me bring up the VM and double check
15:30:08 spotz dansmith: Nice call, table is empty
15:30:20 dansmith spotz: nova-manage cell_v2 discover_hosts
15:30:21 dansmith -v
15:32:19 spotz dansmith: Nada, not even with debug nova-manage --debug cell_v2 discover_hosts
15:32:40 dansmith spotz: got hosts actually registering with their cell db?
15:32:51 dansmith spotz: compute_nodes and services tables in the cell1 db
15:35:41 spotz dansmith: I thought so, but let me go back to the DB vs output from list_cells
15:36:02 dansmith spotz: -v should show you something at least I thought
15:36:15 dansmith spotz: like "checking cell foo"
15:36:57 dansmith yeah, confirmed
15:37:01 spotz dansmith: I have a nova_cell0 DB no cell1 DB
15:37:09 dansmith spotz: well that's a real problem :P
15:37:31 spotz dansmith: So our create_cell didn't work?
15:38:04 dansmith spotz: create_cell just tells nova where the db is for a given cell
15:38:16 dansmith spotz: you should have an apidb, a cell0 db, and a cell1 db
15:38:26 spotz dansmith: Ok so we're definitely missing steps, I do get this for list_cells though cell1 | 143df967-eb22-4d2c-baff-bfc2538816fd |
15:38:45 dansmith yeah, but that doesn't mean anything if you don't have a db to back it up
15:38:53 dansmith spotz: have you looked through our devref docs mriedem pointed to?
15:39:44 spotz dansmith: yeah that's where the steps came from
15:40:31 dansmith spotz: okay I see, you're doing both of those mappings with the same config, which means they probably point to the same place?
15:40:47 dansmith spotz: an unfortunate side-effect of the config file method for getting to this point :/
15:41:00 macsz sfinucan: lol what about no :D
15:41:03 dansmith spotz: can you pastebin your cel_mappings table?
15:41:20 spotz dansmith: sure hang on
15:41:22 macsz sfinucan: and also from nose's docs: "nose itself supports python 3, but many 3rd-party plugins do not!"
15:41:43 sfinucan macsz: Who says we need plugins, eh? ;)
15:42:48 spotz dansmith: https://gist.github.com/spotz/42b9d15f016073458742268735644578
15:43:12 dansmith spotz: oh, no you do have a cell1 db
15:43:24 dansmith spotz: in that db (nova) do you have services and compute_nodes?
15:45:03 macsz sfinucan: I have had enough of beers last week, so i can invest my friday's evenings to convert tox to nose, if you want it :P
15:45:29 sfinucan macsz: *testrepository to nose
15:45:33 sfinucan tox would stay
15:45:49 spotz dansmith: added as a comment on the gist
15:45:58 macsz sfinucan: ah, sorry, was focused on typing "beer"
15:46:18 sfinucan Ima take a look and see if it would even be possible. The testrepository library annoys me in many ways
15:46:24 sfinucan macsz: :D
15:46:38 dansmith spotz: cool, so can you show me the exact result of "nova-manage cell_v2 discover_hosts -v" ?
15:46:59 spotz dansmith: it didn't like -v as a switch when I tried earlier
15:47:09 spotz --debug gave no results to screen
15:47:39 spotz moved it but no help
15:47:41 spotz root@controller:~# nova-manage -v cell_v2 discover_hosts
15:47:42 spotz Option "verbose" from group "DEFAULT" is deprecated for removal. Its value may be silently ignored in the future.
15:48:16 dansmith spotz: -v goes after discover_hosts
15:48:47 spotz dansmith: that's no joy nova-manage: error: unrecognized arguments: -v
15:49:08 dansmith spotz: sorry it's --verbose
15:49:36 spotz that worked adding as a comment
15:50:26 dansmith spotz: right so I think it worked before when you did --debug
15:50:30 macsz sfinucan: yeah... and I hate my change with "rm" as well, but I am so fed up with doing it manually, and I could not see the use for having this file around either way...
15:50:36 dansmith spotz: it just didn't say anything because you didn't have --verbose
15:50:42 dansmith spotz: that "checking" means the mapping is there now
15:50:57 dansmith spotz: so you should have that host_mapping in the api db and maybe even be able to create instances
15:51:29 spotz dansmith: 2017-02-28 15:31:57 | NULL | 1 | 2 | compute |
15:51:54 dansmith spotz: I need more context, can you paste the full thing in the gist?
15:51:55 spotz dansmith: I wonder why it didn't work before though, I've run the command before
15:52:19 dansmith spotz: it needs to run after the host shows up in the cell db, so if you run it before you start the service it won't find it, of course
15:52:58 spotz dansmith: It was up but I'll make sure we make a note for that
15:54:36 spotz dansmith: no joy on the instance, checking logs to see if it's something new
15:55:25 spotz dansmith: still no valid hosts in the nova-conductor.log though I do think we made progress:)
15:56:02 dansmith spotz: well paste some logs
15:59:35 spotz dansmith: api, scheduler and conductor logs up
16:00:01 mriedem spotz: where?

Earlier   Later