| Posted | Nick | Remark | |
|---|---|---|---|
| #openstack-nova - 2019-09-04 | |||
| 11:32:38 | brinzhang | sean-k-mooney, artom: I remember the unit test or functional test can run any linux env, depend on the devstack or openstack env is not necessary, is that? | |
| 11:32:56 | sean-k-mooney | brinzhang: correct | |
| 11:33:06 | sean-k-mooney | you do not need openstack or devstack to run them | |
| 11:33:08 | brinzhang | sean-k-mooney, artom: I checked to the master and run the test, also report the error | |
| 11:33:27 | sean-k-mooney | what os are you running on? | |
| 11:33:34 | brinzhang | Centos7 | |
| 11:33:48 | sean-k-mooney | brinzhang: you might want to clone a clean copy of nova and try it again | |
| 11:34:11 | sean-k-mooney | brinzhang: ok i tested on popos and it worked fine | |
| 11:34:19 | sean-k-mooney | i can check on rhel8 | |
| 11:34:31 | brinzhang | On Monday, It runs ok, but today I git pull | |
| 11:35:00 | sean-k-mooney | i know some test fail in rhel8 but i think that is due to the version of python it uses | |
| 11:35:53 | brinzhang | Emm, in my env, I was installed py27 and py37,the default is py27 | |
| 11:37:01 | sean-k-mooney | no i mean that using py36 on rhel8 some things fail that pass on py36 on pop_os/ubunutu | |
| 11:37:09 | brinzhang | Clone the clean code is so slowly, sometimes maybe disconnect, I don’t want to do that. | |
| 11:37:45 | sean-k-mooney | brinzhang: well i was only suggesting that since it might be related to unclean files in your working directory | |
| 11:38:02 | sean-k-mooney | have you checked there are no tempory files created by your ide/editor | |
| 11:38:13 | sean-k-mooney | im checking master on rhel8 now | |
| 11:38:51 | sean-k-mooney | i guess i can check with a centos container too | |
| 11:39:33 | brinzhang | I will check again in my working directory too, and try to clone it in another directory | |
| 11:40:00 | sean-k-mooney | you can do a local clone by the way | |
| 11:40:39 | sean-k-mooney | e.g. mkdir temp; cd temp; git clone ~/repos/nova; cd nova; tox -e ... | |
| 11:41:46 | sean-k-mooney | it passed for me on rhel8 | |
| 11:41:54 | sean-k-mooney | with no failurs at all | |
| 11:42:03 | sean-k-mooney | which is better then it normally does | |
| 11:42:38 | aspiers | sean-k-mooney: the issue with stestr and temporary editor files (e.g. from emacs) is fixed | |
| 11:43:08 | sean-k-mooney | aspiers: as of when | |
| 11:43:11 | aspiers | sean-k-mooney: https://github.com/mtreinish/stestr/issues/238 | |
| 11:43:26 | aspiers | 2 weeks ago | |
| 11:43:29 | sean-k-mooney | aspiers: i have hit its since march | |
| 11:43:30 | aspiers | or 3 | |
| 11:43:38 | aspiers | yeah you need to upgrade your stestr | |
| 11:43:45 | aspiers | 2.5.0 | |
| 11:44:02 | sean-k-mooney | maybe im just using whicever version tox installs | |
| 11:44:05 | aspiers | Turns out that reporting bugs sometimes helps get them fixed ;-) | |
| 11:44:14 | sean-k-mooney | although i have not recreate all my envs since then | |
| 11:44:24 | aspiers | stestr>=1.0.0 # Apache-2.0 | |
| 11:44:28 | aspiers | from test-requirements | |
| 11:44:36 | aspiers | we should bump that | |
| 11:44:36 | brinzhang | sean-k-mooney: yeah, cloning, but so slowly... | |
| 11:44:51 | sean-k-mooney | brinzhang: if your doing it locally it should be fast | |
| 11:45:00 | sean-k-mooney | e.g. the file system clone i suggested above | |
| 11:46:24 | brinzhang | git clone ~/repos/nova what do you mean? isnot use git clone https://review.opendev.org/openstack/nova clone to another directory? | |
| 11:47:28 | brinzhang | this is the clone speed "Receiving objects: 3% (21352/550825), 6.94 MiB | 27.00 KiB/s" | |
| 11:50:40 | openstackgerrit | Eric Fried proposed openstack/nova master: Bump min for oslo.service & .privsep to fix SIGHUP https://review.opendev.org/679974 | |
| 11:53:54 | openstackgerrit | Silvan Kaiser proposed openstack/nova stable/stein: Exec systemd-run without --user flag in Quobyte driver https://review.opendev.org/660705 | |
| 12:26:25 | sean-k-mooney | brinzhang: git can use filesystem urls | |
| 12:26:45 | sean-k-mooney | brinzhang: so if you have a copy of a git repo you can clone a second copy using the file sytem path | |
| 12:27:09 | sean-k-mooney | so for example i keep my dev repos under ~/repos | |
| 12:27:27 | sean-k-mooney | if i do "mkdir ~/temp; cd temp" | |
| 12:27:53 | efried | alex_xu, luyao: Are you around? | |
| 12:28:39 | sean-k-mooney | then do "git clone ~/repos/openstack/nova" | |
| 12:29:00 | sean-k-mooney | it give me a clean copy of the nova git tree in ~/temp/nova | |
| 12:29:37 | brinzhang | sean-k-moony: Clone completed in another directory, and run tox -e ok, you are right, maybe the tempory file cause this issue. | |
| 12:30:13 | luyao | efried: I'm here | |
| 12:30:36 | brinzhang | sean-k-moony: I know your mean now, and I do it like you were said :P | |
| 12:35:22 | efried | luyao: Do you need any help to make progress on the vpmem series at this point? Do you understand what dansmith is asking for on the bottom two patches? | |
| 12:38:11 | luyao | efried: I do have some questions on the patch anout DB change. | |
| 12:39:41 | efried | luyao: Okay. I don't have a good understanding of such things, but dansmith usually arrives on the scene in about 20 minutes. I know it's late for you... | |
| 12:41:17 | luyao | efried: Ii'll wait for him, and I was going to wait. haha :) | |
| 12:41:31 | efried | okay | |
| 12:49:10 | sean-k-mooney | brinzhang: for what it is worth. whenever i hit test issue like that i first create a second local clone and test on the clean repo with master and with my patch branch if i still hit the issue i then check in a conainter. | |
| 12:49:54 | openstackgerrit | Luyao Zhong proposed openstack/nova master: db: Add resources column in instance_extra table https://review.opendev.org/678447 | |
| 12:49:55 | openstackgerrit | Luyao Zhong proposed openstack/nova master: object: Introduce Resource and ResouceList objs https://review.opendev.org/678448 | |
| 12:49:55 | openstackgerrit | Luyao Zhong proposed openstack/nova master: Add resources dict into _Provider https://review.opendev.org/678449 | |
| 12:49:56 | openstackgerrit | Luyao Zhong proposed openstack/nova master: Retrieve the allocations early https://review.opendev.org/678450 | |
| 12:49:56 | openstackgerrit | Luyao Zhong proposed openstack/nova master: Claim resources in resource tracker https://review.opendev.org/678452 | |
| 12:49:57 | openstackgerrit | Luyao Zhong proposed openstack/nova master: libvirt: Enable driver discovering PMEM namespaces https://review.opendev.org/678453 | |
| 12:49:58 | openstackgerrit | Luyao Zhong proposed openstack/nova master: libvirt: report VPMEM resources by provider tree https://review.opendev.org/678454 | |
| 12:49:58 | openstackgerrit | Luyao Zhong proposed openstack/nova master: libvirt: Support VM creation with vpmems and vpmems cleanup https://review.opendev.org/678455 | |
| 12:49:59 | openstackgerrit | Luyao Zhong proposed openstack/nova master: Parse vpmem related flavor extra spec https://review.opendev.org/678456 | |
| 12:49:59 | openstackgerrit | Luyao Zhong proposed openstack/nova master: libvirt: Enable driver configuring PMEM namespaces https://review.opendev.org/679640 | |
| 12:50:00 | openstackgerrit | Luyao Zhong proposed openstack/nova master: Add functional tests for virtual persistent memory https://review.opendev.org/678470 | |
| 12:51:31 | brinzhang | sean-k-mooney: Good experience, got it. | |
| 13:09:46 | mriedem | bauzas: can you push this simple refactor through? https://review.opendev.org/#/c/679651/ | |
| 13:10:18 | bauzas | mriedem: sure thing | |
| 13:10:58 | luyao | dansmith: Are you around? | |
| 13:17:20 | bauzas | mriedem: gibi: I was paying attention to the qos-bandwith series but I'm a bit lost with it | |
| 13:17:26 | bauzas | gibi: what's the top change ? | |
| 13:17:29 | bauzas | https://review.opendev.org/#/c/679653/ ? | |
| 13:17:44 | bauzas | because I don't see yet changes for cold migrate | |
| 13:17:55 | openstackgerrit | sean mooney proposed openstack/nova master: multi numa nfv testing job https://review.opendev.org/679656 | |
| 13:17:56 | openstackgerrit | sean mooney proposed openstack/nova master: [DNM] test migration with pinning https://review.opendev.org/679754 | |
| 13:18:08 | mriedem | gibi: replied in https://review.opendev.org/#/c/656422/18/nova/compute/manager.py@2122 | |
| 13:18:30 | bauzas | nah nevermind https://review.opendev.org/#/c/679019/4 | |
| 13:18:39 | bauzas | gerrit fucks up with the series | |
| 13:18:48 | bauzas | there are some branches | |
| 13:18:51 | gibi | bauzas: this is the top https://review.opendev.org/#/c/679019/4 | |
| 13:19:07 | gibi | bauzas: but I will respin the whole soonish to fix mriedem's comments | |
| 13:19:11 | gibi | mriedem: looking | |
| 13:19:20 | bauzas | gibi: okay i can wait then | |
| 13:19:46 | mriedem | gibi: tl;dr i think you need the service version check in conductor after scheduler picks a dest and also a check in compute in case the compute rpc api versions are pinned in config | |
| 13:20:00 | mriedem | i.e. the compute services can be running train but configured to only send stein level messages during an upgrade | |
| 13:20:04 | mriedem | artom ran into this yesterday | |
| 13:20:36 | mriedem | both are temporary and can be removed in U | |
| 13:21:22 | brinzhang | mriedem: https://review.opendev.org/#/c/673133/14/nova/compute/api.py@4028, I was updated that comment, found the volume_bdm.save() report an "ObjectActionError: Object action obj_load_attr failed because: attribute id not lazy-loadable" | |
| 13:21:34 | brinzhang | mriedem: That why? | |
| 13:21:40 | gibi | mriedem: ack. I will put the checks in the conductor. Good point about that the service version is no enough | |
| 13:21:53 | mriedem | brinzhang: in unit tests or a tempest run? | |
| 13:22:08 | artom | *me meerkats up* | |
| 13:22:08 | mriedem | brinzhang: if unit tests, you need to mock the BlockDeviceMapping.save call | |