-
Notifications
You must be signed in to change notification settings - Fork 0
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
42crunch url #85
42crunch url #85
Conversation
Signed-off-by: jasmingacic <jasmin.gacic@gmail.com>
Signed-off-by: jasmingacic <jasmin.gacic@gmail.com>
Co-authored-by: Abdallah Abedraba <aabedraba@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM for the most part. Just a typo I found.
Signed-off-by: jasmingacic <jasmin.gacic@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
I'm not too sure about the update to github.com/kubeshop/kusk-gateway
to v1.5.1-0.20221207083051-c3507a110867
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
Summary ======= Merge changes from kubeshop/kuskgateway-api-server#85 that were missed in the migration of `kuskgateway-api-server` to `kusk-gateway`. `Makefile-kusk-gateway-api` --------------------------- Add targets to build images that were lost in the merge. Invoke as `make -f Makefile-kusk-gateway-api all`. --- Signed-off-by: Mohamed Bana <mohamed@bana.io>
Summary ======= Merge changes from kubeshop/kuskgateway-api-server#85 that were missed in the migration of `kuskgateway-api-server` to `kusk-gateway`. `Makefile-kusk-gateway-api` --------------------------- Add targets to build images that were lost in the merge. Invoke as `make -f Makefile-kusk-gateway-api all`. --- Signed-off-by: Mohamed Bana <mohamed@bana.io>
Summary ======= Merge changes from kubeshop/kuskgateway-api-server#85 that were missed in the migration of `kuskgateway-api-server` to `kusk-gateway`. `Makefile-kusk-gateway-api` --------------------------- Add targets to build images that were lost in the merge. Invoke as `make -f Makefile-kusk-gateway-api all`. --- Signed-off-by: Mohamed Bana <mohamed@bana.io>
This PR returns Crunch42URL in the APIItem.
In order to achieve that I added KGW dependency from the main branch hence
github.com/kubeshop/kusk-gateway v1.5.1-0.20221207083051-c3507a110867
Also you will notice I didn't handle any errors for 42Crunch as we don't want to break if we can't get the URL
Unblocks kubeshop/kusk-gateway#974
Signed-off-by: jasmingacic jasmin.gacic@gmail.com