Skip to content
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

[FEATURE] BREAKING: default port mappings via Loadbalancer + new nodefilter syntax #656

Merged
merged 18 commits into from
Jul 7, 2021

Conversation

iwilltry42
Copy link
Member

@iwilltry42 iwilltry42 commented Jun 28, 2021

see #652

To-Do

  • handle edge cases like --port 1234:4321@loadbalancer:0:proxy (loadbalancer identifier does not work with indexing or opts)
  • make all:proxy the default nodefilter for --port

@iwilltry42 iwilltry42 added this to the v5.0.0 milestone Jun 28, 2021
@iwilltry42 iwilltry42 self-assigned this Jun 28, 2021
@iwilltry42 iwilltry42 force-pushed the breaking/port-default-lb branch 2 times, most recently from 7470995 to 85c9db3 Compare July 6, 2021 11:19
@iwilltry42 iwilltry42 changed the base branch from main to main-v5 July 6, 2021 11:19
- generate node names when transforming from simple to cluster config
  - ClusterCreate(clusterconfig) should have a ready made config and not
  generate variables
- ClusterCreate() only prep LB if not already present (to be removed)
- cluster struct: serverloadbalancer is now of type LoadBalancer (Node +
Config)
- use new nodefilter syntax with 'id:index:suffix' instead of
'id[index]' everywhere
  - use suffix when creating the LB
@iwilltry42 iwilltry42 force-pushed the breaking/port-default-lb branch from 85c9db3 to 94d2792 Compare July 6, 2021 11:26
@iwilltry42 iwilltry42 marked this pull request as ready for review July 6, 2021 12:30
@iwilltry42 iwilltry42 changed the title [FEATURE] BREAKING: default port mappings via Loadbalancer [FEATURE] BREAKING: default port mappings via Loadbalancer + new nodefilter syntax Jul 7, 2021
@iwilltry42 iwilltry42 merged commit 99490ca into main-v5 Jul 7, 2021
@iwilltry42 iwilltry42 deleted the breaking/port-default-lb branch July 7, 2021 13:30
@iwilltry42 iwilltry42 linked an issue Jul 20, 2021 that may be closed by this pull request
somaritane referenced this pull request in AbsaOSS/k3d-action Nov 15, 2021
- k3d config examples migrated to `apiVersion: k3d.io/v1alpha3`
  [https://github.com/rancher/k3d/pull/605](https://github.com/rancher/k3d/pull/605)
- `:direct` node filter option used when load balancer is disabled
  [https://github.com/rancher/k3d/pull/656](https://github.com/rancher/k3d/pull/656)
- `--label` --> `--runtime-label`
  [https://github.com/rancher/k3d/pull/598](https://github.com/rancher/k3d/pull/598)
- `--k3s-server-arg` --> `--k3s-arg` with server node filter (`@server:*`)
  [https://github.com/rancher/k3d/pull/605](https://github.com/rancher/k3d/pull/605)
- Updated documentation

Signed-off-by: Timofey Ilinykh <[email protected]>
somaritane referenced this pull request in AbsaOSS/k3d-action Nov 15, 2021
- k3d config examples migrated to `apiVersion: k3d.io/v1alpha3`
  [https://github.com/rancher/k3d/pull/605](https://github.com/rancher/k3d/pull/605)
- `:direct` node filter option used when load balancer is disabled
  [https://github.com/rancher/k3d/pull/656](https://github.com/rancher/k3d/pull/656)
- `--label` --> `--runtime-label`
  [https://github.com/rancher/k3d/pull/598](https://github.com/rancher/k3d/pull/598)
- `--k3s-server-arg` --> `--k3s-arg` with server node filter (`@server:*`)
  [https://github.com/rancher/k3d/pull/605](https://github.com/rancher/k3d/pull/605)
- Updated documentation

Signed-off-by: Timofey Ilinykh <[email protected]>
somaritane referenced this pull request in AbsaOSS/k3d-action Nov 16, 2021
- k3d config examples migrated to `apiVersion: k3d.io/v1alpha3`
  [https://github.com/rancher/k3d/pull/605](https://github.com/rancher/k3d/pull/605)
- `:direct` node filter option used when load balancer is disabled
  [https://github.com/rancher/k3d/pull/656](https://github.com/rancher/k3d/pull/656)
- `--label` --> `--runtime-label`
  [https://github.com/rancher/k3d/pull/598](https://github.com/rancher/k3d/pull/598)
- `--k3s-server-arg` --> `--k3s-arg` with server node filter (`@server:*`)
  [https://github.com/rancher/k3d/pull/605](https://github.com/rancher/k3d/pull/605)
- removed `disableImageVolume` option from examples and test assets
- Updated documentation

Signed-off-by: Timofey Ilinykh <[email protected]>
somaritane referenced this pull request in AbsaOSS/k3d-action Nov 16, 2021
- k3d config examples migrated to `apiVersion: k3d.io/v1alpha3`
  [https://github.com/rancher/k3d/pull/605](https://github.com/rancher/k3d/pull/605)
- `:direct` node filter option used when load balancer is disabled
  [https://github.com/rancher/k3d/pull/656](https://github.com/rancher/k3d/pull/656)
- `--label` --> `--runtime-label`
  [https://github.com/rancher/k3d/pull/598](https://github.com/rancher/k3d/pull/598)
- `--k3s-server-arg` --> `--k3s-arg` with server node filter (`@server:*`)
  [https://github.com/rancher/k3d/pull/605](https://github.com/rancher/k3d/pull/605)
- removed `disableImageVolume` option from examples and test assets
- Updated documentation

Signed-off-by: Timofey Ilinykh <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[FEATURE] Expose ports AFTER k3d creation
1 participant