Sounds like that would simplify things. I like it.

On Tue, Jul 7, 2020 at 11:59 Craig Russell <apache....@gmail.com> wrote:

> Hi Sebb,
>
> Good job figuring out what the problem was (is).
>
> From my perspective, having emeritus under documents is somewhat
> arbitrary. There is no reason for the emeritus documents to have to live
> there, considering that they are all foundation. You could also make a case
> for moving member_apps under foundation since the member apps are probably
> the most personal-sensitive documents we hold.
>
> My recommended solution:
>
> mv documents/emeritus* foundation
>
> A bit of code tweaking to change repository.yml and we're done.
>
> If no objections within 72, we can proceed with this plan.
>
> Craig
>
>
> > On Jul 7, 2020, at 8:31 AM, sebb <seb...@gmail.com> wrote:
> >
> > On Tue, 7 Jul 2020 at 14:39, Craig Russell <apache....@gmail.com> wrote:
> >>
> >> Any new news on the failure in (emeritus) function?
> >
> > Whilst the changes are both to the same repository, they have different
> roots:
> >
> > ^foundation
> > and
> > ^documents
> >
> > Thus the common root is ^
> >
> > It looks like it may be necessary to have write access to the
> > top-level directory to allow the commit to succeed as a batch.
> > I doubt that would be granted, so it may be necessary to find a
> > different approach -- or move the documents?
> >
> > At present not even read-access is allowed at top-level.
> > If read access were sufficient, that might be allowed by Infra.
> >
> >> I don't want to step on any toes if anyone is still working on the
> error.
> >>
> >> Craig
> >>
> >>> On Jul 6, 2020, at 2:17 PM, Craig Russell <apache....@gmail.com>
> wrote:
> >>>
> >>> Hi Sebb,
> >>>
> >>> Thanks for that. Now there is a different problem with (Emeritus). See
> else thread.
> >>>
> >>> Craig
> >>>
> >>>> On Jul 6, 2020, at 1:17 PM, sebb <seb...@gmail.com> wrote:
> >>>>
> >>>> On Mon, 6 Jul 2020 at 18:44, Craig Russell <apache....@gmail.com>
> wrote:
> >>>>>
> >>>>> I merged the roster-emeritus branch and tried out some of the new
> features.
> >>>>>
> >>>>> 1. Request emeritus fails on line 70 of memstat.json.rb [1]
> >>>>> template, err =
> >>>>>  ASF::SVN.svn('cat', EMERITUS_TEMPLATE_URL, {env:env})
> >>>>
> >>>> AFAICT it must be env that is untainted, but not sure if so or where
> >>>> to fix this yet.
> >>>>
> >>>>>
> >>>>> 2. Move to emeritus status for a member with an outstanding request
> fails on line 21 of memstat.json.rb [2]
> >>>>> ASF::SVN.multiUpdate_ members_txt, message, env, _ do |text|
> >>>>
> >>>> Should be fixed; whimsy4 is on an old Ruby which does not have
> URI::File
> >>>>
> >>>>> Any ideas?
> >>>>>
> >>>>> Craig
> >>>>>
> >>>>> [1] {
> >>>>> "exception": "#<SecurityError: Insecure operation - spawn>",
> >>>>> "backtrace": [
> >>>>>  "/usr/local/rvm/rubies/ruby-2.4.1/lib/ruby/2.4.0/open3.rb:199:in
> `spawn'",
> >>>>>  "/usr/local/rvm/rubies/ruby-2.4.1/lib/ruby/2.4.0/open3.rb:199:in
> `popen_run'",
> >>>>>  "/usr/local/rvm/rubies/ruby-2.4.1/lib/ruby/2.4.0/open3.rb:95:in
> `popen3'",
> >>>>>  "/usr/local/rvm/rubies/ruby-2.4.1/lib/ruby/2.4.0/open3.rb:258:in
> `capture3'",
> >>>>>  "/x1/srv/whimsy/lib/whimsy/asf/svn.rb:387:in `svn'",
> >>>>>  "/x1/srv/whimsy/www/roster/views/actions/memstat.json.rb:70:in
> `_evaluate'",
> >>>>>  "/x1/srv/whimsy/www/roster/main.rb:204:in `block in <top
> (required)>'",
> >>>>>  "/x1/srv/whimsy/lib/whimsy/asf/rack.rb:223:in `call'",
> >>>>>  "/x1/srv/whimsy/lib/whimsy/asf/rack.rb:48:in `call'",
> >>>>>  "/x1/srv/whimsy/lib/whimsy/asf/rack.rb:200:in `call'",
> >>>>>  "/x1/srv/whimsy/lib/whimsy/asf/rack.rb:254:in `call'",
> >>>>>
> "/usr/local/rvm/gems/ruby-2.4.1/gems/passenger-6.0.2/src/ruby_supportlib/phusion_passenger/rack/thread_handler_extension.rb:97:in
> `process_request'",
> >>>>>
> "/usr/local/rvm/gems/ruby-2.4.1/gems/passenger-6.0.2/src/ruby_supportlib/phusion_passenger/request_handler/thread_handler.rb:157:in
> `accept_and_process_next_request'",
> >>>>>
> "/usr/local/rvm/gems/ruby-2.4.1/gems/passenger-6.0.2/src/ruby_supportlib/phusion_passenger/request_handler/thread_handler.rb:110:in
> `main_loop'",
> >>>>>
> "/usr/local/rvm/gems/ruby-2.4.1/gems/passenger-6.0.2/src/ruby_supportlib/phusion_passenger/request_handler.rb:415:in
> `block (3 levels) in start_threads'",
> >>>>>
> "/usr/local/rvm/gems/ruby-2.4.1/gems/passenger-6.0.2/src/ruby_supportlib/phusion_passenger/utils.rb:113:in
> `block in create_thread_and_abort_on_exception'"
> >>>>> ]
> >>>>>
> >>>>> [2] {
> >>>>> "exception": "#<NameError: uninitialized constant URI::File\nDid you
> mean?  File>",
> >>>>> "backtrace": [
> >>>>>  "/x1/srv/whimsy/lib/whimsy/asf/svn.rb:752:in `multiUpdate_'",
> >>>>>  "/x1/srv/whimsy/www/roster/views/actions/memstat.json.rb:21:in
> `_evaluate'",
> >>>>>  "/x1/srv/whimsy/www/roster/main.rb:204:in `block in <top
> (required)>'",
> >>>>>  "/x1/srv/whimsy/lib/whimsy/asf/rack.rb:223:in `call'",
> >>>>>  "/x1/srv/whimsy/lib/whimsy/asf/rack.rb:48:in `call'",
> >>>>>  "/x1/srv/whimsy/lib/whimsy/asf/rack.rb:200:in `call'",
> >>>>>  "/x1/srv/whimsy/lib/whimsy/asf/rack.rb:254:in `call'",
> >>>>>
> "/usr/local/rvm/gems/ruby-2.4.1/gems/passenger-6.0.2/src/ruby_supportlib/phusion_passenger/rack/thread_handler_extension.rb:97:in
> `process_request'",
> >>>>>
> "/usr/local/rvm/gems/ruby-2.4.1/gems/passenger-6.0.2/src/ruby_supportlib/phusion_passenger/request_handler/thread_handler.rb:157:in
> `accept_and_process_next_request'",
> >>>>>
> "/usr/local/rvm/gems/ruby-2.4.1/gems/passenger-6.0.2/src/ruby_supportlib/phusion_passenger/request_handler/thread_handler.rb:110:in
> `main_loop'",
> >>>>>
> "/usr/local/rvm/gems/ruby-2.4.1/gems/passenger-6.0.2/src/ruby_supportlib/phusion_passenger/request_handler.rb:415:in
> `block (3 levels) in start_threads'",
> >>>>>
> "/usr/local/rvm/gems/ruby-2.4.1/gems/passenger-6.0.2/src/ruby_supportlib/phusion_passenger/utils.rb:113:in
> `block in create_thread_and_abort_on_exception'"
> >>>>> ]
> >>>>> }
> >>>>> }
> >>>>> Craig L Russell
> >>>>> c...@apache.org
> >>>>>
> >>>
> >>> Craig L Russell
> >>> c...@apache.org
> >>>
> >>
> >> Craig L Russell
> >> c...@apache.org
> >>
>
> Craig L Russell
> c...@apache.org
>
> --
Matt Sicker <boa...@gmail.com>

Reply via email to