Skip to content

chore: remove legacy AgentIP address #14640

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Sep 12, 2024
Merged

Conversation

spikecurtis
Copy link
Contributor

@spikecurtis spikecurtis commented Sep 11, 2024

Removes the support for the Agent's "legacy IP" which was a hardcoded IP address all agents used to use, before we introduced "single tailnet". Single tailnet went GA in 2.7.0.

Copy link
Contributor Author

spikecurtis commented Sep 11, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @spikecurtis and the rest of your teammates on Graphite Graphite

@spikecurtis spikecurtis marked this pull request as ready for review September 11, 2024 09:56
@spikecurtis spikecurtis force-pushed the spike/legacy-agent-ip branch 3 times, most recently from 4f41732 to bc16937 Compare September 11, 2024 12:00
@@ -11,8 +11,7 @@ import {
} from "../helpers";
import { beforeCoderTest } from "../hooks";

// we no longer support versions prior to single tailnet: https://github.com/coder/coder/commit/d7cbdbd9c64ad26821e6b35834c59ecf85dcd9d4
const clientVersion = "v0.27.0";
const clientVersion = "v2.8.0";
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Theoretically, this should work with 2.7.0. I tried it and it doesn't work. Rather than try to debug, I just set it to 2.8.0, which is what it'll be in the PR above this one on the stack.

Copy link
Contributor Author

spikecurtis commented Sep 12, 2024

Merge activity

@spikecurtis spikecurtis merged commit fb3523b into main Sep 12, 2024
30 checks passed
@spikecurtis spikecurtis deleted the spike/legacy-agent-ip branch September 12, 2024 03:40
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.

2 participants