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

[RH8] Show warning message when site.xcatsslversion has a value during upgrade #6090

Merged
merged 4 commits into from
Mar 13, 2019

Conversation

neo954
Copy link
Contributor

@neo954 neo954 commented Mar 13, 2019

The PR is to fix issue #6089

The modification include

##Show warning message when site.xcatsslversion has an value during upgrade

##Documentation updates

@neo954 neo954 requested a review from immarvin March 13, 2019 09:29
@neo954 neo954 added the sprint4 label Mar 13, 2019
@neo954 neo954 added this to the 2.14.6 milestone Mar 13, 2019
@neo954
Copy link
Contributor Author

neo954 commented Mar 13, 2019

The unit test result.

# xcatconfig -u
__      ___   ___ _  _ ___ _  _  ___
\ \    / /_\ | _ \ \| |_ _| \| |/ __|                      _     +-+-+-+-+-+-+-+
 \ \/\/ / _ \|   / .` || || .` | (_ |                     oo\    |W|A|R|N|I|N|G|
  \_/\_/_/ \_\_|_\_|\_|___|_|\_|\___|                    (__)\   +-+-+-+-+-+-+-+
+--------------------------------------------------------------+ +-+-+-+-+-+-+-+
| The default value of `site.xcatsslversion' was updated.      |:|W|A|R|N|I|N|G|
| Run `chdef -t site xcatsslversion=' to update your system to |:+-+-+-+-+-+-+-+
| the new default value. See `man site' for more details.      |:+-+-+-+-+-+-+-+
+--------------------------------------------------------------+:|W|A|R|N|I|N|G|
 ::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::::+-+-+-+-+-+-+-+

@immarvin
Copy link
Contributor

i am ok with the change

@immarvin immarvin merged commit d6b36a5 into xcat2:rhels8 Mar 13, 2019
@neo954 neo954 changed the title [RH8] Show warning message when site.xcatsslversion has an value during upgrade [RH8] Show warning message when site.xcatsslversion has a value during upgrade Mar 13, 2019
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.

2 participants