Skip to content
This repository was archived by the owner on Aug 16, 2022. It is now read-only.

Conversation

cwillum
Copy link
Contributor

@cwillum cwillum commented Aug 12, 2022

Signed-off-by: cwillum cwmmoore@amazon.com

For statements and changes that emphasize deprecation.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Fix #840

Signed-off-by: cwillum <cwmmoore@amazon.com>
@cwillum cwillum requested a review from lizsnyder August 12, 2022 20:40
@cwillum cwillum self-assigned this Aug 12, 2022
```

1. Choose the version you'd like and install it:
1. Choose the version you'd like and install it: (See [note](#rpm-package) above.)
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this be "Choose the version you'd like to install:"?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yes, that's better - changed based on the suggestion

Open Distro is no longer being developed and exists only as an archive. Please visit [OpenSearch documentation](https://opensearch.org/docs/latest) to get started with OpenSearch.
{: .warning}

1. Download the tarball:
Copy link
Contributor

Choose a reason for hiding this comment

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

These tarballs don't exist anywhere, so the commands don't work:

% Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0

Do you think we should remove them?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I moved the warning in the excerpt above to fill the place where the now unavailable tarballs were. Also removed the curl's for checksum, since they're not available either. I put a simple explanation in their place: (No longer available). Just so someone knows there used to be something there.

Signed-off-by: cwillum <cwmmoore@amazon.com>
@cwillum
Copy link
Contributor Author

cwillum commented Aug 15, 2022

Merged. Moving the repo back to archived status. Changes are effective on documentation website.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants