| *** tmazur has quit IRC | 00:20 | |
| *** benj_ has quit IRC | 01:37 | |
| *** benj_ has joined #openstack-horizon | 02:01 | |
| *** vishalmanchanda has joined #openstack-horizon | 06:15 | |
| *** dklyle has quit IRC | 06:16 | |
| opendevreview | Vishal Manchanda proposed openstack/horizon stable/wallaby: doc: Update our IRC server to OFTC https://review.opendev.org/c/openstack/horizon/+/794629 | 06:33 |
|---|---|---|
| *** josecastroleon has joined #openstack-horizon | 06:34 | |
| *** lvdombrkr has joined #openstack-horizon | 06:34 | |
| opendevreview | Vishal Manchanda proposed openstack/horizon stable/victoria: doc: Update our IRC server to OFTC https://review.opendev.org/c/openstack/horizon/+/794630 | 06:34 |
| *** ralonsoh has joined #openstack-horizon | 06:43 | |
| ralonsoh | hi folks, can you help me with this patch https://review.opendev.org/c/openstack/horizon/+/794263 ? | 06:44 |
| ralonsoh | logs: https://5eff182f98534d7a4f80-3f28c2c0c6b192e8ac5a4ae954bec2d0.ssl.cf2.rackcdn.com/794263/1/check/horizon-integration-tests/ea355aa/job-output.txt | 06:44 |
| ralonsoh | I'm checking the Neutron logs and I don't see anything wrong there | 06:44 |
| ralonsoh | do you know where exactly those tests are failing? | 06:45 |
| vishalmanchanda | ralonsoh: hi basically these tests are failing. | 06:46 |
| vishalmanchanda | https://5eff182f98534d7a4f80-3f28c2c0c6b192e8ac5a4ae954bec2d0.ssl.cf2.rackcdn.com/794263/1/check/horizon-integration-tests/ea355aa/tox/test_reports/ | 06:46 |
| amotoki | ralonsoh: vishalmanchanda: I also started to investigate the integration test failure in detail today. | 06:47 |
| vishalmanchanda | ralonsoh: somehow devstack recent changes the Neutron default backend to OVN causes the failure. | 06:47 |
| vishalmanchanda | ralonsoh: If I changed backend to OVN for integration job it pass https://review.opendev.org/c/openstack/horizon/+/788673 | 06:48 |
| ralonsoh | vishalmanchanda, but did you identify the issue? I mean, what command is failing? | 06:49 |
| opendevreview | OpenStack Proposal Bot proposed openstack/horizon master: Imported Translations from Zanata https://review.opendev.org/c/openstack/horizon/+/794712 | 06:51 |
| vishalmanchanda | ralonsoh: I think not able to delete router and router interfaces in the integration tests. | 06:51 |
| opendevreview | OpenStack Proposal Bot proposed openstack/horizon stable/victoria: Imported Translations from Zanata https://review.opendev.org/c/openstack/horizon/+/794713 | 06:51 |
| opendevreview | Vishal Manchanda proposed openstack/horizon master: Explicitly set horizon integration jobs to use ML2/OVS https://review.opendev.org/c/openstack/horizon/+/788673 | 06:53 |
| amotoki | ralonsoh: vishalmanchanda: perhaps I identify what is the cause now. | 06:56 |
| ralonsoh | amotoki, what did you see? | 06:57 |
| ralonsoh | I can | 06:57 |
| amotoki | ralonsoh: vishalmanchanda: the reason is the horizon integration test check the status of a new router interface too strict. | 06:57 |
| ralonsoh | how is that? | 06:57 |
| amotoki | in L3-agent case, a new router interface will be created with "Down" status | 06:57 |
| ralonsoh | self.assertTrue( | 06:58 |
| ralonsoh | interfaces_page.is_interface_status(interface_name, 'Down')) | 06:58 |
| ralonsoh | ahhhhhh | 06:58 |
| amotoki | yeah | 06:58 |
| vishalmanchanda | ohk. | 06:58 |
| amotoki | I am looking for URL in opendev.org but you are faster | 06:58 |
| amotoki | in case of OVN, a new port has "ACTIVE" status | 06:58 |
| amotoki | this is a page capture https://a51aacd792b2e969869d-8e1073b04daf76ff4b20d216b286b6f9.ssl.cf2.rackcdn.com/788673/2/check/horizon-integration-tests/d089bf4/tox/test_reports/TestRouters.test_router_add_delete_interface/page.html | 06:58 |
| ralonsoh | right, status is active | 06:59 |
| ralonsoh | incredible catch | 06:59 |
| ralonsoh | very good, for sure | 06:59 |
| amotoki | as far as I checked, subsequent tests fail due to the failuree in the first test. | 06:59 |
| vishalmanchanda | amotoki: yes right. | 06:59 |
| amotoki | ralonsoh: I will take care of the fix. | 06:59 |
| ralonsoh | amotoki, thanks a lot! | 07:00 |
| amotoki | ralonsoh: thanks for checking it anyway. I could not take enough time to look into it till today :( | 07:00 |
| *** tosky has joined #openstack-horizon | 07:23 | |
| opendevreview | Akihiro Motoki proposed openstack/horizon master: integration tests: Relax router interface status check https://review.opendev.org/c/openstack/horizon/+/794718 | 07:24 |
| amotoki | vishalmanchanda: let's see if anything changes in the above patch ^ | 07:25 |
| *** rdopiera has joined #openstack-horizon | 07:32 | |
| *** CeeMac has joined #openstack-horizon | 08:18 | |
| vishalmanchanda | rdopiera: hi, Could you please take a look at https://review.opendev.org/c/openstack/horizon/+/794718 | 09:09 |
| *** e0ne has joined #openstack-horizon | 09:30 | |
| opendevreview | Vishal Manchanda proposed openstack/horizon stable/ussuri: doc: Update our IRC server to OFTC https://review.opendev.org/c/openstack/horizon/+/794758 | 09:33 |
| amotoki | vishalmanchanda: regarding on your backport of doc/source/contributor/contributing.rst, did you see http://lists.openstack.org/pipermail/openstack-discuss/2021-June/022867.html ? | 09:54 |
| amotoki | vishalmanchanda: according to this, global redirect will be setup so it will be no longerr referred. do we really need this backport? | 09:55 |
| amotoki | vishalmanchanda: on the other hand, the first item says "Agreed to backport the changes as much as possible". I am not sure what we want. | 09:56 |
| amotoki | vishalmanchanda: I am not against the backport. I just would like to clarify the situation. | 09:57 |
| vishalmanchanda | amotoki: I didn't see any issue in backporting to stable branches. | 10:00 |
| vishalmanchanda | amotoki: So you want to remove doc/source/contributor/contributing.rst from stable branches | 10:00 |
| vishalmanchanda | amotoki: I saw that thread that' s why I purpose backport to stable branches "Agreed to backport the changes as much as possible". | 10:02 |
| amotoki | vishalmanchanda: from what I read from the email, a backport would be okay. contributing.rst will have no meaning once the global redirect is setup,. After the redirect is setup, contributing.rst in stable branches will not be referred and a project can remove it for clarification (per project decision),. right? | 10:04 |
| vishalmanchanda | amotoki: yes. | 10:05 |
| amotoki | vishalmanchanda: okay, thanks | 10:06 |
| vishalmanchanda | amotoki: So you are in favor of removing ``contributing.rst`` from stable branches once global redirect is setup? | 10:07 |
| amotoki | vishalmanchanda: no, at the moment the global redirect is not setup yet, so I don't think we should drop 'contributing.rst' right now. | 10:08 |
| amotoki | vishalmanchanda: I think an option right now would be either of a backport or to wait for a while. I am okay with either. | 10:09 |
| amotoki | vishalmanchanda: I believe I understand the situation correctly and the backport itself is simple enough, so I am okay to approve the bacports. | 10:09 |
| opendevreview | Vishal Manchanda proposed openstack/horizon master: [DNR] Test horizon with latest XStatic-Angular version https://review.opendev.org/c/openstack/horizon/+/794774 | 10:33 |
| opendevreview | Merged openstack/horizon master: integration tests: Relax router interface status check https://review.opendev.org/c/openstack/horizon/+/794718 | 10:40 |
| opendevreview | Merged openstack/horizon master: Imported Translations from Zanata https://review.opendev.org/c/openstack/horizon/+/794712 | 10:40 |
| opendevreview | Merged openstack/horizon stable/victoria: Imported Translations from Zanata https://review.opendev.org/c/openstack/horizon/+/794713 | 10:41 |
| opendevreview | Vadym Markov proposed openstack/horizon master: CSS fix makes "Delete item" button active https://review.opendev.org/c/openstack/horizon/+/793420 | 10:44 |
| opendevreview | Vadym Markov proposed openstack/horizon master: Fix wrong source when volume launched as instance https://review.opendev.org/c/openstack/horizon/+/793970 | 10:45 |
| *** e0ne has quit IRC | 13:02 | |
| opendevreview | Radomir Dopieralski proposed openstack/horizon master: Initial test of django sass processor https://review.opendev.org/c/openstack/horizon/+/794809 | 13:33 |
| *** tmazur has joined #openstack-horizon | 13:50 | |
| vmarkov | Question for discussion. In Admin-System-System Information-Compute Services we have "Last Updated" column, which always shows "0 minutes" because nova returns heartbeat timer on this report. However, documentation for novaclient shows example reports with reasonable update times and these examples haven't updated since 2016. From position of UX, it is reasonable to rename "Last Updated" column or remove it at all, because failed | 14:00 |
| vmarkov | heartbeat anyway raises alert | 14:00 |
| *** gmann is now known as gmann_afk | 14:40 | |
| *** dklyle has joined #openstack-horizon | 14:46 | |
| *** josecastroleon has quit IRC | 15:04 | |
| *** josecastroleon has joined #openstack-horizon | 15:05 | |
| *** gmann_afk is now known as gmann | 15:13 | |
| *** josecastroleon has quit IRC | 16:20 | |
| *** ralonsoh has quit IRC | 16:47 | |
| *** rdopiera has quit IRC | 17:41 | |
| *** lvdombrkr has quit IRC | 18:04 | |
| opendevreview | Merged openstack/horizon master: Default role checker should be case-insensitive. https://review.opendev.org/c/openstack/horizon/+/791301 | 18:45 |
| *** lvdombrkr has joined #openstack-horizon | 19:37 | |
| *** vishalmanchanda has quit IRC | 19:54 | |
| opendevreview | Tatiana Ovchinnikova proposed openstack/horizon master: Magic Search filter facet removal fix https://review.opendev.org/c/openstack/horizon/+/794899 | 20:51 |
| *** gmann is now known as gmann_afk | 21:51 | |
| *** tosky has quit IRC | 22:01 | |
| *** tmazur has quit IRC | 22:27 | |
| *** whoami-rajat has quit IRC | 23:40 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!