| opendevreview | Qiu Fossen proposed openstack/nova master: Allow migrating PMEM's data https://review.opendev.org/c/openstack/nova/+/802225 | 06:54 |
|---|---|---|
| qiujunting | sean-k-mooney I have updated this commit:https://review.opendev.org/c/openstack/nova/+/802225 | 08:09 |
| *** tosky is now known as Guest7142 | 13:37 | |
| *** tosky_ is now known as tosky | 13:37 | |
| opendevreview | Federico Ressi proposed openstack/nova master: Debug Nova APIs call failures https://review.opendev.org/c/openstack/nova/+/806683 | 13:55 |
| opendevreview | Nicolas Parquet proposed openstack/nova master: Add v2.91 microversion, allowing @ and dot (.) characters in keypair name https://review.opendev.org/c/openstack/nova/+/781076 | 14:02 |
| opendevreview | Nicolas Parquet proposed openstack/nova master: Add v2.91 microversion, allowing @ and dot (.) characters in keypair name https://review.opendev.org/c/openstack/nova/+/781076 | 14:06 |
| opendevreview | Balazs Gibizer proposed openstack/placement master: Spec: support any trait in allocation candidates https://review.opendev.org/c/openstack/placement/+/649992 | 16:32 |
| opendevreview | Balazs Gibizer proposed openstack/placement master: Spec: support mixing required traits with any traits https://review.opendev.org/c/openstack/placement/+/649368 | 16:32 |
| *** tbachman is now known as Guest7155 | 16:35 | |
| *** tbachman is now known as Guest7156 | 16:38 | |
| gibi | bauzas: what is you way to mark placement bugs in storyboard triaged or incomplete? | 16:44 |
| bauzas | good question, I f*** don't know | 16:44 |
| gibi | there is invalid state, so I might use that for incomplete | 16:45 |
| gibi | or I can tag it with incomplete | 16:45 |
| gibi | tag | 16:45 |
| bauzas | cool | 16:46 |
| bauzas | context ? | 16:46 |
| gibi | https://storyboard.openstack.org/#!/story/2009315 | 16:46 |
| gibi | I need more info to try to reporduce | 16:46 |
| gibi | I'm not even sure what generate file the reporter referring to | 16:46 |
| bauzas | gosh, even hyperlinks don't work | 16:53 |
| gibi | melwitt: hi! I think we fixed https://storyboard.openstack.org/#!/story/2009178 isn't it? if yes could you mark the story implemented? | 16:53 |
| gibi | bauzas: where? | 16:54 |
| bauzas | gibi: https://storyboard.openstack.org/#!/story/2009315 redirects me to my main board | 16:54 |
| gibi | bauzas: that is interesting | 16:54 |
| gibi | bauzas: such links tent to work for me | 16:55 |
| gibi | (it worked now in an private window too) | 16:55 |
| bauzas | fun, it works with private mode | 16:55 |
| gibi | hahh | 16:56 |
| bauzas | hmmm, now it works | 16:56 |
| gibi | storyboard knows you don't like itr | 16:56 |
| bauzas | I just clicked on "Stories" before calling the URL again | 16:56 |
| gibi | bauzas: could you hit https://review.opendev.org/c/openstack/placement/+/800611 it is an easy fix | 17:02 |
| bauzas | gibi: ack | 17:04 |
| gibi | thanks | 17:05 |
| melwitt | gibi: yes I will check on it and update it, thanks for heads up | 17:09 |
| gibi | thanks | 17:14 |
| opendevreview | Balazs Gibizer proposed openstack/placement master: Spec: support any trait in allocation candidates https://review.opendev.org/c/openstack/placement/+/649992 | 17:23 |
| opendevreview | Balazs Gibizer proposed openstack/placement master: Add yoga spec directory https://review.opendev.org/c/openstack/placement/+/819660 | 17:23 |
| opendevreview | Balazs Gibizer proposed openstack/placement master: Spec: support mixing required traits with any traits https://review.opendev.org/c/openstack/placement/+/649368 | 17:24 |
| gibi | bauzas: I did a couple of triage and invalidation of placement stories, the incomplete ones just tagged but not marked as invalid so they are still in the primary list | 17:32 |
| bauzas | ++ | 17:32 |
| gibi | it seems cdent used rfe and bug tags, so I did that too | 17:32 |
| gibi | and now I leave for today :) | 17:32 |
| dansmith | gmann: lbragstad: I had to add this to our functional fixture to fix the fails on my revert patch: https://termbin.com/3v6a | 17:35 |
| dansmith | but I'm not sure why, as we should still be passing with the legacy rules in place for functional, AFAIK | 17:35 |
| dansmith | I thing that ^ change is appropriate anyway, just so we're mocking keystone like we expect in the real world, but... I'm not sure why it's suddenly necessary | 17:36 |
| sean-k-mooney | hum "echo just testing! | nc termbin.com 9999" that is a userful site to knowabout | 17:36 |
| lbragstad | that change makes sense\ | 17:36 |
| lbragstad | from a role implication standpoint | 17:36 |
| gmann | dansmith: for unit test this is done in base context https://github.com/openstack/nova/blob/master/nova/tests/unit/policies/base.py#L74 | 17:37 |
| gmann | so you mean for functional tests right | 17:37 |
| gmann | dansmith: ohk, got it. this one | 17:38 |
| gmann | https://bddca1d27ef6e7362ec8-bff4e0c7dcef4e0ba8c46a21501c0936.ssl.cf5.rackcdn.com/816206/9/check/nova-tox-functional-py39/f17f814/testr_results.html | 17:38 |
| dansmith | gmann: yeah functional | 17:38 |
| gmann | +1 | 17:38 |
| dansmith | I have to jump on a call in a sec | 17:38 |
| dansmith | with that patch applied, I pass all those functional fails, I just don't know why I need it | 17:38 |
| gmann | dansmith: I do not think we run functional test on real policy instead on fake policy so it should not fail but I will be able to check tomorrow. | 17:40 |
| dansmith | gmann: hmm, okay then I don't know why it would have changed at all then | 18:47 |
| *** tosky is now known as Guest7167 | 20:17 | |
| *** tosky_ is now known as tosky | 20:17 | |
| opendevreview | Merged openstack/placement master: Enable HTTPProxyToWSGI middleware to find actual client ips https://review.opendev.org/c/openstack/placement/+/800611 | 21:54 |
| *** tosky_ is now known as tosky | 21:57 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!