Skip to content

Always cleanup cEOS mount dir before creating cEOS#9986

Merged
wangxin merged 1 commit intosonic-net:masterfrom
wangxin:cleanup-ceos-mnt
Sep 15, 2023
Merged

Always cleanup cEOS mount dir before creating cEOS#9986
wangxin merged 1 commit intosonic-net:masterfrom
wangxin:cleanup-ceos-mnt

Conversation

@wangxin
Copy link
Copy Markdown
Collaborator

@wangxin wangxin commented Sep 15, 2023

Description of PR

Summary:
Fixes # (issue)

Type of change

  • Bug fix
  • Testbed and Framework(new/improvement)
  • Test case(new/improvement)

Back port request

  • 201911
  • 202012
  • 202205

Approach

What is the motivation for this PR?

Host dir like "/data/ceos/ceos_vms6-1_VM0100" is mounted to cEOS container as flash disk. This dir contains all kinds of configuration files that may affect cEOS behaviors.

For example, older version of cEOS has file ".arista_archive_config" which enables archiving of configs every 1 minute. On newer version of cEOS, this config file is removed by default.
However, after cEOS is upgraded, the left over ".arista_archive_config" file will cause newer cEOS to archive configs every 1 minute by default.

How did you do it?

To ensure that the cEOS container is always "cleanly" started, this change added code to always cleanup the mount dir before creating cEOS.

How did you verify/test it?

Any platform specific information?

Supported testbed topology if it's a new test case?

Documentation

Host dir like "/data/ceos/ceos_vms6-1_VM0100" is mounted to cEOS container
as flash disk. This dir contains all kinds of configuration files that may affect
cEOS behaviors.

For example, older version of cEOS has file ".arista_archive_config" which enables
archiving of configs every 1 minute. On newer version of cEOS, this config file
is removed by default.

However, after cEOS is upgraded, the left over ".arista_archive_config" file will
cause newer cEOS to archive configs every 1 minute by default.

To ensure that the cEOS container is always "cleanly" started, this change
added code to always cleanup the mount dir before creating cEOS.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
Copy link
Copy Markdown
Contributor

@wsycqyz wsycqyz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@wangxin wangxin merged commit 7a02844 into sonic-net:master Sep 15, 2023
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Sep 15, 2023
Host dir like "/data/ceos/ceos_vms6-1_VM0100" is mounted to cEOS container
as flash disk. This dir contains all kinds of configuration files that may affect
cEOS behaviors.

For example, older version of cEOS has file ".arista_archive_config" which enables
archiving of configs every 1 minute. On newer version of cEOS, this config file
is removed by default.

However, after cEOS is upgraded, the left over ".arista_archive_config" file will
cause newer cEOS to archive configs every 1 minute by default.

To ensure that the cEOS container is always "cleanly" started, this change
added code to always cleanup the mount dir before creating cEOS.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202205: #10036

mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Sep 15, 2023
Host dir like "/data/ceos/ceos_vms6-1_VM0100" is mounted to cEOS container
as flash disk. This dir contains all kinds of configuration files that may affect
cEOS behaviors.

For example, older version of cEOS has file ".arista_archive_config" which enables
archiving of configs every 1 minute. On newer version of cEOS, this config file
is removed by default.

However, after cEOS is upgraded, the left over ".arista_archive_config" file will
cause newer cEOS to archive configs every 1 minute by default.

To ensure that the cEOS container is always "cleanly" started, this change
added code to always cleanup the mount dir before creating cEOS.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
mssonicbld pushed a commit to mssonicbld/sonic-mgmt that referenced this pull request Sep 15, 2023
Host dir like "/data/ceos/ceos_vms6-1_VM0100" is mounted to cEOS container
as flash disk. This dir contains all kinds of configuration files that may affect
cEOS behaviors.

For example, older version of cEOS has file ".arista_archive_config" which enables
archiving of configs every 1 minute. On newer version of cEOS, this config file
is removed by default.

However, after cEOS is upgraded, the left over ".arista_archive_config" file will
cause newer cEOS to archive configs every 1 minute by default.

To ensure that the cEOS container is always "cleanly" started, this change
added code to always cleanup the mount dir before creating cEOS.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202012: #10037

@mssonicbld
Copy link
Copy Markdown
Collaborator

Cherry-pick PR to 202305: #10038

mssonicbld pushed a commit that referenced this pull request Sep 15, 2023
Host dir like "/data/ceos/ceos_vms6-1_VM0100" is mounted to cEOS container
as flash disk. This dir contains all kinds of configuration files that may affect
cEOS behaviors.

For example, older version of cEOS has file ".arista_archive_config" which enables
archiving of configs every 1 minute. On newer version of cEOS, this config file
is removed by default.

However, after cEOS is upgraded, the left over ".arista_archive_config" file will
cause newer cEOS to archive configs every 1 minute by default.

To ensure that the cEOS container is always "cleanly" started, this change
added code to always cleanup the mount dir before creating cEOS.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
mssonicbld pushed a commit that referenced this pull request Sep 15, 2023
Host dir like "/data/ceos/ceos_vms6-1_VM0100" is mounted to cEOS container
as flash disk. This dir contains all kinds of configuration files that may affect
cEOS behaviors.

For example, older version of cEOS has file ".arista_archive_config" which enables
archiving of configs every 1 minute. On newer version of cEOS, this config file
is removed by default.

However, after cEOS is upgraded, the left over ".arista_archive_config" file will
cause newer cEOS to archive configs every 1 minute by default.

To ensure that the cEOS container is always "cleanly" started, this change
added code to always cleanup the mount dir before creating cEOS.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
mssonicbld pushed a commit that referenced this pull request Sep 15, 2023
Host dir like "/data/ceos/ceos_vms6-1_VM0100" is mounted to cEOS container
as flash disk. This dir contains all kinds of configuration files that may affect
cEOS behaviors.

For example, older version of cEOS has file ".arista_archive_config" which enables
archiving of configs every 1 minute. On newer version of cEOS, this config file
is removed by default.

However, after cEOS is upgraded, the left over ".arista_archive_config" file will
cause newer cEOS to archive configs every 1 minute by default.

To ensure that the cEOS container is always "cleanly" started, this change
added code to always cleanup the mount dir before creating cEOS.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
AharonMalkin pushed a commit to AharonMalkin/sonic-mgmt that referenced this pull request Jan 25, 2024
Host dir like "/data/ceos/ceos_vms6-1_VM0100" is mounted to cEOS container
as flash disk. This dir contains all kinds of configuration files that may affect
cEOS behaviors.

For example, older version of cEOS has file ".arista_archive_config" which enables
archiving of configs every 1 minute. On newer version of cEOS, this config file
is removed by default.

However, after cEOS is upgraded, the left over ".arista_archive_config" file will
cause newer cEOS to archive configs every 1 minute by default.

To ensure that the cEOS container is always "cleanly" started, this change
added code to always cleanup the mount dir before creating cEOS.

Signed-off-by: Xin Wang <xiwang5@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants