-
Notifications
You must be signed in to change notification settings - Fork 526
coreupdate: document serving packages from s3 #1204
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
content looks solid to me.
coreupdate/coreupdate-aws-s3.md
Outdated
0 directories, 2 files | ||
``` | ||
|
||
Use the `updateservicectl package create bulk` command to create the package on an CoreUpdate instance. In the example below, CoreUpdate is running at: `http://coreupdate.example.com:8000`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
... a CoreUpdate instance ...
coreupdate/coreupdate-aws-s3.md
Outdated
$ updateservicectl --server=https://public.update.core-os.net package download --dir=/packages/ --version=1632.2.1 | ||
``` | ||
|
||
Now the /packages/ directory contains a json file with update metadata and the gzipped update payload: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
JSON ... Gzipped
coreupdate/coreupdate-aws-s3.md
Outdated
@@ -0,0 +1,58 @@ | |||
# Configure CoreUpdate to Serve Packages from AWS S3 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
... serve packages ...
coreupdate/coreupdate-aws-s3.md
Outdated
|
||
Note the use of the flags `--user` and `--key` these will be required. Most often the user will be `admin` and the key can be found in the `/etc/coreupdate/config.yaml` file. | ||
|
||
Be certain to format the URL passed to the `--base-url` flag as described in the AWS document: " [AWS S3 Regions and Endpoints][aws-endpoints]". |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove extra space - "[AWS ...
coreupdate/coreupdate-aws-s3.md
Outdated
2018/02/06 15:59:41 Please upload payloads to https://s3-us-west-1.amazonaws.com/core-update-support. | ||
``` | ||
|
||
Upload the update package to s3: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
AWS S3
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think to the S3 bucket
reads better
``` | ||
|
||
Consult the document [CoreUpdate - Configure Machines][core-update-config] for details on configuring a Container Linux host to use CoreUpdate. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we leave two blank lines between doc and links
coreupdate/coreupdate-aws-s3.md
Outdated
Consult the document [CoreUpdate - Configure Machines][core-update-config] for details on configuring a Container Linux host to use CoreUpdate. | ||
|
||
[updateservicectl]: https://github.com/coreos/updateservicectl/releases | ||
[airgap]: https://coreos.com/products/coreupdate/docs/latest/on-premises-deployment.html#air-gapped-package-management |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use relative to markdown, rather than absolute to rendered: [airgap]: on-premises-deployment.md#air-gapped-package-management
coreupdate/coreupdate-aws-s3.md
Outdated
[updateservicectl]: https://github.com/coreos/updateservicectl/releases | ||
[airgap]: https://coreos.com/products/coreupdate/docs/latest/on-premises-deployment.html#air-gapped-package-management | ||
[aws-endpoints]: https://docs.aws.amazon.com/general/latest/gr/rande.html#s3_region | ||
[core-update-config]: https://coreos.com/products/coreupdate/docs/latest/configure-machines.html |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No description provided.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Already approved PR. I'm not doing further checks on the docs change, just merging to unblock this fix. |
No description provided.