Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove the os.conf creation in the rear.spec file #1713

Merged
merged 1 commit into from Jan 30, 2018
Merged

Remove the os.conf creation in the rear.spec file #1713

merged 1 commit into from Jan 30, 2018

Conversation

gdha
Copy link
Member

@gdha gdha commented Jan 26, 2018

Remove the os.conf creation in the rear.spec file and fixed in function SetOSVendorAndVersion the proper detection of RedHatEnterpriseServer. Added new script usr/share/rear/init/default/005_verify_os_conf.sh to create the os.conf if it did not exist yet - issue #1639

…on SetOSVendorAndVersion the proper detection of

RedHatEnterpriseServer. Added new script usr/share/rear/init/default/005_verify_os_conf.sh to create the os.conf if
it did not exist yet - issue #1639
@gdha gdha added enhancement Adaptions and new features cleanup labels Jan 26, 2018
@gdha gdha added this to the ReaR v2.4 milestone Jan 26, 2018
@gdha gdha self-assigned this Jan 26, 2018
@gdha gdha requested review from schlomo and jsmeix January 26, 2018 17:02
Copy link
Member

@jsmeix jsmeix left a comment

Choose a reason for hiding this comment

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

From plain looking at the code
it looks good to me.

@jsmeix
Copy link
Member

jsmeix commented Jan 29, 2018

@gdha
I did not have a closer look how those things work
but I wonder why os.conf needs to be created at all
during "rear WORKFLOW" runtime?
I.e. I wonder why it is not sufficient to only rely
on the SetOSVendorAndVersion() function
during "rear WORKFLOW" runtime?

@gdha
Copy link
Member Author

gdha commented Jan 29, 2018

@jsmeix os.conf is important during the recover phase as we cannot trust that the SetOSVendorAndVersion() function will work properly.

@gdha gdha merged commit 893bb8f into rear:master Jan 30, 2018
@jsmeix
Copy link
Member

jsmeix commented Jan 30, 2018

Yes, but why is /etc/rear/os.conf autocreated in the original sytem?

Why not only autocreate a $ROOTFS_DIR/etc/rear/os.conf in the recovery system
e.g. in the same way as $ROOTFS_DIR/etc/rear-release gets autocreated by
build/default/970_add_rear_release.sh ?

In general I do not like it when any "rear WORKFOLW" automatically
changes the original system without an explicit user request.
I think in general the original system should be sacrosanct - except there is
really a very good reason to automatically change the original system.

@gdha
Copy link
Member Author

gdha commented Jan 30, 2018

@jsmeix I prefer to have the discussion in #1639 if that is ok for you?

@jsmeix
Copy link
Member

jsmeix commented Jan 30, 2018

@gdha
of course you are right.
This pull request is done and further things should
happen in #1639

@schlomo
Copy link
Member

schlomo commented Mar 6, 2023

@gdha this is old stuff, but I have a very important question: What needs the /etc/rear/os.conf file on the original system? Wouldn't it be enough to only write that to the rescue system instead?

The reason I'm asking are two problems I encounter:

  1. the file will never change, even if the OS is upgraded to a newer version
  2. the file is created in the source tree if I run ReaR from checkout, and it stays there forever even if I try ReaR on a different VM

So I'd like to find out if I can move it to the rescue system only. Or maybe we actually don't need it any more because /etc/os-release is now the de-facto standard in any case?

@jsmeix
Copy link
Member

jsmeix commented Mar 14, 2023

See also
#2954 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants