Skip to content

Update aerospike readme.#2256

Open
kportertx wants to merge 10 commits intodocker-library:masterfrom
kportertx:update-aerospike-readme
Open

Update aerospike readme.#2256
kportertx wants to merge 10 commits intodocker-library:masterfrom
kportertx:update-aerospike-readme

Conversation

@kportertx
Copy link
Copy Markdown

No description provided.

@kportertx kportertx marked this pull request as draft January 3, 2023 22:29
Comment thread aerospike/content.md Outdated
Comment thread aerospike/content.md Outdated
@kportertx
Copy link
Copy Markdown
Author

kportertx commented Jan 4, 2023

@tianon BTW, I set this in draft because I wanted someone on our side to review these changes. This was a quick repackaging of our GitHub docs once I realized how stale the official image docs were. Eventually, I'll look into automating keeping these various DockerHub docs for these images in sync. I'll take it out of draft once that review has been done.

I think the workflows should all pass now.

@kportertx kportertx marked this pull request as ready for review January 6, 2023 23:53
@kportertx
Copy link
Copy Markdown
Author

@tianon, we've reviewed and approve these changes - welcome to merge when able. Thank you for your assistance.

Comment thread aerospike/content.md
FEATKEY=$(base64 ~/Desktop/evaluation-features.conf)
docker run -d -e "FEATURES=$FEATKEY" -e "FEATURE_KEY_FILE=env-b64:FEATURES" --name aerospike -p 3000-3002:3000-3002 %%IMAGE%%:ee-[version]
```
By using Aerospike Community Edition you agree to the [COMMUNITY_LICENSE](community/COMMUNITY_LICENSE).
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Just one minor bit of feedback. This link isn't going to work here or on Docker Hub. Can you update it to a full URL?

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants