Earlier  
Later
New Search
Posted Nick Remark
#openstack-nova - 2017-01-03
22:40:11 mriedem let me try with multiline
22:40:27 jlvillal Oh you got the regex to not match
22:42:18 mriedem yeah, i fixed the unit test to use the actual error response text we're getting in the CI job
22:42:24 mriedem anyway, removing .$ fixed it
22:42:46 mriedem here it comes
22:43:02 openstackgerrit Matt Riedemann proposed openstack/nova: placement: Do not save 0-valued inventory https://review.openstack.org/414214
22:43:03 mriedem jaypipes: jlvillal: ^
22:43:29 jlvillal mriedem: Thanks!
22:44:40 jlvillal mriedem: Ah, so re.MULTILINE would also have worked. So I wasn't totally wrong :)
22:45:13 jlvillal At least in my test. So '$' impacts the multiline searching
22:45:15 mriedem maybe the string ends with a newline?
22:45:36 mriedem i could still add multiline if we cared
22:45:41 mriedem since we're using search though,
22:45:46 mriedem i don't think we need ^ or $
22:46:00 mriedem anyway, here's to throwing things at the wall with regex until something works :)
22:46:02 jlvillal mriedem: Right as long as no '^' or '$' don't need the multiline
23:24:41 openstackgerrit Dan Peschman proposed openstack/nova: Option to always create network metadata information as static https://review.openstack.org/336293
23:26:56 openstackgerrit Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(1/2) https://review.openstack.org/408955
23:27:08 dtp mriedem added some tests ^
23:27:16 openstackgerrit Takashi NATSUME proposed openstack/nova: Enable cold migration with target host(2/2) https://review.openstack.org/408964
23:27:36 openstackgerrit Takashi NATSUME proposed openstack/nova: api-ref: Add parameters in cold migrate action https://review.openstack.org/410042
23:28:55 openstackgerrit Takashi NATSUME proposed openstack/nova: Remove mox in unit/api/openstack/test_wsgi.py https://review.openstack.org/414941
23:29:03 openstackgerrit Takashi NATSUME proposed openstack/nova: Add functional tests for cold migration to same host https://review.openstack.org/414926
23:29:13 openstackgerrit Takashi NATSUME proposed openstack/nova: Refactor a test method including 3 test cases https://review.openstack.org/394738
23:29:21 openstackgerrit Takashi NATSUME proposed openstack/nova: Remove mox in test_block_device_mapping_v1.py https://review.openstack.org/308746
23:29:28 openstackgerrit Takashi NATSUME proposed openstack/nova: Avoid forcing translation on logging calls https://review.openstack.org/413876
23:29:37 openstackgerrit Takashi NATSUME proposed openstack/nova: api-ref: Fix parameters in os-server-groups.inc https://review.openstack.org/415377
23:29:46 openstackgerrit Takashi NATSUME proposed openstack/python-novaclient: Microversion 2.41 - Enable cold migration with target host https://review.openstack.org/406707
23:29:56 openstackgerrit Takashi NATSUME proposed openstack/nova: Refactor a test method including 7 test cases https://review.openstack.org/395436
23:39:47 EmilienM mriedem: hi! happy new year. What port should we use to deploy nova placement api
23:40:10 EmilienM I'm confused between 8780 or 80 or
23:41:15 EmilienM doc says 8778 http://docs.openstack.org/developer/nova/placement.html?highlight=placement
23:49:09 mriedem EmilienM: i think bauzas has a patch up to change those docs
23:49:20 EmilienM mriedem: which port should we use?
23:49:35 mriedem because of this https://review.openstack.org/#/c/413118/
23:49:36 EmilienM in production, not in devstack.
23:50:18 mriedem EmilienM: see https://review.openstack.org/#/c/413158/
23:51:12 EmilienM using 80 is a terrible idea for operators
23:53:01 EmilienM the problem is that you can't mix vhosts with *:80 and IP:80
23:54:23 EmilienM why making things complex? it's first OpenStack API service i've seen where 80 is used by default
23:55:21 mriedem EmilienM: honestly some of this discussion is already in the ML and a lot of it was had between bauzas, cdent and sdague
23:55:55 dansmith EmilienM: are you sure you can't mix those two? because I thought you could
23:56:19 dansmith however, I thought this was mostly about not choosing a "conventional port" for placement and just relying on the service catalog?
23:56:51 dansmith https://httpd.apache.org/docs/2.4/vhosts/examples.html#mixed
23:57:10 mwhahaha for ssl you wouldn't be able to mix them i don't think
23:57:36 dansmith why not? with SNI it should be the same, no?
23:57:37 mwhahaha so it's not a good practice to assume that
23:57:56 mwhahaha it depends on the way apache handles the vhost configs and I thought they weren't merged together
#openstack-nova - 2017-01-04
00:00:30 EmilienM mwhahaha: i'm trying it now
00:00:43 mwhahaha maybe we need to go dig into apache more but it seems odd to have this be the first service with such a configuration and from a deployer's standpoint it complicates things
00:00:51 mwhahaha just some feedback on it
00:01:29 dansmith but it's not like it has to be on 80, right?
00:02:45 mwhahaha no it doesn't but it would be nice to have some sort of standard expectation around a standalone deployment if not 80
00:02:49 EmilienM dansmith: we're *trying* to deploy this service in tripleo, with ssl activated. So we need to pick a port and 443 seems like the one you would choose.
00:03:24 dansmith mwhahaha: I dunno, it seems kinda silly that we have any such defaults as it has only facilitated us avoiding using the service catalog properly in many places :)
00:04:00 mwhahaha i don't see how this is a fix for that at all, but ok
00:08:00 openstackgerrit Zhenyu Zheng proposed openstack/nova: Add query parameters white list for server list/detail https://review.openstack.org/408571
00:52:46 openstackgerrit STEW TY proposed openstack/nova: Transform instance.unrescue notifications (WIP) https://review.openstack.org/388275
00:59:34 openstackgerrit Michael Still proposed openstack/nova: Use a service account to make vendordata requests. https://review.openstack.org/415597
00:59:35 openstackgerrit Michael Still proposed openstack/nova: Optionally make dynamic vendordata failures fatal. https://review.openstack.org/416391
01:05:38 openstackgerrit Hieu LE proposed openstack/nova: Remove mox from unit/virt/libvirt/test_imagebackend.py (3) https://review.openstack.org/335765
01:12:28 openstackgerrit Hieu LE proposed openstack/nova: Remove mox from unit/virt/libvirt/test_imagebackend.py (4) https://review.openstack.org/335820
01:16:24 openstackgerrit Takashi NATSUME proposed openstack/nova: Avoid forcing translation on logging calls https://review.openstack.org/413876
01:18:35 openstackgerrit Zhenyu Zheng proposed openstack/nova: Add sort_key white list for server list/detail https://review.openstack.org/415142
01:19:33 openstackgerrit Hieu LE proposed openstack/nova: Remove mox from unit/virt/libvirt/test_imagebackend.py (5) https://review.openstack.org/335921
01:33:02 openstackgerrit Li Yingjun proposed openstack/nova: Notifications for flavor operations https://review.openstack.org/398171
01:47:25 mriedem claudiub|2: were you going to update this? https://review.openstack.org/#/c/248029/10/releasenotes/notes/hyper-v-set-boot-order-1e76b08ca6783add.yaml
01:48:19 mriedem claudiub|2: or did you just mean that gen 2 VMs don't support boot from ISO?
01:49:23 mriedem jlvillal: that did the trick with the regex http://logs.openstack.org/23/415523/1/check/gate-tempest-dsvm-ironic-ipa-wholedisk-agent_ipmitool-tinyipa-ubuntu-xenial/15dd658/logs/screen-n-cpu.txt.gz?level=TRACE#_2017-01-04_00_19_32_511
01:49:26 mriedem no longer an error
01:57:37 mriedem +2 on jay's fix now
02:14:03 openstackgerrit Matt Riedemann proposed openstack/nova: Make test_attach_interfaces work with use_neutron=True by default https://review.openstack.org/416001
02:14:03 openstackgerrit Matt Riedemann proposed openstack/nova: Make test_nova_manage pass with CONF.use_neutron=True by default https://review.openstack.org/416003
02:14:04 openstackgerrit Matt Riedemann proposed openstack/nova: Make test_user_data and test_multiple_create work with use_neutron=True https://review.openstack.org/415729
02:14:05 openstackgerrit Matt Riedemann proposed openstack/nova: Make test_floating_ip* pass with CONF.use_neutron=True by default https://review.openstack.org/415999
02:14:05 openstackgerrit Matt Riedemann proposed openstack/nova: Stub out os_vif.unplug in libvirt instance destroy tests https://review.openstack.org/416002
02:14:05 openstackgerrit Matt Riedemann proposed openstack/nova: Make test_metadata pass with CONF.use_neutron=True by default https://review.openstack.org/416004
02:14:06 openstackgerrit Matt Riedemann proposed openstack/nova: Make test_security_group_default_rules work with use_neutron=True by default https://review.openstack.org/415767
02:14:06 openstackgerrit Matt Riedemann proposed openstack/nova: Make last remaining unit tests work with Neutron by default https://review.openstack.org/416006
02:14:07 openstackgerrit Matt Riedemann proposed openstack/nova: Make test_virtual_interfaces work with CONF.use_neutron=True by default https://review.openstack.org/415732
02:14:07 openstackgerrit Matt Riedemann proposed openstack/nova: Make test_server_usage work with CONF.use_neutron=True by default https://review.openstack.org/415770
02:14:08 openstackgerrit Matt Riedemann proposed openstack/nova: Make several API unit tests pass with CONF.use_neutron=True by default https://review.openstack.org/415771
02:14:08 openstackgerrit Matt Riedemann proposed openstack/nova: Make test_quota work with CONF.use_neutron=True by default https://review.openstack.org/415707
02:14:09 openstackgerrit Matt Riedemann proposed openstack/nova: Make test_security_groups work with CONF.use_neutron=True by default https://review.openstack.org/415747
02:14:09 openstackgerrit Matt Riedemann proposed openstack/nova: Enable Neutron by default https://review.openstack.org/416083
02:14:10 openstackgerrit Matt Riedemann proposed openstack/nova: Make test_tenant_networks pass with CONF.use_neutron=True by default https://review.openstack.org/415755
02:14:10 openstackgerrit Matt Riedemann proposed openstack/nova: Make test_compute pass with CONF.use_neutron=True by default https://review.openstack.org/399978
02:15:05 jlvillal mriedem, Thanks :)
02:15:13 openstackgerrit Matt Riedemann proposed openstack/nova: Enable Neutron by default https://review.openstack.org/416083
02:15:13 openstackgerrit Matt Riedemann proposed openstack/nova: Make last remaining unit tests work with Neutron by default https://review.openstack.org/416006
02:36:19 openstackgerrit Matt Riedemann proposed openstack/nova: Add support matrix for attach and detach interfaces https://review.openstack.org/409967
02:40:51 mtreinish mriedem: ^^^ did johnthetubaguy ever get that mapping to tempest tests (or config options) started?
02:41:12 mriedem mtreinish: there are patches...and a bp
02:41:13 mriedem sec
02:41:28 mriedem mtreinish: https://blueprints.launchpad.net/nova/+spec/feature-classification-ocata
02:41:39 mriedem i think raj_singh and other people from OSIC have been working on it moastly
02:41:41 mriedem *mostly
02:41:43 mriedem and sfinucan
02:42:05 mtreinish cool, I'll take a look
03:04:15 openstackgerrit zhangyujun proposed openstack/nova: Update IronicDriver return hypervisor_type https://review.openstack.org/379968

Earlier   Later