Skip to content
This repository has been archived by the owner on Feb 9, 2024. It is now read-only.

[7.0.x] loosen same OS requirement #1530

Merged
merged 1 commit into from May 12, 2020

Conversation

a-palchikov
Copy link
Contributor

Description

This PR loosens the same OS requirement preflight check with the following semantics:

  • OS distributions are the same if both name and versions are exact match
  • OS distributions are compatible (and check succeeds) if name is an exact match and versions match on the prefix up to the first '.'
  • otherwise, OS distributions are not compatible

Also, this has been brought up multiple times (frequently in the form of asking whether it's possible to suppress just the OS check).

Type of change

  • This change has a user-facing impact

Linked tickets and other PRs

TODOs

  • Self-review the change
  • Write tests
  • Perform manual testing
  • Address review feedback

Testing done

Unit-tested change.

@a-palchikov a-palchikov requested review from a team, r0mant and knisbet May 8, 2020 20:26
* Loosen the same OS requirement in the following way:
 * consider OS distributions to be same if name and version are an exact match
 * consider OS distributions to be compatible if name is an exact match and versions match on prefix including the first dot
 * otherwise consider OS distributions to be incompatible

Updates #1112.
@a-palchikov a-palchikov merged commit 988d767 into version/7.0.x May 12, 2020
@a-palchikov a-palchikov mentioned this pull request Jul 15, 2020
3 tasks
a-palchikov added a commit that referenced this pull request Jul 24, 2020
* Expand improvements (#1361)

* Forward port changes from 6.3.x
* Fix tests
* Do not pull/unpack planet package as it needs special attention to file/directory context
* Forward port changes from #1348
* Format multiple operations in 'gravity plan' if outputting as text as a table. Forward ports #1373

* Revert LVMSystemDir to schema backwards compatible and mark it deprecated in code.
Handle completed operation error gracefully in expand.

* Remove container name in selinux Makefile to avoid conflicts with multiple jenkins jobs

* Rework client/agent logic to remove the agent from the agent group
running by the cluster controller after the operation is completed.

* Bump e

* Port another slew of PRs:
 * #1361
 * #1424
 * #1530
 * #1531
 * #1542
 * #1603
 * #1648
 * https://github.com/gravitational/gravity.e/pull/4298

* Regenerate proto sources

* Address review comments

* Bump e. Update cli args test to use a common set of redacted flags for redact test cases

* Revert another test change

* Correct planet path for journalctl
@a-palchikov a-palchikov deleted the dmitri/7.0.x/1112-same-os branch September 29, 2020 18:55
helgi pushed a commit to helgi/gravity that referenced this pull request Jun 21, 2021
* Expand improvements (gravitational#1361)

* Forward port changes from 6.3.x
* Fix tests
* Do not pull/unpack planet package as it needs special attention to file/directory context
* Forward port changes from gravitational#1348
* Format multiple operations in 'gravity plan' if outputting as text as a table. Forward ports gravitational#1373

* Revert LVMSystemDir to schema backwards compatible and mark it deprecated in code.
Handle completed operation error gracefully in expand.

* Remove container name in selinux Makefile to avoid conflicts with multiple jenkins jobs

* Rework client/agent logic to remove the agent from the agent group
running by the cluster controller after the operation is completed.

* Bump e

* Port another slew of PRs:
 * gravitational#1361
 * gravitational#1424
 * gravitational#1530
 * gravitational#1531
 * gravitational#1542
 * gravitational#1603
 * gravitational#1648
 * https://github.com/gravitational/gravity.e/pull/4298

* Regenerate proto sources

* Address review comments

* Bump e. Update cli args test to use a common set of redacted flags for redact test cases

* Revert another test change

* Correct planet path for journalctl
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants