magma
magma copied to clipboard
docs(agw): Added Docker AGW install docs
Signed-off-by: Shubham Tatvamasi [email protected]
Summary
Test Plan
Additional Information
- [ ] This change is backwards-breaking
Thanks for opening a PR! :100:
- All commits must be signed off. This is enforced by
DCO check
. - All PR titles must follow the semantic commits format. This is enforced by
Semantic PR
.
Howto
- Reviews. The "Reviewers" listed for this PR are the Magma maintainers who will shepherd it.
- Checks. All required CI checks must pass before merge.
-
Merge. Once approved and passing CI checks, use the
ready2merge
label to indicate the maintainers can merge your PR.
More info
Please take a moment to read through the Magma project's
- Contributing Conventions for norms around contributed code
If this is your first Magma PR, also consider reading
- Developer Onboarding for onboarding as a new Magma developer
- Development Workflow for guidance on your first pull request
- CI Checks for points of contact for failing or flaky CI checks
- GitHub-to-Slack mappings for Magma maintainers for guidance on how to contact maintainers on Slack
dp-workflow
0 tests 0 :heavy_check_mark: 0s :stopwatch: 0 suites 0 :zzz: 0 files 0 :x:
Results for commit 6b903601.
:recycle: This comment has been updated with latest results.
Oops! Looks like you failed the Markdown lint check
.
Howto
- Instructions on formatting your Markdown changes
- Guide to the different CI checks and resolution guidelines
:recycle: Updated: :white_check_mark: The check is passing the Markdown lint check after the last commit.
In Prerequisites section: Mention as aarch64 or 64 bit-X86
Regarding the registry location - Can the user use the location below to test 1.8 AGW images ? I am not able to use it in my test environment. Can you add the docker registry details for test environment as "docker-test.artifactory.magmacore.org/" ? DOCKER_REGISTRY=,DOCKER_REGISTRY=docker.artifactory.magmacore.org
At the time of releasing this document, the docker images will be available. I don't think we should update this in docs.
Looks good to me.