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

Replace httpbin with mocktarget #683

Merged
merged 4 commits into from
Aug 8, 2023

Conversation

danistrebel
Copy link
Collaborator

@danistrebel danistrebel commented Aug 7, 2023

Description

What's changed, or what was fixed?

  • replace httpbin for mocktarget to improve test stability
  • Currently out of scope and still rely on httpbin.org
    • OIDC Mocktarget / Identityfacade / Lab Okta integration (This requires query param support on the mock backend)
    • Envoy Quickstart (Some analysis required)

Issues Fixed

  • Fixes Nightly build failures

Housekeeping

(please check all that apply [x], do not edit the text)

  • I have run all the tests locally and they all pass.
  • I have followed the relevant style guide for my changes.

Full Repo Validation Required

(please check all that apply [x], do not edit the text)

  • PR requires full pipeline run (Run for changes only by default).

CC: @apigee-devrel-reviewers

@danistrebel
Copy link
Collaborator Author

/gcbrun

@apigee-devrel-bot
Copy link

Pipeline Report

Pipeline Result Elapsed Wall Time
references/cicd-sharedflow-pipeline pass 219s
references/identity-facade fail 61s
references/js-callout pass 109s
references/proxy-template pass 286s
references/recaptcha-enterprise pass 171s
tools/endpoints-oas-importer pass 174s
TOTAL PIPELINE pass 1020s

View details in Cloud Build (permission required)

Commit version: 33bf3f9

@danistrebel
Copy link
Collaborator Author

/gcbrun

@apigee-devrel-bot
Copy link

Pipeline Report

Pipeline Result Elapsed Wall Time
references/cicd-sharedflow-pipeline pass 202s
references/dutch-healthcare fail 296s
references/java-callout fail 21s
references/js-callout pass 107s
references/proxy-template pass 312s
references/recaptcha-enterprise pass 188s
tools/endpoints-oas-importer pass 172s
TOTAL PIPELINE pass 1298s

View details in Cloud Build (permission required)

Commit version: 58a8185

@danistrebel
Copy link
Collaborator Author

/gcbrun

@apigee-devrel-bot
Copy link

Pipeline Report

Pipeline Result Elapsed Wall Time
references/cicd-sharedflow-pipeline pass 193s
references/dutch-healthcare pass 288s
references/java-callout fail 18s
references/js-callout pass 104s
references/proxy-template pass 307s
references/recaptcha-enterprise pass 170s
tools/endpoints-oas-importer pass 174s
TOTAL PIPELINE pass 1254s

View details in Cloud Build (permission required)

Commit version: 37c14eb

@danistrebel
Copy link
Collaborator Author

/gcbrun

@apigee-devrel-bot
Copy link

Pipeline Report

Pipeline Result Elapsed Wall Time
references/cicd-sharedflow-pipeline pass 231s
references/dutch-healthcare pass 291s
references/java-callout pass 114s
references/js-callout pass 107s
references/proxy-template pass 309s
references/recaptcha-enterprise pass 164s
tools/endpoints-oas-importer pass 166s
TOTAL PIPELINE pass 1382s

View details in Cloud Build (permission required)

Commit version: 87a376f

@danistrebel
Copy link
Collaborator Author

/gcbrun

@apigee-devrel-bot
Copy link

Pipeline Report

Pipeline Result Elapsed Wall Time
references/cicd-sharedflow-pipeline pass 228s
references/dutch-healthcare pass 295s
references/identity-facade pass 246s
references/java-callout pass 105s
references/js-callout pass 109s
references/oidc-mock pass 16s
references/proxy-template pass 306s
references/recaptcha-enterprise pass 179s
tools/apigee-envoy-quickstart pass 383s
tools/endpoints-oas-importer pass 178s
TOTAL PIPELINE pass 2045s

View details in Cloud Build (permission required)

Commit version: b3fcfb9

@danistrebel
Copy link
Collaborator Author

/gcbrun

1 similar comment
@OmidTahouri
Copy link
Collaborator

/gcbrun

@apigee-devrel-bot
Copy link

Pipeline Report

Pipeline Result Elapsed Wall Time
references/cicd-sharedflow-pipeline pass 235s
references/dutch-healthcare pass 311s
references/identity-facade pass 251s
references/java-callout pass 116s
references/js-callout pass 109s
references/oidc-mock pass 14s
references/proxy-template pass 303s
references/recaptcha-enterprise pass 169s
tools/apigee-envoy-quickstart pass 380s
tools/endpoints-oas-importer pass 162s
TOTAL PIPELINE pass 2050s

View details in Cloud Build (permission required)

Commit version: e4c2280

Copy link
Collaborator

@OmidTahouri OmidTahouri left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@danistrebel danistrebel merged commit be1cb00 into apigee:main Aug 8, 2023
7 checks passed
@danistrebel danistrebel mentioned this pull request Aug 8, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants