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.

Sign up for GitHub

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

Jump to bottom

add controller and router packages and container images #1800

Closed
wants to merge 25 commits into from
Closed

add controller and router packages and container images #1800

wants to merge 25 commits into from

Conversation

qrkourier
Copy link
Member

No description provided.

@qrkourier qrkourier marked this pull request as ready for review March 7, 2024 18:36
@qrkourier qrkourier requested review from a team as code owners March 7, 2024 18:36
@qrkourier
Copy link
Member Author

qrkourier commented Mar 8, 2024 •
edited

Package builds

Container image builds

Ops Doc Preview

@qrkourier qrkourier marked this pull request as draft March 8, 2024 20:56
@qrkourier qrkourier marked this pull request as ready for review March 8, 2024 22:17
qrkourier
qrkourier commented Mar 11, 2024
View reviewed changes
.github/workflows/fablab-db-creation.yml Show resolved Hide resolved
.github/workflows/publish-linux-packages.yml Show resolved Hide resolved
dist/dist-packages/linux/nfpm-openziti-controller.yaml Outdated Show resolved Hide resolved
dist/dist-packages/linux/openziti-controller/bootstrap.bash Show resolved Hide resolved
dist/dist-packages/linux/openziti-controller/entrypoint.bash Outdated Show resolved Hide resolved
ziti/cmd/create/create_config_controller_test.go Outdated Show resolved Hide resolved
ziti/cmd/create/create_config_environment.go Outdated Show resolved Hide resolved
ziti/cmd/create/create_config_test.go Show resolved Hide resolved
ziti/cmd/helpers/env_helpers.go Outdated Show resolved Hide resolved
ziti/constants/constants.go Show resolved Hide resolved
@qrkourier qrkourier marked this pull request as draft March 12, 2024 21:06
@qrkourier qrkourier force-pushed the linux-package-controller branch 4 times, most recently from 461c4b5 to 8ed2af3 Compare March 14, 2024 20:30
@qrkourier qrkourier marked this pull request as ready for review March 14, 2024 21:48
@qrkourier
Copy link
Member Author

New CI artifacts tested in a repo fork:
https://github.com/qrkourier/ziti/actions/runs/8303508508

scareything
scareything reviewed Mar 19, 2024
View reviewed changes
dist/dist-packages/linux/openziti-controller/bootstrap.bash Outdated Show resolved Hide resolved
dist/dist-packages/linux/openziti-controller/bootstrap.bash Outdated Show resolved Hide resolved
ziti/constants/constants.go Outdated Show resolved Hide resolved
ziti/constants/constants.go Outdated Show resolved Hide resolved
ziti/constants/constants.go Outdated Show resolved Hide resolved
ziti/cmd/create/create_config_controller.go Show resolved Hide resolved
ziti/cmd/create/create_config_test.go Outdated Show resolved Hide resolved
Patrick Woodlock and others added 25 commits April 11, 2024 11:50
@qrkourier
Admission from Rockfield IT OpenZiti Adopters:
a94f273
@RockfieldIT @qrkourier
signed
5c34dd2
@plorenz @qrkourier
Return circuit id to client dial responses and hosting dial requests. F…
cf47862 
…ixes #1831
@plorenz @qrkourier
Update deps and changelog
c3e2366
@plorenz @qrkourier
Don't save old link statues in memory, the info is in link events. Fixes
f63497f 
 #1896
@plorenz @qrkourier
Optimize network run loop. Fixes #1897
dcc001b 
* When routers change (connect/disconnect), we re-check the mesh. We are passing in the routers to the channel and they can build up, which is unnecessary. We only need to signal once for all the routers that change since the last time we checked, that the mesh needs to be checked.
* We pass link changes through the run method, which is unnecessary since we pass it to a new goroutine. This inefficiency should be fixed.
* We're evaluating all faulted links for rerouting, even if they were still pending. Only reroute connected links.
* We're using time.After in each for loop, which can accumulate timers. Use a single Ticker instead.
@plorenz @qrkourier
Fix panic in PeerConnected. Fixes #1914
36e11b5
@plorenz @qrkourier
Fix panic on first ha controller startup. Fixes #1911
c3be5d8
@dependabot @qrkourier
Bump the third-party group with 6 updates
dece07a 
Bumps the third-party group with 6 updates:

| Package | From | To |
| --- | --- | --- |
| [github.com/hashicorp/go-hclog](https://github.com/hashicorp/go-hclog) | `1.6.2` | `1.6.3` |
| [github.com/jedib0t/go-pretty/v6](https://github.com/jedib0t/go-pretty) | `6.5.6` | `6.5.8` |
| [golang.org/x/crypto](https://github.com/golang/crypto) | `0.21.0` | `0.22.0` |
| [golang.org/x/net](https://github.com/golang/net) | `0.22.0` | `0.24.0` |
| [golang.org/x/sync](https://github.com/golang/sync) | `0.6.0` | `0.7.0` |
| gopkg.in/AlecAivazis/survey.v1 | `1.8.7` | `1.8.8` |


Updates `github.com/hashicorp/go-hclog` from 1.6.2 to 1.6.3
- [Release notes](https://github.com/hashicorp/go-hclog/releases)
- [Commits](hashicorp/go-hclog@v1.6.2...v1.6.3)

Updates `github.com/jedib0t/go-pretty/v6` from 6.5.6 to 6.5.8
- [Release notes](https://github.com/jedib0t/go-pretty/releases)
- [Commits](jedib0t/go-pretty@v6.5.6...v6.5.8)

Updates `golang.org/x/crypto` from 0.21.0 to 0.22.0
- [Commits](golang/crypto@v0.21.0...v0.22.0)

Updates `golang.org/x/net` from 0.22.0 to 0.24.0
- [Commits](golang/net@v0.22.0...v0.24.0)

Updates `golang.org/x/sync` from 0.6.0 to 0.7.0
- [Commits](golang/sync@v0.6.0...v0.7.0)

Updates `gopkg.in/AlecAivazis/survey.v1` from 1.8.7 to 1.8.8

---
updated-dependencies:
- dependency-name: github.com/hashicorp/go-hclog
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: third-party
- dependency-name: github.com/jedib0t/go-pretty/v6
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: third-party
- dependency-name: golang.org/x/crypto
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: third-party
- dependency-name: golang.org/x/net
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: third-party
- dependency-name: golang.org/x/sync
  dependency-type: direct:production
  update-type: version-update:semver-minor
  dependency-group: third-party
- dependency-name: gopkg.in/AlecAivazis/survey.v1
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: third-party
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot @qrkourier
Bump lewagon/wait-on-check-action from 1.3.3 to 1.3.4 in the all group
20e86a5 
Bumps the all group with 1 update: [lewagon/wait-on-check-action](https://github.com/lewagon/wait-on-check-action).


Updates `lewagon/wait-on-check-action` from 1.3.3 to 1.3.4
- [Release notes](https://github.com/lewagon/wait-on-check-action/releases)
- [Commits](lewagon/wait-on-check-action@v1.3.3...v1.3.4)

---
updated-dependencies:
- dependency-name: lewagon/wait-on-check-action
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: all
...

Signed-off-by: dependabot[bot] <support@github.com>
@plorenz @qrkourier
Fix SDK dial error when 'token is malformed'. Api session JWT wasn't …
d7be895 
…being forward to controller. Fixes #1916
@plorenz @qrkourier
Improve link mgmt backwards compatibility. Fixes #1900
9835220 
* We're sending peer change messages to older routers that don't understand them.
* We should probably use the older controller link management code for routers without the link management stability fixes
* Includes a few minor cleanups
@dependabot @qrkourier
Bump the openziti group with 7 updates
0ee522a 
Bumps the openziti group with 7 updates:

| Package | From | To |
| --- | --- | --- |
| [github.com/openziti/channel/v2](https://github.com/openziti/channel) | `2.0.122` | `2.0.125` |
| [github.com/openziti/foundation/v2](https://github.com/openziti/foundation) | `2.0.40` | `2.0.42` |
| [github.com/openziti/identity](https://github.com/openziti/identity) | `1.0.73` | `1.0.74` |
| [github.com/openziti/metrics](https://github.com/openziti/metrics) | `1.2.48` | `1.2.49` |
| [github.com/openziti/runzmd](https://github.com/openziti/runzmd) | `1.0.41` | `1.0.42` |
| [github.com/openziti/storage](https://github.com/openziti/storage) | `0.2.33` | `0.2.34` |
| [github.com/openziti/transport/v2](https://github.com/openziti/transport) | `2.0.125` | `2.0.128` |


Updates `github.com/openziti/channel/v2` from 2.0.122 to 2.0.125
- [Commits](openziti/channel@v2.0.122...v2.0.125)

Updates `github.com/openziti/foundation/v2` from 2.0.40 to 2.0.42
- [Commits](openziti/foundation@v2.0.40...v2.0.42)

Updates `github.com/openziti/identity` from 1.0.73 to 1.0.74
- [Commits](openziti/identity@v1.0.73...v1.0.74)

Updates `github.com/openziti/metrics` from 1.2.48 to 1.2.49
- [Commits](openziti/metrics@v1.2.48...v1.2.49)

Updates `github.com/openziti/runzmd` from 1.0.41 to 1.0.42
- [Commits](openziti/runzmd@v1.0.41...v1.0.42)

Updates `github.com/openziti/storage` from 0.2.33 to 0.2.34
- [Commits](openziti/storage@v0.2.33...v0.2.34)

Updates `github.com/openziti/transport/v2` from 2.0.125 to 2.0.128
- [Commits](openziti/transport@v2.0.125...v2.0.128)

---
updated-dependencies:
- dependency-name: github.com/openziti/channel/v2
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: openziti
- dependency-name: github.com/openziti/foundation/v2
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: openziti
- dependency-name: github.com/openziti/identity
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: openziti
- dependency-name: github.com/openziti/metrics
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: openziti
- dependency-name: github.com/openziti/runzmd
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: openziti
- dependency-name: github.com/openziti/storage
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: openziti
- dependency-name: github.com/openziti/transport/v2
  dependency-type: direct:production
  update-type: version-update:semver-patch
  dependency-group: openziti
...

Signed-off-by: dependabot[bot] <support@github.com>
@plorenz @qrkourier
Use cmap IterCB instead of IterBuffered where possible. Fixes #1902
608a222
@scareything @qrkourier
respect service policies for admin identities (#1919)
6080e95 
respect service policies for admin identities
@plorenz @qrkourier
Do some CLI tidying. Fixes #1921
ca56c0d 
* Remove version, use, install and upgrade commands
* Replace version with command which outputs CLI version
* Remove redundant version commands
* Remove ziti fabric add/remove-identity commands
* Remove lots of unused code
* Remove 'ziti tunnel enroll' command, which is replaced by 'ziti edge enroll'
@plorenz @qrkourier
Rename simple-transfer to smoketest
abf10bc
@plorenz @qrkourier
Load test for controller with links, terminators, clients and traffic
35add35
@plorenz @qrkourier
Add TLS handshake rate limiter. Fixes #1904
b435194
@dovholuknf @qrkourier
remove oidc endpoint from controller config gen for now
4497db1
@plorenz @qrkourier
Add release validation workflow. Fixes #1923
bb173f9
@plorenz @qrkourier
Move rate limiter logging to debug
109797e
@plorenz @qrkourier
Set version to 1.0
a9c6168
@plorenz @qrkourier
Update changelog, deps and make the tls handshake rate limiter off by…
ff18231 
… default
@qrkourier
add Linux packages and container bootstrapping for controller and router
b1706e7
@qrkourier qrkourier closed this Apr 11, 2024
@qrkourier qrkourier deleted the linux-package-controller branch April 11, 2024 15:51
@qrkourier qrkourier mentioned this pull request Apr 11, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@scareything scareything Awaiting requested review from scareything

@dovholuknf dovholuknf Awaiting requested review from dovholuknf

Assignees
No one assigned
Labels
quickstartrelease Release a quickstart container image and update the quickstart-latest Git tag used by /quick/ URLs
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@qrkourier @scareything @dovholuknf @RockfieldIT @plorenz