[openstack-dev] [horizon] [heat-dashboard] Horizon plugin settings for new xstatic modules

Akihiro Motoki amotoki at gmail.com
Tue Mar 20 03:58:40 UTC 2018


Hi Kaz,

These repositories are under horizon project. It looks better to keep the
current core team.
It potentially brings some confusion if we treat some horizon plugin team
specially.
Reviewing xstatic repos would be a small burden, wo I think it would work
without problem even if only horizon-core can approve xstatic reviews.


2018-03-20 10:02 GMT+09:00 Kaz Shinohara <ksnhr.tech at gmail.com>:

> Hi Ivan, Horizon folks,
>
>
> Now totally 8 xstatic-** repos for heat-dashboard have been landed.
>
> In project-config for them, I've set same acl-config as the existing
> xstatic repos.
> It means only "xstatic-core" can manage the newly created repos on gerrit.
> Could you kindly add "heat-dashboard-core" into "xstatic-core" like as
> what horizon-core is doing ?
>
> xstatic-core
> https://review.openstack.org/#/admin/groups/385,members
>
> heat-dashboard-core
> https://review.openstack.org/#/admin/groups/1844,members
>
> Of course, we will surely touch only what we made, just would like to
> manage them smoothly by ourselves.
> In case we need to touch the other ones, will ask Horizon team for help.
>
> Thanks in advance.
>
> Regards,
> Kaz
>
>
> 2018-03-14 15:12 GMT+09:00 Xinni Ge <xinni.ge1990 at gmail.com>:
> > Hi Horizon Team,
> >
> > I reported a bug about lack of ``ADD_XSTATIC_MODULES`` plugin option,
> >  and submitted a patch for it.
> > Could you please help to review the patch.
> >
> > https://bugs.launchpad.net/horizon/+bug/1755339
> > https://review.openstack.org/#/c/552259/
> >
> > Thank you very much.
> >
> > Best Regards,
> > Xinni
> >
> > On Tue, Mar 13, 2018 at 6:41 PM, Ivan Kolodyazhny <e0ne at e0ne.info>
> wrote:
> >>
> >> Hi Kaz,
> >>
> >> Thanks for cleaning this up. I put +1 on both of these patches
> >>
> >> Regards,
> >> Ivan Kolodyazhny,
> >> http://blog.e0ne.info/
> >>
> >> On Tue, Mar 13, 2018 at 4:48 AM, Kaz Shinohara <ksnhr.tech at gmail.com>
> >> wrote:
> >>>
> >>> Hi Ivan & Horizon folks,
> >>>
> >>>
> >>> Now we are submitting a couple of patches to have the new xstatic
> >>> modules.
> >>> Let me request you to have review the following patches.
> >>> We need Horizon PTL's +1 to move these forward.
> >>>
> >>> project-config
> >>> https://review.openstack.org/#/c/551978/
> >>>
> >>> governance
> >>> https://review.openstack.org/#/c/551980/
> >>>
> >>> Thanks in advance:)
> >>>
> >>> Regards,
> >>> Kaz
> >>>
> >>>
> >>> 2018-03-12 20:00 GMT+09:00 Radomir Dopieralski <openstack at sheep.art.pl
> >:
> >>> > Yes, please do that. We can then discuss in the review about
> technical
> >>> > details.
> >>> >
> >>> > On Mon, Mar 12, 2018 at 2:54 AM, Xinni Ge <xinni.ge1990 at gmail.com>
> >>> > wrote:
> >>> >>
> >>> >> Hi, Akihiro
> >>> >>
> >>> >> Thanks for the quick reply.
> >>> >>
> >>> >> I agree with your opinion that BASE_XSTATIC_MODULES should not be
> >>> >> modified.
> >>> >> It is much better to enhance horizon plugin settings,
> >>> >>  and I think maybe there could be one option like
> ADD_XSTATIC_MODULES.
> >>> >> This option adds the plugin's xstatic files in STATICFILES_DIRS.
> >>> >> I am considering to add a bug report to describe it at first, and
> give
> >>> >> a
> >>> >> patch later maybe.
> >>> >> Is that ok with the Horizon team?
> >>> >>
> >>> >> Best Regards.
> >>> >> Xinni
> >>> >>
> >>> >> On Fri, Mar 9, 2018 at 11:47 PM, Akihiro Motoki <amotoki at gmail.com>
> >>> >> wrote:
> >>> >>>
> >>> >>> Hi Xinni,
> >>> >>>
> >>> >>> 2018-03-09 12:05 GMT+09:00 Xinni Ge <xinni.ge1990 at gmail.com>:
> >>> >>> > Hello Horizon Team,
> >>> >>> >
> >>> >>> > I would like to hear about your opinions about how to add new
> >>> >>> > xstatic
> >>> >>> > modules to horizon settings.
> >>> >>> >
> >>> >>> > As for Heat-dashboard project embedded 3rd-party files issue,
> >>> >>> > thanks
> >>> >>> > for
> >>> >>> > your advices in Dublin PTG, we are now removing them and
> >>> >>> > referencing as
> >>> >>> > new
> >>> >>> > xstatic-* libs.
> >>> >>>
> >>> >>> Thanks for moving this forward.
> >>> >>>
> >>> >>> > So we installed the new xstatic files (not uploaded as openstack
> >>> >>> > official
> >>> >>> > repos yet) in our development environment now, but hesitate to
> >>> >>> > decide
> >>> >>> > how to
> >>> >>> > add the new installed xstatic lib path to STATICFILES_DIRS in
> >>> >>> > openstack_dashboard.settings so that the static files could be
> >>> >>> > automatically
> >>> >>> > collected by *collectstatic* process.
> >>> >>> >
> >>> >>> > Currently Horizon defines BASE_XSTATIC_MODULES in
> >>> >>> > openstack_dashboard/utils/settings.py and the relevant static
> fils
> >>> >>> > are
> >>> >>> > added
> >>> >>> > to STATICFILES_DIRS before it updates any Horizon plugin
> dashboard.
> >>> >>> > We may want new plugin setting keywords ( something similar to
> >>> >>> > ADD_JS_FILES)
> >>> >>> > to update horizon XSTATIC_MODULES (or directly update
> >>> >>> > STATICFILES_DIRS).
> >>> >>>
> >>> >>> IMHO it is better to allow horizon plugins to add xstatic modules
> >>> >>> through horizon plugin settings. I don't think it is a good idea to
> >>> >>> add a new entry in BASE_XSTATIC_MODULES based on horizon plugin
> >>> >>> usages. It makes difficult to track why and where a xstatic module
> in
> >>> >>> BASE_XSTATIC_MODULES is used.
> >>> >>> Multiple horizon plugins can add a same entry, so horizon code to
> >>> >>> handle plugin settings should merge multiple entries to a single
> one
> >>> >>> hopefully.
> >>> >>> My vote is to enhance the horizon plugin settings.
> >>> >>>
> >>> >>> Akihiro
> >>> >>>
> >>> >>> >
> >>> >>> > Looking forward to hearing any suggestions from you guys, and
> >>> >>> > Best Regards,
> >>> >>> >
> >>> >>> > Xinni Ge
> >>> >>> >
> >>> >>> >
> >>> >>> >
> >>> >>> > ____________________________________________________________
> ______________
> >>> >>> > OpenStack Development Mailing List (not for usage questions)
> >>> >>> > Unsubscribe:
> >>> >>> > OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
> >>> >>> > http://lists.openstack.org/cgi-bin/mailman/listinfo/
> openstack-dev
> >>> >>> >
> >>> >>>
> >>> >>>
> >>> >>>
> >>> >>> ____________________________________________________________
> ______________
> >>> >>> OpenStack Development Mailing List (not for usage questions)
> >>> >>> Unsubscribe:
> >>> >>> OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
> >>> >>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
> >>> >>
> >>> >>
> >>> >>
> >>> >>
> >>> >> --
> >>> >> 葛馨霓 Xinni Ge
> >>> >>
> >>> >>
> >>> >> ____________________________________________________________
> ______________
> >>> >> OpenStack Development Mailing List (not for usage questions)
> >>> >> Unsubscribe:
> >>> >> OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
> >>> >> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
> >>> >>
> >>> >
> >>> >
> >>> >
> >>> > ____________________________________________________________
> ______________
> >>> > OpenStack Development Mailing List (not for usage questions)
> >>> > Unsubscribe:
> >>> > OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
> >>> > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
> >>> >
> >>>
> >>>
> >>> ____________________________________________________________
> ______________
> >>> OpenStack Development Mailing List (not for usage questions)
> >>> Unsubscribe:
> >>> OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
> >>> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
> >>
> >>
> >>
> >> ____________________________________________________________
> ______________
> >> OpenStack Development Mailing List (not for usage questions)
> >> Unsubscribe: OpenStack-dev-request at lists.openstack.org?subject:
> unsubscribe
> >> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
> >>
> >
> >
> >
> > --
> > 葛馨霓 Xinni Ge
> >
> > ____________________________________________________________
> ______________
> > OpenStack Development Mailing List (not for usage questions)
> > Unsubscribe: OpenStack-dev-request at lists.openstack.org?subject:
> unsubscribe
> > http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
> >
>
> __________________________________________________________________________
> OpenStack Development Mailing List (not for usage questions)
> Unsubscribe: OpenStack-dev-request at lists.openstack.org?subject:unsubscribe
> http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstack.org/pipermail/openstack-dev/attachments/20180320/23fe98d7/attachment.html>


More information about the OpenStack-dev mailing list