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

Sthwang/make port optional #2

Merged
merged 76 commits into from
Jan 17, 2024
Merged

Sthwang/make port optional #2

merged 76 commits into from
Jan 17, 2024

Conversation

sthwang-metal
Copy link
Owner

No description provided.

rizzza and others added 30 commits August 28, 2023 22:24
@rizzza
add origin target ip address validation (infratographer#213)
31c32cb 
Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>
@rizzza
Use new event hooks (infratographer#212)
ecdd625 
* new manual event hooks

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

* update tests for auth-relationships

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

* relationships

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

* add location to manual hook

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

* finesse manual hook

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

---------

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>
@renovate
Update module github.com/99designs/gqlgen to v0.17.36 (infratographer…
5ae9392 
…#200)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate
Update module go.infratographer.com/permissions-api to v0.2.5 (infrat…
97712d9 
…ographer#216)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@rizzza
loadbalancer-api client (infratographer#217)
881485c 
Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>
@tylerauerbeck
Add restricted ports (infratographer#219)
e947d17 
* Add configuration to define restricted ports which the API cannot create for a loadbalancer

Signed-off-by: Tyler Auerbeck <tylerauerbeck@users.noreply.github.com>

* Cleanup

Signed-off-by: Tyler Auerbeck <tylerauerbeck@users.noreply.github.com>

* Use validate func in schema to check restricted ports

Signed-off-by: Tyler Auerbeck <tylerauerbeck@users.noreply.github.com>

* Fix lints

Signed-off-by: Tyler Auerbeck <tylerauerbeck@users.noreply.github.com>

---------

Signed-off-by: Tyler Auerbeck <tylerauerbeck@users.noreply.github.com>
Co-authored-by: Tyler Auerbeck <tylerauerbeck@users.noreply.github.com>
@renovate
Update Helm release reloader to v1.0.39 (infratographer#167)
b46a5ae 
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate
Update Helm release common to v2.9.2 (infratographer#195)
4b700b8 
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@rizzza
implement MetadataNode (infratographer#218)
d51c67c 
Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>
@rizzza
Fix up mutation hooks to ensure additionalSubjects are present in the…
c02b206 
… change message (infratographer#231)

* query all for hooks and add additionalSubjects to msg

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

* origin delete additional subjects

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

---------

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>
@renovate
Update module github.com/99designs/gqlgen to v0.17.38 (infratographer…
055d5a1 
…#225)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate
Update module go.infratographer.com/x to v0.3.9 (infratographer#233)
26b0bd2 
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate
Update module github.com/wundergraph/graphql-go-tools to v1.66.3 (inf…
de7fc35 
…ratographer#146)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate
Update module github.com/Yamashou/gqlgenc to v0.15.1 (infratographer#234
975231c 
)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate
Update module github.com/wundergraph/graphql-go-tools to v1.66.4 (inf…
9c0f290 
…ratographer#236)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate
Update module github.com/testcontainers/testcontainers-go/modules/pos…
bbf1d30 
…tgres to v0.24.1 (infratographer#237)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate
Update module go.infratographer.com/permissions-api to v0.2.6 (infrat…
bdb4f0d 
…ographer#223)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate
Update Helm release reloader to v1.0.41 (infratographer#222)
39a7f66 
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate
Update module go.uber.org/zap to v1.26.0 (infratographer#238)
ffcf1d5 
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@sthwang-metal
Add Resolver Validations (infratographer#243)
9f449dd 
* add more checks to resolvers

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>

* add tests

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>

* add more tests

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>

* update permission checks

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>

* update query method

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>

* remove tests

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>

* update error

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>

---------

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>
@rizzza
Ensure multiple locations are set in additionalSubjects, if required (i…
7340843 
…nfratographer#242)

* generate :allthethings:

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

* handle case to populate multiple location ids

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

* inverted logic in port delete hook

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

---------

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>
@rizzza
Move loadbalancer delete auth relationships into resolver (infratogra…
9935d1c 
…pher#230)

* move loadbalancer delete auth relationships into resolver

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

* bump to go 1.21 to get context.WithoutCancel

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

* disable entc extension for generating hooks

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

* use go 1.21 context.WithCancel for delete auth relationship

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

* testcontainers bug fix, back to go.mod version

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

---------

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>
@rizzza
Resolver input/error handling and logging (infratographer#228)
0218550 
* remove old

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

* loadbalancer error logs, do no expose internals

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

* origin error logs, do no expose internals

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

* handle invalid gidx

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

* pool error logs, do no expose internals

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

* port error logs, do no expose internals

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

* provider error logs, do no expose internals

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

* minor logging additions

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

* rollback error log

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

* last look

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

---------

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>
@sthwang-metal
manualhooks tests (infratographer#245)
f986072 
* write manualhooks tests

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>

* add init script

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>

* get init script

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>

* terminate container only if running

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>

* refactor setup and teardown

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>

* remove leftover testdata

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>

* refactor to re-use code

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>

* use pointer receivers

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>

* use TestMain pragma, swap ifErrPanic, add t to sub

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>

* add more assertions, add createXXX tests

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>

* less code

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>

---------

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>
@sthwang-metal @tylerauerbeck
add origin weights (infratographer#251)
7ef91b3 
* Add ability to set weight for an origin

Signed-off-by: Tyler Auerbeck <tylerauerbeck@users.noreply.github.com>

* Linters gonna lint

Signed-off-by: Tyler Auerbeck <tylerauerbeck@users.noreply.github.com>

* add weight to origin node

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>

* add migration to add origin weight column

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>

* postgres compatible

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>

* change to int instead of bigint

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>

---------

Signed-off-by: Tyler Auerbeck <tylerauerbeck@users.noreply.github.com>
Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>
Co-authored-by: Tyler Auerbeck <tylerauerbeck@users.noreply.github.com>
@renovate
Update module github.com/labstack/echo/v4 to v4.11.2 (infratographer#252
59340e5 
)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@sthwang-metal @rizzza
Refactor Existing Tests (infratographer#253)
019e34a 
* refactor setup and teardown

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>

* refactor builders

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>

* generate code

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>

* use TestMain pragma, swap ifErrPanic, add t to sub

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>

---------

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>
Co-authored-by: Matt Siwiec <rizzza@users.noreply.github.com>
@renovate
Update module github.com/spf13/viper to v1.17.0 (infratographer#255)
79b13b6 
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate
Update docker/setup-qemu-action action to v3 (infratographer#240)
2113d58 
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate
Update module github.com/testcontainers/testcontainers-go/modules/pos…
26fb5ec 
…tgres to v0.25.0 (infratographer#246)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
tylerauerbeck and others added 29 commits November 20, 2023 22:13
@tylerauerbeck
Adds ability to limit number of loadbalancers an owner can create (in…
780f19f 
…fratographer#281)

* Adds ability to limit number of loadbalancers an owner can create

Signed-off-by: Tyler Auerbeck <tylerauerbeck@users.noreply.github.com>

* Fix data race in test

Signed-off-by: Tyler Auerbeck <tylerauerbeck@users.noreply.github.com>

---------

Signed-off-by: Tyler Auerbeck <tylerauerbeck@users.noreply.github.com>
Co-authored-by: Tyler Auerbeck <tylerauerbeck@users.noreply.github.com>
@rizzza
Add node Metadata fragment query to client (infratographer#283)
3d091d9 
* metadata node

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

* add client query for node metadata

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

---------

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>
@rizzza
Add ip address metadata types; update metadata helper (infratographer…
63fdc7a 
…#284)

* add ip address metadata types

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

* flexible metadata helper

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

* add load-balancer-api metadata source

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

---------

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>
@renovate
Update Helm release reloader to v1.0.52 (infratographer#249)
40a661c 
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@sthwang-metal
(DEL-2899) Make Port Name Optional (infratographer#287)
d85a9ac 
* make port name optional

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>

* fix unit tests

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>

---------

Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>
@rizzza
Update metadata integration (infratographer#289)
627379f 
* do not return internal server error on failure to update metadata

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

* add deployment extraEnvFrom for additional environment

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

---------

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>
@rizzza
Check owner permissions in resolvers (infratographer#291)
6bb9a56 
* check permissions against ownerID, rather than resource

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

* establish permissions auth-relationship in ports

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

---------

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>
@rizzza
associate object relationship only on create (infratographer#294)
b3afa82 
Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>
@rizzza
move query for metadata out of transaction (infratographer#296)
c7072e9 
Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>
@renovate
Update module github.com/mattn/go-sqlite3 to v1.14.19 (infratographer…
fa07f68 
…#293)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate
Update module go.infratographer.com/permissions-api to v0.2.7 (infrat…
c183b8a 
…ographer#280)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate
Update module github.com/hasura/go-graphql-client to v0.10.1 (infrato…
5f701ab 
…grapher#286)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate
Update module github.com/brianvoe/gofakeit/v6 to v6.26.0 (infratograp…
6e45cfa 
…her#285)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate
Update actions/setup-go action to v5 (infratographer#300)
20d537c 
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate
Update module github.com/brianvoe/gofakeit/v6 to v6.26.3 (infratograp…
d2bec89 
…her#301)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@rizzza
origins and ports do not need to establish auth relationships (infrat…
19a92fd 
…ographer#302)

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>
@sthwang-metal
add optional label
5d3fbb6 
Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>
@sthwang-metal
add migration
fb722ea 
Signed-off-by: Stephen Hwang <126002920+sthwang-metal@users.noreply.github.com>
@renovate
Update ghcr.io/infratographer/load-balancer-api Docker tag to v0.1.2 (i…
67a7d06 
…nfratographer#299)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate
Update Helm release common to v2.14.1 (infratographer#297)
e5171c3 
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate
Update Helm release reloader to v1.0.58 (infratographer#292)
b4721c0 
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@rizzza
Create port additionalSubjects not populated in all cases (infratogra…
d9fefb0 
…pher#308)

* ignore go work files

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

* oidc=false

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

* ensure additionalSubjects are present in port create when no pool ids associated

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>

---------

Signed-off-by: Matt Siwiec <rizzza@users.noreply.github.com>
@renovate
Update module github.com/labstack/echo/v4 to v4.11.4 (infratographer#303
9202148 
)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate
Update ghcr.io/infratographer/load-balancer-api Docker tag to v0.1.3 (i…
939930a 
…nfratographer#310)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate @rizzza
Update module github.com/hasura/go-graphql-client to v0.10.2 (infrato…
1e53aef 
…grapher#309)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: Matt Siwiec <rizzza@users.noreply.github.com>
@renovate
Update testcontainers-go monorepo to v0.27.0 (infratographer#305)
2cd7b95 
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate
Update module github.com/docker/go-connections to v0.5.0 (infratograp…
819381c 
…her#306)

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@renovate
Update golang.org/x/exp digest to be819d1 (infratographer#282)
ec08118 
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
@sthwang-metal
Merge branch 'main' into sthwang/make-port-optional
0fb8053
@sthwang-metal sthwang-metal merged commit 47cd740 into main Jan 17, 2024
16 of 17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sthwang-metal @rizzza @tylerauerbeck