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

[doc] Change downloads link #25374

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ddhodge
Copy link
Contributor

@ddhodge ddhodge commented Dec 19, 2024

downloads.yugabyte.com -> software.yugabyte.com

@netlify /preview/releases/

@ddhodge ddhodge self-assigned this Dec 19, 2024
@ddhodge ddhodge added the area/documentation Documentation needed label Dec 19, 2024
Copy link

netlify bot commented Dec 19, 2024

Deploy Preview for infallible-bardeen-164bc9 ready!

Name Link
🔨 Latest commit 1b491f4
🔍 Latest deploy log https://app.netlify.com/sites/infallible-bardeen-164bc9/deploys/676497aa917d740008cee3b2
😎 Deploy Preview https://deploy-preview-25374--infallible-bardeen-164bc9.netlify.app/preview/releases/
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@@ -316,8 +316,8 @@ for ip in $ALL_NODES; do \
echo =======$ip=======; \
ssh -i $PEM $ADMIN_USER@$ip \
"cd ~/yb-software; \
curl -k -o yugabyte-${YB_VERSION}-linux.tar.gz \
https://downloads.yugabyte.com/yugabyte-${YB_VERSION}-linux.tar.gz"; \
curl -k -o -L yugabyte-${YB_VERSION}-linux.tar.gz \
Copy link
Member

Choose a reason for hiding this comment

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

This should maybe be:

curl -OL https://software.yugabyte.com/yugabyte-${YB_VERSION}-linux.tar.gz

The -k option means not to check the SSL cert, but we have valid SSL certs.
The -O option means name the downloaded file the same as it is on the remote side which is yugabyte-${YB_VERSION}-linux.tar.gz

@@ -30,11 +30,11 @@ Refer to [issue 8531](https://github.com/yugabyte/yugabyte-db/issues/8531) for d
### Downloads
<ul class="nav yb-pills">
<li>
<a href="https://downloads.yugabyte.com/yugabyte-2.7.2.0-darwin.tar.gz">
<a href="https://software.yugabyte.com/yugabyte-2.7.2.0-darwin.tar.gz">
Copy link
Member

Choose a reason for hiding this comment

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

I don't think we have a pattern for these old style releases in snarf. This URL doens't work for me.

@@ -332,7 +332,7 @@ You need to perform the following steps:
Installing YugabyteDB Anywhere on Airgapped hosts, without access to any Internet traffic (inbound or outbound) requires the following:

- Whitelisting endpoints: to install Replicated and YugabyteDB Anywhere on a host with no Internet connectivity, you have to first download the binaries on a computer that has Internet connectivity, and then copy the files over to the appropriate host. In case of restricted connectivity, the following endpoints have to be whitelisted to ensure that they are accessible from the host marked for installation:
`https://downloads.yugabyte.com`
`https://software.yugabyte.com`
`https://download.docker.com`
Copy link
Member

Choose a reason for hiding this comment

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

Should this be both downloads.yugabyte.com and software.yugabyte.com?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Actually this is a Replicated install instructions include file that I don't think is even used in the docs

@@ -57,7 +57,7 @@ You can perform an airgapped installation on RHEL 8 and CentOS 8.
1. Download the airgapped bundle:

```sh
https://s3.us-west-2.amazonaws.com/downloads.yugabyte.com/repos/airgapped/yb-voyager-latest-rhel-8-x86_64.tar.gz
https://software.yugabyte.com/repos/airgapped/yb-voyager-latest-rhel-8-x86_64.tar.gz
Copy link
Member

Choose a reason for hiding this comment

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

This pattern doesn't exist in scarf yet either. https://downloads.yugabyte.com/repos/airgapped/yb-voyager-latest-rhel-8-x86_64.tar.gz works but https://software.yugabyte.com/repos/airgapped/yb-voyager-latest-rhel-8-x86_64.tar.gz does not.

curl -k -o yugabyte-${YB_VERSION}-linux.tar.gz \
https://downloads.yugabyte.com/yugabyte-${YB_VERSION}-linux.tar.gz"; \
curl -k -o -L yugabyte-${YB_VERSION}-linux.tar.gz \
https://software.yugabyte.com/yugabyte-${YB_VERSION}-linux.tar.gz"; \
Copy link
Member

Choose a reason for hiding this comment

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

This seems out of date, the URL doesn't include the releases/ component and there is no arch encoded in the file name. It should be updated to match the same pattern as in install-software.md

@@ -83,7 +83,7 @@ USER root
RUN apt-get update && export DEBIAN_FRONTEND=noninteractive && \
apt-get install -y netcat --no-install-recommends

RUN curl -sSLo ./yugabyte.tar.gz https://downloads.yugabyte.com/yugabyte-${YB_VERSION}-linux.tar.gz \
RUN curl -sSLo ./yugabyte.tar.gz https://software.yugabyte.com/yugabyte-${YB_VERSION}-linux.tar.gz \
Copy link
Member

Choose a reason for hiding this comment

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

Deprecated URL, see above.

@@ -77,7 +77,7 @@ USER root
RUN apt-get update && apt-get install -y \
netcat --no-install-recommends
# download and initialize the file structure
RUN curl -sSLo ./yugabyte.tar.gz https://downloads.yugabyte.com/yugabyte-${YB_VERSION}-linux.tar.gz \
RUN curl -sSLo ./yugabyte.tar.gz https://software.yugabyte.com/yugabyte-${YB_VERSION}-linux.tar.gz \
Copy link
Member

Choose a reason for hiding this comment

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

deprecated URL

@@ -75,7 +75,7 @@ Install the new version of YugabyteDB in a new directory on every YugabyteDB nod
For example:

```sh
wget https://downloads.yugabyte.com/yugabyte-$NEW_VER.tar.gz
wget https://software.yugabyte.com/yugabyte-$NEW_VER.tar.gz
Copy link
Member

Choose a reason for hiding this comment

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

deprecated URL

@@ -26,13 +26,13 @@ The YCQL shell was previously named `cqlsh`. Although the `cqlsh` binary is avai
If you prefer, you can install a standalone version using a shell script:

```sh
curl -sSL https://downloads.yugabyte.com/get_clients.sh | bash
curl -sSL https://software.yugabyte.com/get_clients.sh | bash
Copy link
Member

Choose a reason for hiding this comment

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

No pattern for this in scarf yet.

@@ -675,7 +675,7 @@ Use the `yugabyted start` command to start a one-node YugabyteDB cluster for run

To use encryption in transit, OpenSSL must be installed on the nodes.

If you want to use backup and restore, start the node with `--backup_daemon=true` to initialize the backup and restore agent. You also need to download and extract the [YB Controller release](https://downloads.yugabyte.com/ybc/2.1.0.0-b9/ybc-2.1.0.0-b9-linux-x86_64.tar.gz) to the yugabyte-{{< yb-version version="stable" >}} release directory.
If you want to use backup and restore, start the node with `--backup_daemon=true` to initialize the backup and restore agent. You also need to download and extract the [YB Controller release](https://software.yugabyte.com/ybc/2.1.0.0-b9/ybc-2.1.0.0-b9-linux-x86_64.tar.gz) to the yugabyte-{{< yb-version version="stable" >}} release directory.
Copy link
Member

Choose a reason for hiding this comment

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

No pattern for this yet in scarf

Copy link
Contributor

@premkumr premkumr left a comment

Choose a reason for hiding this comment

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

Something wrong about the URL format. I'm getting a lot of 404s

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/documentation Documentation needed
Projects
Status: In progress
Development

Successfully merging this pull request may close these issues.

3 participants