Re: [foreman-dev] Moving foreman-tasks to the core: the plan

2017-01-27 Thread Ivan Necas
On Wed, Jan 25, 2017 at 12:38 PM, Lukas Zapletal wrote: > I like the idea of having plain ActiveJob for simple background tasks > and providing tasks/dynflow for complex orchestration processes (also > available via ActiveJob API). While I understand whole motivation why > dynflow/task was created

Re: [foreman-dev] Moving foreman-tasks to the core: the plan

2017-01-25 Thread Lukas Zapletal
I like the idea of having plain ActiveJob for simple background tasks and providing tasks/dynflow for complex orchestration processes (also available via ActiveJob API). While I understand whole motivation why dynflow/task was created in the first place (providing a way to do one-way orchestrations

Re: [foreman-dev] Moving foreman-tasks to the core: the plan

2017-01-24 Thread Ivan Necas
Ohad Levy writes: > On Tue, Jan 24, 2017 at 9:51 AM, Ivan Necas wrote: > >> Ewoud Kohl van Wijngaarden writes: >> >> > On Tue, Jan 24, 2017 at 12:10:34AM +0100, Ivan Necas wrote: >> >> Ohad Levy writes: >> >> >> >> > On Fri, Jan 20, 2017 at 7:14 PM, Lukas Zapletal >> wrote: >> >> > >> >> >> W

Re: [foreman-dev] Moving foreman-tasks to the core: the plan

2017-01-24 Thread Ohad Levy
On Tue, Jan 24, 2017 at 9:51 AM, Ivan Necas wrote: > Ewoud Kohl van Wijngaarden writes: > > > On Tue, Jan 24, 2017 at 12:10:34AM +0100, Ivan Necas wrote: > >> Ohad Levy writes: > >> > >> > On Fri, Jan 20, 2017 at 7:14 PM, Lukas Zapletal > wrote: > >> > > >> >> Why we haven't considered ActiveJ

Re: [foreman-dev] Moving foreman-tasks to the core: the plan

2017-01-23 Thread Ivan Necas
Ewoud Kohl van Wijngaarden writes: > On Tue, Jan 24, 2017 at 12:10:34AM +0100, Ivan Necas wrote: >> Ohad Levy writes: >> >> > On Fri, Jan 20, 2017 at 7:14 PM, Lukas Zapletal wrote: >> > >> >> Why we haven't considered ActiveJob API in the first place? This looks >> >> like ideal solution, easy

Re: [foreman-dev] Moving foreman-tasks to the core: the plan

2017-01-23 Thread Ewoud Kohl van Wijngaarden
On Tue, Jan 24, 2017 at 12:10:34AM +0100, Ivan Necas wrote: > Ohad Levy writes: > > > On Fri, Jan 20, 2017 at 7:14 PM, Lukas Zapletal wrote: > > > >> Why we haven't considered ActiveJob API in the first place? This looks > >> like ideal solution, easy development and documented API. > >> > >> Ca

Re: [foreman-dev] Moving foreman-tasks to the core: the plan

2017-01-23 Thread Ivan Necas
Ohad Levy writes: > On Fri, Jan 20, 2017 at 7:14 PM, Lukas Zapletal wrote: > >> Why we haven't considered ActiveJob API in the first place? This looks >> like ideal solution, easy development and documented API. >> >> Can't ActiveJob help us with memory issues we have with external task >> execu

Re: [foreman-dev] Moving foreman-tasks to the core: the plan

2017-01-22 Thread Ohad Levy
On Fri, Jan 20, 2017 at 7:14 PM, Lukas Zapletal wrote: > Why we haven't considered ActiveJob API in the first place? This looks > like ideal solution, easy development and documented API. > > Can't ActiveJob help us with memory issues we have with external task > executor in Katello? I mean what

Re: [foreman-dev] Moving foreman-tasks to the core: the plan

2017-01-22 Thread Ivan Necas
Lukas Zapletal writes: > Why we haven't considered ActiveJob API in the first place? This looks > like ideal solution, easy development and documented API. > > Can't ActiveJob help us with memory issues we have with external task > executor in Katello? I mean what if we run background tasks insid

Re: [foreman-dev] Moving foreman-tasks to the core: the plan

2017-01-20 Thread Lukas Zapletal
Why we haven't considered ActiveJob API in the first place? This looks like ideal solution, easy development and documented API. Can't ActiveJob help us with memory issues we have with external task executor in Katello? I mean what if we run background tasks inside passenger process by default for

Re: [foreman-dev] Moving foreman-tasks to the core: the plan

2017-01-20 Thread Marek Hulán
On pátek 20. ledna 2017 2:51:06 CET Ewoud Kohl van Wijngaarden wrote: > On Thu, Jan 19, 2017 at 06:04:58PM +0100, Ivan Necas wrote: > > Lukas Zapletal writes: > > >> I'm not sure I follow what you mean by administrative tasks. Note that > > >> reports > > >> import and puppet envs import are core

Re: [foreman-dev] Moving foreman-tasks to the core: the plan

2017-01-19 Thread Ewoud Kohl van Wijngaarden
On Thu, Jan 19, 2017 at 06:04:58PM +0100, Ivan Necas wrote: > Lukas Zapletal writes: > > >> > >> I'm not sure I follow what you mean by administrative tasks. Note that > >> reports > >> import and puppet envs import are core actiones that now run as a foreman > >> task > >> (both synchronous and

Re: [foreman-dev] Moving foreman-tasks to the core: the plan

2017-01-19 Thread Ivan Necas
Lukas Zapletal writes: >> >> I'm not sure I follow what you mean by administrative tasks. Note that >> reports >> import and puppet envs import are core actiones that now run as a foreman >> task >> (both synchronous and asynchronous variants). By making the UI part >> optional >> users would not

Re: [foreman-dev] Moving foreman-tasks to the core: the plan

2017-01-05 Thread Lukas Zapletal
> > I'm not sure I follow what you mean by administrative tasks. Note that > reports > import and puppet envs import are core actiones that now run as a foreman > task > (both synchronous and asynchronous variants). By making the UI part > optional > users would not be able to monitor their progres

Re: [foreman-dev] Moving foreman-tasks to the core: the plan

2017-01-05 Thread Marek Hulán
Few comments below On čtvrtek 5. ledna 2017 15:25:47 CET Lukas Zapletal wrote: > Is it only the Rails App that depends on Foreman Core? What's reasonable > plan to me is: > > a) Extract core part that does not depend on Foreman core into a simple gem > (that's what you propose I guess) and make i

Re: [foreman-dev] Moving foreman-tasks to the core: the plan

2017-01-05 Thread Lukas Zapletal
Is it only the Rails App that depends on Foreman Core? What's reasonable plan to me is: a) Extract core part that does not depend on Foreman core into a simple gem (that's what you propose I guess) and make it a hard dependency of core. b) The rest of foreman-tasks (Rails App/Engine - administrati

Re: [foreman-dev] Moving foreman-tasks to the core: the plan

2017-01-04 Thread Eric D Helms
On Wed, Jan 4, 2017 at 4:29 AM, Ivan Necas wrote: > Hello friends, > > You've might noticed we started using foreman-tasks inside the > foreman-core since [1] was merged. Unfortunately, we hit issues in the > packaging phase and the proposed workarounds were not accepted ([2] > [3]). > > Based on

Re: [foreman-dev] Moving foreman-tasks to the core: the plan

2017-01-04 Thread Marek Hulán
+1 -- Marek On středa 4. ledna 2017 10:29:19 CET Ivan Necas wrote: > Hello friends, > > You've might noticed we started using foreman-tasks inside the > foreman-core since [1] was merged. Unfortunately, we hit issues in the > packaging phase and the proposed workarounds were not accepted ([2] >

Re: [foreman-dev] Moving foreman-tasks to the core: the plan

2017-01-04 Thread Michael Moll
Hi, On Wed, Jan 04, 2017 at 10:29:19AM +0100, Ivan Necas wrote: > In the mean time, I would like to kindly ask for trying to > find an acceptable workaround in [2] and [3] until this is done. > > [...] > > [2] - https://github.com/theforeman/foreman-packaging/pull/1436 > [3] - https://github.com/

[foreman-dev] Moving foreman-tasks to the core: the plan

2017-01-04 Thread Ivan Necas
Hello friends, You've might noticed we started using foreman-tasks inside the foreman-core since [1] was merged. Unfortunately, we hit issues in the packaging phase and the proposed workarounds were not accepted ([2] [3]). Based on various discussions, as well as on the fact that foreman-tasks is