Skip to content
This repository has been archived by the owner on Oct 23, 2024. It is now read-only.

*taskID*.slave.mesos can be an invalid A record. #509

Open
drewkerrigan opened this issue Oct 9, 2017 · 0 comments
Open

*taskID*.slave.mesos can be an invalid A record. #509

drewkerrigan opened this issue Oct 9, 2017 · 0 comments

Comments

@drewkerrigan
Copy link
Contributor

When lookup fails in generator.go's slaveRecords for slave.PID.Host, we default to using labels.DomainFrag(slave.PID.Host, labels.Sep, spec) for the rg.SlaveIPs[slave.ID] value.

Later in taskContextRecord, we use that for *taskID*.slave.mesos records. A test case for this and possibly a fix should be created so that invalid A records are not created in this situation.

My suspicion is that a failure would occur later on down the line because resolver.go will attempt a .To4() on all A records which should fail this case.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant