Skip to content

Conversation

@arm4b
Copy link
Member

@arm4b arm4b commented Dec 14, 2020

Per previous discussion in #679, adding an option to install 3rd party PPA based on user's input.

On Ubuntu Xenial the installer would stop, show insecure warning and wait for user's input to continue:

WARNING!
python3.6 package is required as a dependency for StackStorm and none of your software repositories provide it.
We recommend switching to Ubuntu 18.04 LTS (Bionic) as a base OS.

Alternatively we'll try to add 3rd party python3.6 APT repository: https://launchpad.net/~deadsnakes/+archive/ubuntu/ppa.
By continue you're aware of security risk associated with using unofficial 3rd party PPA repository.

@arm4b arm4b added this to the 3.4.0 milestone Dec 14, 2020
@arm4b arm4b requested review from a team, amanda11 and blag December 14, 2020 19:16
echo "Alternatively we'll try to add 3rd party python3.6 APT repository: https://launchpad.net/~deadsnakes/+archive/ubuntu/ppa."
echo "By continue you're aware of security risk associated with using unofficial 3rd party PPA repository."
echo ""
read -p "Press [y] to continue or [n] to cancel adding it: " choice
Copy link
Member Author

@arm4b arm4b Dec 14, 2020

Choose a reason for hiding this comment

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

I decided to not provide a dedicated installation flag for Ubuntu Xenial installer as everything could be already done with yes | or echo 'y' | from the user side anyways as answer to this [y/n] question.

Copy link
Member Author

@arm4b arm4b Dec 14, 2020

Choose a reason for hiding this comment

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

@amanda11 ^^ I believe that should answer your question about running installer in non-interactive mode.

Even if we add --install-py3-from-insecure-ppa or whatever flag for extra security, - it makes little sense as users would be able to use yes | installer or echo y | installer to answer the question in a native Linux way.

Copy link
Contributor

Choose a reason for hiding this comment

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

OK - so we just need to update the E2E tests to install with an echo "y" | installer then?
Fine with me.

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah that's fine with me as well.

Copy link
Contributor

@amanda11 amanda11 left a comment

Choose a reason for hiding this comment

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

LGTM - just a minor comment on wording in warning.

Also, are we adding the --use-deadsnakes-repo-for-python3.6 option in under a separate PR to avoid the question?

Copy link
Contributor

@blag blag left a comment

Choose a reason for hiding this comment

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

Some suggestions.

@blag blag merged commit 34b0396 into master Dec 15, 2020
@blag blag deleted the u16-py3-ppa branch December 15, 2020 04:00
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.

4 participants