[Fedora Patch] kdump.service: Start kdump after network is online and remote fs is mounted

[WANG Chao]

Signed-off-by: WANG Chao 
---
 kdump.service | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/kdump.service b/kdump.service
index 07e97fa..55b7ca2 100644
--- a/kdump.service
+++ b/kdump.service
@@ -1,6 +1,6 @@
 [Unit]
 Description=Crash recovery kernel arming
-After=network.target
+After=network.target network-online.target remote-fs.target

[Service]
 Type=oneshot

[Dave Young]

On 08/15/2013 12:27 PM, WANG Chao wrote:
> Signed-off-by: WANG Chao
>
> kdump.service | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/kdump.service b/kdump.service
> index 07e97fa..55b7ca2 100644
> --- a/kdump.service
> +++ b/kdump.service
> @@ -1,6 +1,6 @@
>  [Unit]
>  Description=Crash recovery kernel arming
> -After=network.target
> +After=network.target network-online.target remote-fs.target
>
> [Service]
>  Type=oneshot

Can you resend it with proper changelog? what is the problem? why do we
need this and how do you fix it? Probably also how did you test it if
necessary.

Nit pick: I think we can assume below tag instead of [Fedora patch]:

[PATCH] for rawhide fix
[F19 PATCH] for Fedora 19 specific fix
[F18 PATCH] for Fedora 18 specific fix
and so on...

[WANG Chao]

On 08/15/13 at 02:59pm, Dave Young wrote:
> On 08/15/2013 12:27 PM, WANG Chao wrote:
> > Signed-off-by: WANG Chao
> >
> > kdump.service | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/kdump.service b/kdump.service
> > index 07e97fa..55b7ca2 100644
> > --- a/kdump.service
> > +++ b/kdump.service
> > @@ -1,6 +1,6 @@
> >  [Unit]
> >  Description=Crash recovery kernel arming
> > -After=network.target
> > +After=network.target network-online.target remote-fs.target
> >
> > [Service]
> >  Type=oneshot
>
> Can you resend it with proper changelog? what is the problem? why do we
> need this and how do you fix it? Probably also how did you test it if
> necessary.

Sure, I'll repost an update.
> Nit pick: I think we can assume below tag instead of [Fedora patch]:
>
> [PATCH] for rawhide fix
> [F19 PATCH] for Fedora 19 specific fix
> [F18 PATCH] for Fedora 18 specific fix
> and so on...

Great, I'll follow these rules.

Thanks
WANG Chao