Deprecate support for EL6 #994

amanda11 · 2020-07-13T10:22:37Z

RHEL / CentOS 6 support has been deprecated.
Account for st2mistral not being present on upgrades for some distributions.
Updates for MongoDB 4.0 being used on all distributions.

addressing st2docs - remove EL6 from StackStorm/community#39

…arily being there on upgrade

amanda11 · 2020-07-13T11:56:56Z

EL6 removal from ewcdocs change that affects the build failure in stdocs, PR extremenetworks/ipfabric-docs#169

blag

Pretty good so far, but could use a bit of polish.

blag · 2020-07-13T17:56:58Z

docs/source/install/upgrades.rst

@@ -136,8 +136,12 @@ This is the standard upgrade procedure:

      sudo yum update st2 st2web st2chatops st2mistral

+   Omit st2mistral if it is not installed on your distribution.


These two comments could use a bit of expansion. Why would st2mistral not be installed on a server? It's required by StackStorm <= 3.2, so it should be there. For StackStorm >= 3.3, it won't be available. We should explain that here, possibly include a link to the changelog entry.

@blag it isn’t installed though on EL8 and neither for Bionic with ST 3.2 though. That was what I was trying to get across.
So I was thinking of it as part of a upgrade on el8 with what we have now.

I was trying to keep in the docs in step with where we were with the install. So was thinking of st2mistral not installed on EL8 etc. Rather than the changes for the end of ST 3.3 when st2mistral would be gone completely - as I think we've not taken that out yet from the install yet for EL7 etc.

docs/source/install/ewc.rst

docs/source/install/puppet.rst

punkrokk · 2020-07-15T19:04:38Z

docs/source/install/system_requirements.rst

@@ -28,9 +28,6 @@ be automatically added when you install |st2|.
 | `RHEL 7 <https://www.redhat.com/en/technologies/linux-platforms/enterprise-linux>`_ /                 | `bento/centos-7.4 <https://app.vagrantup.com/bento/boxes/centos-7.4>`_       | `Red Hat Enterprise Linux (RHEL) 7.2 (HVM)  <https://aws.amazon.com/marketplace/pp/B019NS7T5I/ref=srh_res_product_title?ie=UTF8&sr=0-2&qid=1457037671547>`_       |


On line 25 above, can you update for RHEL/CentOS 8.2?

I've updated 8.2 centos. The only 8.2 AWS EC2 I could find is one from pro-computers. The AWS marketplace one is still the one above. Which do we want to link to?

punkrokk · 2020-07-15T19:05:58Z

docs/source/install/uninstall.rst

@@ -113,6 +103,12 @@ below. Only execute the instructions for your distribution.
    sudo yum erase st2 st2mistral st2chatops st2web st2python bwc-ui st2flow




Should we have a note here for RHEL/CentOS 8.x?

I think it would be:
sudo yum erase st2 st2chatops st2web st2python bwc-ui st2flow

cc @armab

@punkrokk Is that not covered by the note below on line 106- to omit st2mistral?
That was the wording we had on the 3.2 documentation to cover RH8/Ubuntu 18.

HOWEVER I'm not sure if it was deliberate but some of the changes in the 3.2 documentation aren't on the master branch - so the notes that were added for 3.2 for RH 8 about missing mistral etc, seemed to have got dropped from master.

Yes it's probably covered by the note. I'm a fan of having a nice codeblock to copy and paste, which is why I made the suggestion and it seemed in line with the rest of the page.

@punkrokk Let me know if the changes made are preferred. I changed the note that is in the current st2docs (https://docs.stackstorm.com/install/uninstall.html) with code blocks like you suggested.

I noticed a few other updates needed, e.g.
st2mistral and postgres not used on ubuntu 18.
st2python not used on rh 7/8
So a few more changes, and gone for the code blocks.

punkrokk · 2020-07-15T19:06:40Z

StackStorm/st2#4972

Co-authored-by: JP Bourget <[email protected]>

punkrokk · 2020-07-16T04:24:40Z

@amanda11 Please see the errors on the circle builds.

amanda11 · 2020-07-16T08:44:30Z

@amanda11 Please see the errors on the circle builds.

@punkrokk Fixed the st2docs error - the ewcdocs will fail until extremenetworks/ipfabric-docs#169 is merged as it is because ewcdocs refers to the RH 6 file. HOWEVER I'm not sure why the ipfabric-docs fails it's CI checks - seems to look for a branch that doesn't exist.

docs/source/install/__mongodb_note.rst

Co-authored-by: Eugen C. <[email protected]>

arm4b · 2020-07-17T17:20:22Z

#996 should help with EWC Docs/dependency issue.
It's just a good time to stop building and updating them as EWC code was Open Sourced.

amanda11 · 2020-07-18T17:27:49Z

I think all review comments have been addressed, please feedback any extra changes required

docs/source/install/puppet.rst

arm4b

LGTM 👍
@amanda11 Thanks for taking the ownership here!

Looks like @punkrokk and @blag feedback is addressed, so I'll merge the PR.

amanda11 added 2 commits July 13, 2020 11:06

Remove EL6 support

66cf21d

Minor changes on mongo notes and to account for st2mistral not necess…

82310b0

…arily being there on upgrade

blag suggested changes Jul 13, 2020

View reviewed changes

Use same omit mistral comments as in 3.2 documentation

3790efa

punkrokk suggested changes Jul 15, 2020

View reviewed changes

punkrokk mentioned this pull request Jul 15, 2020

EL7/U16 -> Mongo 4.0 Master Issue StackStorm/st2#4972

Closed

6 tasks

amanda11 and others added 2 commits July 15, 2020 22:41

Update docs/source/install/puppet.rst

26c2241

Co-authored-by: JP Bourget <[email protected]>

Update docs/source/install/ewc.rst

390c154

Co-authored-by: JP Bourget <[email protected]>

Update 8.2 centos and fix list

f25c4e7

amanda11 added 2 commits July 16, 2020 09:54

Amend uninstall

9206987

No postgres on ubuntu 18

9b5b292

arm4b reviewed Jul 16, 2020

View reviewed changes

docs/source/install/__mongodb_note.rst Outdated Show resolved Hide resolved

arm4b mentioned this pull request Jul 16, 2020

Game plan: EL6 Deprecation (v3.3.0) StackStorm/community#39

Closed

24 tasks

arm4b added this to the 3.3.0 milestone Jul 16, 2020

Update docs/source/install/__mongodb_note.rst

43c2bc1

Co-authored-by: Eugen C. <[email protected]>

arm4b mentioned this pull request Jul 17, 2020

Stop building EWCdocs #996

Merged

Merge branch 'master' into remove_el6

30eab33

arm4b requested review from blag and punkrokk July 19, 2020 14:44

punkrokk approved these changes Jul 19, 2020

View reviewed changes

arm4b reviewed Jul 23, 2020

View reviewed changes

docs/source/install/puppet.rst Outdated Show resolved Hide resolved

Update docs/source/install/puppet.rst

82f374d

arm4b approved these changes Jul 23, 2020

View reviewed changes

arm4b merged commit 54b5397 into StackStorm:master Jul 23, 2020

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Deprecate support for EL6 #994

Deprecate support for EL6 #994

amanda11 commented Jul 13, 2020 •

edited

Loading

amanda11 commented Jul 13, 2020

blag left a comment

blag Jul 13, 2020

amanda11 Jul 13, 2020 •

edited

Loading

punkrokk Jul 15, 2020

amanda11 Jul 16, 2020

punkrokk Jul 15, 2020

amanda11 Jul 15, 2020

punkrokk Jul 16, 2020

amanda11 Jul 16, 2020

punkrokk commented Jul 15, 2020

punkrokk commented Jul 16, 2020

amanda11 commented Jul 16, 2020

arm4b commented Jul 17, 2020 •

edited

Loading

amanda11 commented Jul 18, 2020

arm4b left a comment

		@@ -136,8 +136,12 @@ This is the standard upgrade procedure:

		sudo yum update st2 st2web st2chatops st2mistral

		Omit st2mistral if it is not installed on your distribution.

		@@ -28,9 +28,6 @@ be automatically added when you install \|st2\|.
		\| `RHEL 7 <https://www.redhat.com/en/technologies/linux-platforms/enterprise-linux>`_ / \| `bento/centos-7.4 <https://app.vagrantup.com/bento/boxes/centos-7.4>`_ \| `Red Hat Enterprise Linux (RHEL) 7.2 (HVM) <https://aws.amazon.com/marketplace/pp/B019NS7T5I/ref=srh_res_product_title?ie=UTF8&sr=0-2&qid=1457037671547>`_ \|

		@@ -113,6 +103,12 @@ below. Only execute the instructions for your distribution.
		sudo yum erase st2 st2mistral st2chatops st2web st2python bwc-ui st2flow

Deprecate support for EL6 #994

Deprecate support for EL6 #994

Conversation

amanda11 commented Jul 13, 2020 • edited Loading

amanda11 commented Jul 13, 2020

blag left a comment

Choose a reason for hiding this comment

blag Jul 13, 2020

Choose a reason for hiding this comment

amanda11 Jul 13, 2020 • edited Loading

Choose a reason for hiding this comment

punkrokk Jul 15, 2020

Choose a reason for hiding this comment

amanda11 Jul 16, 2020

Choose a reason for hiding this comment

punkrokk Jul 15, 2020

Choose a reason for hiding this comment

amanda11 Jul 15, 2020

Choose a reason for hiding this comment

punkrokk Jul 16, 2020

Choose a reason for hiding this comment

amanda11 Jul 16, 2020

Choose a reason for hiding this comment

punkrokk commented Jul 15, 2020

punkrokk commented Jul 16, 2020

amanda11 commented Jul 16, 2020

arm4b commented Jul 17, 2020 • edited Loading

amanda11 commented Jul 18, 2020

arm4b left a comment

Choose a reason for hiding this comment

amanda11 commented Jul 13, 2020 •

edited

Loading

amanda11 Jul 13, 2020 •

edited

Loading

arm4b commented Jul 17, 2020 •

edited

Loading