Where communities thrive


  • Join over 1.5M+ people
  • Join over 100K+ communities
  • Free without limits
  • Create your own community
People
Repo info
Activity
  • 04:19

    kflynn on 0.80.0-rc1

    (compare)

  • 02:57

    kflynn on master

    Phase tweaks to the circuitbrea… (compare)

  • 02:54

    kflynn on dev

    (compare)

  • 02:54
    kflynn commented #1844
  • 02:54

    kflynn on master

    Hopefully-improved Ingress docs Switch getting-started to CRDs. Try to detect - and bypass - RB… and 6 more (compare)

  • 02:54
    kflynn closed #1844
  • 00:27
    stefansedich edited #1846
  • 00:26
    stefansedich edited #1846
  • 00:25
    stefansedich synchronize #1846
  • Sep 19 23:49
    stefansedich synchronize #1846
  • Sep 19 23:28
    stefansedich edited #1846
  • Sep 19 23:28
    stefansedich opened #1846
  • Sep 19 20:33
    ppeble synchronize #1845
  • Sep 19 20:23
    kflynn synchronize #1844
  • Sep 19 20:23

    kflynn on dev

    The Empty test also should have… Merge branch 'flynn/dev/0.80.0-… (compare)

  • Sep 19 20:22
    nbkrause synchronize #1844
  • Sep 19 20:21

    nbkrause on dev

    Fix mapping names Signed-off-b… (compare)

  • Sep 19 20:11
    ppeble opened #1845
  • Sep 19 20:10
    kflynn commented #1844
  • Sep 19 20:09
    kflynn synchronize #1844
datawirebot
@datawirebot
<rdl> I'd love to look into it but we just have a ton of stuff we're working on right now so I can't promise we'll get to this issue any time soon
<entone> for sure
<rdl> not that it's not important, of course
<entone> yeah, seems pretty fundamental
<entone> and it's just a simple boolean logic error
<entone> if_exists() rather than just if()
<rdl> if you want to PR even an untested thing, that would be fine
<rdl> we're working on CI to run for external contribs
<rdl> if it's that simple
<rdl> but the big blocker for us usually is the test writing
<entone> yeah, haha, I think that's everyone 🙂
datawirebot
@datawirebot
<flynn> So here’s the funny thing: we have tests for case_sensitive and they pass.
<flynn> @entone ^ Do you have a Mapping that demonstrates this?
datawirebot
@datawirebot
<Luke> apiVersion: ambassador/v1 kind: Mapping name: redirect-test host_redirect: true host: http://myapp.example.com prefix: /redirect case_sensitive: false path_redirect: / service: http://www.google.com
@flynn there is one that doesn't work for me
<flynn> @Luke Aha, that makes more sense then, if host_redirect is also involved. @entone, is that also your case?
datawirebot
@datawirebot
<entone> haha, yes, me and Luke work together
<entone> yeah, looking at the code, the path is drastically different for host_redirect
<entone> the code path that is
<entone> @Luke did we try it without host_redirect, I think I suggested that
<Luke> hmmm, i'm not entirely sure, just removing host_redirect all together?
datawirebot
@datawirebot
<flynn> @entone Ah cool. Happy to take a PR for this, of course! 🙂
<cory> for what its worth we have mappings with case_sensitive: False and they appear to work correctly
<entone> with a host_redirect?
<cory> no, without host_redirect
<cory> just noting that the feature doesn’t appear to be completely broken, so that might very well the cause
<entone> the function to generate the config needs some love in general, seems almost impossible to unit test it at this point
<entone> @flynn I'll see what I can do this week, but my Python ins't what it used to be, maybe it'll come flooding back once I'm in there 🙂
datawirebot
@datawirebot
<flynn> @entone It’s sadly complex, yes, especially given interdependencies between Mapping and other things. What I tend to do is to make shell, then use ambassador dump --ir --v2 $configdir > wtf.json — that way, the Ambassador IR and V2 Envoy config end up in wtf.json, and you can start by looking at that.
<flynn> (where $configdir is a directory with one or more Ambassador config resources — or if they’re annotated K8s resources, add the --k8s flag as well)
datawirebot
@datawirebot
<entone> awesome, thanks for the tip
datawirebot
@datawirebot

<Craig> Quick question on Cert-Manager integration with Ambassador. I setup my cert-manager and it is using lets-encrypt. Seems to work fine and I got a cert that will renew in 3 month(ish).

https://www.getambassador.io/user-guide/cert-manager

Do I need to keep that challenge service (acme-challenge-service) mapped for renewal or can I close it and renewal will still happen?

Eric Fulton
@Sahasrara
Hey there, quick question:
I was under the impression that service_port was used to dictate which port envoy listens on its public ip. That doesn't seem to be the case since as a non-root user, the ingress controller is listening on 80.
Even though its set to 8080 by default (and explicitly in my case)
Sharath MK
@Sharathmk99
Hi Team, I'm new to ambassador. Is it possible to integrate 3rd party IDP, if it's success then only route to internal service?
Daniel Barrigas
@danielguedesb
Hey guys, using ambassador 0.61.0 behind a gcp https lb and set up the redirect from http to https. However, for some reason, it enters a loop and keeps redirecting until it reaches a pre-defined limit.
  service_port: 8080
  x_forwarded_proto_redirect: true
  use_remote_address: false
This is the configuration on the k8s ambassador service, according to the docs it should work. Anything I might be missing here? Thanks
Hugo Barrigas
@hfgbarrigas
having the same issue as @danielguedesb , though version 0.40.0 works just fine
haozong
@dotw
@richarddli Hi, how to configure authentication for a specific ambassador instance if I have more than one instance in cluster?
I can't find any tips in the doc
Baily Troyer
@BailyTroyer
Is there better documentation to get up-and-running with ambasssador canary releases? Running the examples provided here simply don't work. https://blog.getambassador.io/canary-deployments-a-b-testing-and-microservices-with-ambassador-f104d0458736 & https://www.getambassador.io/docs/dev-guide/canary-release-concepts/
Saket Kashyap
@saketkashyap
Hi there, I followed the article - https://blog.getambassador.io/deploying-java-apps-with-kubernetes-and-the-ambassador-api-gateway-c6e9d9618f1b to deploy ambassador as LoadBalancer on Kubernetes cluster running on Docker-For-Desktop on windows 10.
I deployed my app, created a service of my app, exposed the service on nodeport to test the service, then configured it to cluster ip, deployed an ambassador service with mapping and then applied the https://getambassador.io/yaml/ambassador/ambassador-no-rbac.yaml
but , when i try to hit on localhost, the browser shows connection rest
can any one help?
Saket Kashyap
@saketkashyap
was able to resolve the issue
had an issue with targetPort in ambassador service
zhaohb
@zhaohb
now I met a problem datawire/ambassador#1761
can any one help ?