Add multi-level container domain name support #60
No reviewers
Labels
No labels
P0
P1
P2
P3
good first issue
Infrastructure
blocked
bug
config
discussion
documentation
duplicate
enhancement
go
help wanted
internal
invalid
kludge
observability
perfomance
question
refactoring
wontfix
No milestone
No project
No assignees
5 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference: TrueCloudLab/frostfs-contract#60
Loading…
Reference in a new issue
No description provided.
Delete branch "elebedeva/frostfs-contract:fix-container-domain"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Close #56
Signed-off-by: Ekaterina Lebedeva ekaterina.lebedeva@yadro.com
b56b74cb2d
to6f0764d013
6f0764d013
to7d44583ee3
7d44583ee3
to281ce322c7
WIP: Add multi-level container domain name supportto Add multi-level container domain name support@ -232,3 +232,3 @@
return true
}
return parentExpired(ctx, 0, fragments)
if parentExpired(ctx, 1, fragments) {
It seems now
parentExpired
always accepts the1
as the second arg. Should we drop this arg at all?fixed
besides mentined comments, LGTM
@ -334,6 +334,8 @@ func TestNNSIsAvailable(t *testing.T) {
"myemail@frostfs.info", refresh, retry, expire, ttl)
c.Invoke(t, false, "isAvailable", "domain.com")
c.Invoke(t, true, "isAvailable", "dom.domain.com")
The problem in case was for
Register
, could you also add an explicit test for it?added
281ce322c7
to4dcb575caa