Skip to content

functions/billing: test failure preventing PR merges #1985 #1988

New issue
New issue
Closed
#1990
Closed
functions/billing: test failure preventing PR merges #1985#1988
#1990
Assignees
ace-n
Labels
🚨This issue needs some love.api: cloudfunctionsIssues related to the Cloud Run functions API.lang: nodejsIssues specific to JavaScript or TypeScript.priority: p1Important issue which blocks shipping the next release. Will be fixed prior to next release.samplesIssues that are directly related to samples.type: bugError or flaw in code with unintended results or allowing sub-optimal usage patterns.

Description

@engelke
engelke
opened on Sep 28, 2020

This test appears to have been failing for at least six months. Latest failure log has this:

1) "before all" hook in "{root}":
 Error: Invalid Credentials
  at Gaxios._request (node_modules/gaxios/build/src/gaxios.js:89:23)
  at process._tickCallback (internal/process/next_tick.js:68:7)

Activity

  1. engelke
    added
    type: bugError or flaw in code with unintended results or allowing sub-optimal usage patterns.
    priority: p1Important issue which blocks shipping the next release. Will be fixed prior to next release.
    samplesIssues that are directly related to samples.
    lang: nodejsIssues specific to JavaScript or TypeScript.
    on Sep 28, 2020
  2. product-auto-label
    added
    api: cloudfunctionsIssues related to the Cloud Run functions API.
    on Sep 28, 2020

  3. grant commented on Sep 30, 2020

    @grant
    grant
    on Sep 30, 2020
    Contributor

    I have observed this issue too.

    I'd suggest disabling the test until it's more reliable.

    CC @ace-n

  4. yoshi-automation
    added
    🚨This issue needs some love.
    and removed
    🚨This issue needs some love.
    on Oct 5, 2020
  5. JustinBeckwith
    assigned
    ace-n
    on Oct 6, 2020

  6. ace-n commented on Oct 6, 2020

    @ace-n
    ace-n
    on Oct 6, 2020
    Contributor

    Test is no longer required, but #1990 should fix this.

  7. gcf-merge-on-green
    closed this as completedin #1990on Oct 7, 2020
  8. gcf-merge-on-green
    added a commit that references this issue on Oct 7, 2020

    (Maybe) Fix broken billing sample (#1990)

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

Metadata

Metadata

Assignees

Labels

🚨This issue needs some love.api: cloudfunctionsIssues related to the Cloud Run functions API.lang: nodejsIssues specific to JavaScript or TypeScript.priority: p1Important issue which blocks shipping the next release. Will be fixed prior to next release.samplesIssues that are directly related to samples.type: bugError or flaw in code with unintended results or allowing sub-optimal usage patterns.

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions