Skip to content
This repository has been archived by the owner on Jul 12, 2023. It is now read-only.

Create e2e-runner service. #417

Merged
merged 5 commits into from
Aug 30, 2020
Merged

Conversation

yegle
Copy link
Contributor

@yegle yegle commented Aug 29, 2020

This commit refactored the existing e2e-test CLI into a web service.

It currently provides /default and /revise handlers, maps to the
-revise=false and -revise=true flag of the original CLI.

This service is currently guarded by an IAM policy to avoid hammering
the API servers repeatedly.

Fixes #265

Release Note*

A simple e2e test is added to test APIs provided by the verification server and the key server.

@googlebot googlebot added the cla: yes Auto: added by CLA bot when all committers have signed a CLA. label Aug 29, 2020
@google-cla google-cla bot added the cla: yes Auto: added by CLA bot when all committers have signed a CLA. label Aug 29, 2020
This commit refactored the existing e2e-test CLI into a web service.

It currently provides /default and /revise handlers, maps to the
`-revise=false` and `-revise=true` flag of the original CLI.

This service is currently guarded by an IAM policy to avoid hammering
the API servers repeatedly.
@yegle
Copy link
Contributor Author

yegle commented Aug 29, 2020

/cc @icco

cmd/e2e-runner/e2e.go Outdated Show resolved Hide resolved
cmd/e2e-runner/main.go Show resolved Hide resolved
cmd/e2e-runner/main.go Outdated Show resolved Hide resolved
pkg/config/e2e_runner_server_config.go Show resolved Hide resolved
scripts/build Show resolved Hide resolved
@yegle
Copy link
Contributor Author

yegle commented Aug 30, 2020

All comments addressed. PTAL.

Copy link
Contributor

@icco icco left a comment

Choose a reason for hiding this comment

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

/lgtm

@google-oss-robot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: icco, yegle

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@google-oss-robot google-oss-robot merged commit 583963b into google:main Aug 30, 2020
@google google locked and limited conversation to collaborators Oct 6, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes Auto: added by CLA bot when all committers have signed a CLA.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

e2e test container
4 participants