Skip to content

New Cloud Run end-to-end deployment facility #1991

Closed

Description

@grayside

Cloud Run's current facility to deploy end-to-end tests is a bundle of BASH scripts. They seem very prone to flakes, and have generally proven difficult to maintain. #1740 is just the latest case of a flaky test.

A new approach to deploying the samples is needed, one which de-emphasizes BASH. Step 1 I'll put together a proposal.

Activity

  1. added
    priority: p1Important issue which blocks shipping the next release. Will be fixed prior to next release.
    type: cleanupAn internal cleanup or hygiene concern.
    api: runIssues related to the Cloud Run API.
    on Oct 1, 2020
  2. self-assigned this
    on Oct 1, 2020
  3. fhinkel commented on Oct 8, 2020

    @fhinkel
    Contributor

    @grayside Thanks! Do you want to keep this as P1?

  4. grayside commented on Oct 8, 2020

    @grayside
    CollaboratorAuthor

    Has Cloud Run flakiness been blocking PRs? If not this can be a P2.

  5. added
    priority: p2Moderately-important priority. Fix may not be included in next release.
    and removed
    priority: p1Important issue which blocks shipping the next release. Will be fixed prior to next release.
    on Nov 3, 2020
  6. removed their assignment
    on Nov 3, 2020
  7. averikitsch commented on Nov 9, 2020

    @averikitsch
    Contributor

    List of samples to update:

    • hello-broken
    • helloworld
    • idp-sql
    • image-processing
    • logging-manual
    • markdown-preview
      • editor
      • renderer
    • pubsub
    • system-package
  8. averikitsch commented on Dec 8, 2020

    @averikitsch
    Contributor

    @grayside for the PubSub sample tests, can I mock the Pub/Sub message by sending a POST request or should I test the user journey by setting up the Pub/Sub topic and sub and checking the logs for the message? The user journey tests seems like it could be flaky due to the need to check the logs.

  9. averikitsch commented on Dec 14, 2020

    @averikitsch
    Contributor

    All samples have been updated.

  10. averikitsch commented on Dec 31, 2020

    @averikitsch
    Contributor

    reopening as there are test failures

  11. averikitsch commented on Jan 6, 2021

    @averikitsch
    Contributor

    The last set of test errors were due to Cloud Build quotas

  12. grayside commented on Jan 8, 2021

    @grayside
    CollaboratorAuthor

    In discussion with Averi, #2124 doesn't fix Cloud Build quotas, so we are seeing two different classes of flakiness. Cloud Build Quotas will be handled in a follow-up.

  13. averikitsch commented on Jan 14, 2021

    @averikitsch
    Contributor

    Note: There has not been any test failures for the Run samples since the mocha timeouts extension PR was merged. Cloud build quotas have not been an issue, but could be if more Run samples are added or if multiple PRs are testing at the same time.

  14. averikitsch commented on Feb 4, 2021

    @averikitsch
    Contributor

    Cloud Build flakiness has not been an issue since the record error in December. Closing this issue and will open a new one if tests continue to be flaky.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

api: runIssues related to the Cloud Run API.priority: p2Moderately-important priority. Fix may not be included in next release.samplesIssues that are directly related to samples.type: cleanupAn internal cleanup or hygiene concern.

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions