devstack overwrites screenrc

Agreed. We almost always run stack.sh in a vm or container so hadn't hit
this issue.
On Dec 9, 2011 11:15 AM, "Scott Moser" <email address hidden> wrote:

> Public bug reported:
>
> Devstack rudely overwrites a users screenrc file without even checking
> if it is present.
>
> I much prefer the old method of setting up screen (possibly I'm
> remembering 'nova.sh') where it configured the screen it launched with a
> hardstatus line, rather than changing a user's defaults or setting some
> for them.
>
> note, this interacts badly with 'byobu'.
>
> ** Affects: devstack
> Importance: Undecided
> Status: New
>
> ** Description changed:
>
> Devstack rudely overwrites a users screenrc file without even checking
> if it is present.
>
> I much prefer the old method of setting up screen (possibly I'm
> remembering 'nova.sh') where it configured the screen it launched with a
> hardstatus line, rather than changing a user's defaults or setting some
> for them.
> +
> + note, this interacts badly with 'byobu'.
>
> --
> You received this bug notification because you are subscribed to
> devstack.
> https://bugs.launchpad.net/bugs/902297
>
> Title:
> devstack overwrites screenrc
>
> Status in devstack - openstack dev environments:
> New
>
> Bug description:
> Devstack rudely overwrites a users screenrc file without even checking
> if it is present.
>
> I much prefer the old method of setting up screen (possibly I'm
> remembering 'nova.sh') where it configured the screen it launched with
> a hardstatus line, rather than changing a user's defaults or setting
> some for them.
>
> note, this interacts badly with 'byobu'.
>
> To manage notifications about this bug go to:
> https://bugs.launchpad.net/devstack/+bug/902297/+subscriptions
>

thoughts - https://review.openstack.org/2247 ?

in nova.sh you had the ability to not set the modeline - needed?

Reviewed: https://review.openstack.org/2247
Committed: http://github.com/openstack-dev/devstack/commit/d9eafd58913453b70b2664c272ac3063ae14280c
Submitter: Jenkins
Branch: master

 status fixcommitted
 done

commit d9eafd58913453b70b2664c272ac3063ae14280c
Author: Jesse Andrews <email address hidden>
Date: Sat Dec 10 13:55:44 2011 -0800

    move back to nova.sh method of configuring screen

    instead of overwriting screenrc (which is useful for new VMs), we manually set
    the screen status line - fixes bug 902297

    Change-Id: I507dc36e85e2bc3635503cde426bab8a2e966f06