Update README.md and scripts to adopt atespaces for existing demos.#355
Open
Haven Xia (HavenXia) wants to merge 1 commit into
Open
Update README.md and scripts to adopt atespaces for existing demos.#355Haven Xia (HavenXia) wants to merge 1 commit into
Haven Xia (HavenXia) wants to merge 1 commit into
Conversation
Haven Xia (HavenXia)
force-pushed
the
atespace-readme
branch
from
287c420 to
2981451
Compare
Haven Xia (HavenXia)
force-pushed
the
atespace-readme
branch
from
2981451 to
6cc63aa
Compare
|
|
||
| // 1. Identify Actor (Robust extraction) | ||
| // New architecture uses Host: <actor-id>.actors.resources.substrate.k8s.io | ||
| // New architecture uses Host: <actor-id>.<atespace>.actors.resources.substrate.ate.dev |
Collaborator
There was a problem hiding this comment.
What does "New Architecture" means here? Maybe reword it?
| kubectl ate delete actor my-counter-1 | ||
| kubectl ate delete actor my-counter-1 -a demo | ||
| ``` | ||
|
|
Collaborator
There was a problem hiding this comment.
Should we also delete the namespace here? Same for other demos.
|
|
||
| ### Workload Connectivity (Uniform DNS) | ||
| Substrate has standardized on a **Uniform DNS Mesh**. You no longer need to define `SessionDiscovery` rules. Every actor created from a template is automatically reachable through the **Substrate Router** via its unique ID: | ||
| Substrate has standardized on a **Uniform DNS Mesh**. You no longer need to define `SessionDiscovery` rules. Every actor created from a template is automatically reachable through the **Substrate Router** via its atespace and ID: |
Collaborator
There was a problem hiding this comment.
Remove the "You no longer need to define SessionDiscovery rules."? Seems like an old artifact.
Collaborator
There was a problem hiding this comment.
This file (and the _grpc) companion are auto-generated, we should change the documentation in ateapi.go and then regenerate files.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Update README.md and scripts to adopt atespaces for existing demos.
It's intentional to let user choose atespace in demos since actors are also created by them.