Giter VIP home page Giter VIP logo

Comments (16)

fejta avatar fejta commented on August 18, 2024 1

Still planning to work on this

from oss-test-infra.

fejta avatar fejta commented on August 18, 2024 1

Stop-gap is to make GH reporter not report anything with a client.GerritReportLabel

https://github.com/kubernetes/test-infra/blob/62f82f4febdca9ef41e1d5649ba1d83b730c696c/prow/gerrit/adapter/adapter.go#L279

from oss-test-infra.

krzyzacy avatar krzyzacy commented on August 18, 2024 1

https://kunit-review.googlesource.com/c/linux/+/2649 - the spam might be related, maybe not?

from oss-test-infra.

fejta avatar fejta commented on August 18, 2024

/assign @fejta @krzyzacy

from oss-test-infra.

krzyzacy avatar krzyzacy commented on August 18, 2024

oh I know this one...

from oss-test-infra.

krzyzacy avatar krzyzacy commented on August 18, 2024

I cannot find the original issue though...

The problem is github reporter was trying to report a gerrit job as there's no way to tell if a prowjob is from github or gerrit. (and this prow instance talks to both github and gerrit)

from oss-test-infra.

fejta avatar fejta commented on August 18, 2024

That is a bug though, right?

from oss-test-infra.

BenTheElder avatar BenTheElder commented on August 18, 2024

what's the status on this?

from oss-test-infra.

fejta avatar fejta commented on August 18, 2024

Still happening

{
 insertId: "ftk6hug4urda6x"  
 jsonPayload: {
  component: "crier"   
  error: "error setting status: status code 404 not one of [201], body: {"message":"Not Found","documentation_url":"https://developer.github.com/v3/repos/statuses/#create-a-status"}"   
  file: "prow/crier/controller.go:279"   
  func: "k8s.io/test-infra/prow/crier.(*Controller).processNextItem"   
  jobName: "e7b6bb67-d005-11e9-84a3-c68839286cc4"   
  jobStatus: {
   build_id: "1169669630457810944"    
   completionTime: "2019-09-05T18:00:55Z"    
   description: "Job succeeded."    
   pod_name: "e7b6bb67-d005-11e9-84a3-c68839286cc4"    
   prev_report_states: {
    gerrit-reporter: "success"     
   }
   startTime: "2019-09-05T17:52:19Z"    
   state: "success"    
   url: "https://prow.gflocks.com/view/gcs/oss-prow/pr-logs/pull/kunit-review.googlesource.com_linux/2449/kunit-test-presubmit/1169669630457810944"    
  }
  level: "error"   
  msg: "failed to report job"   
  prowjob: "default/e7b6bb67-d005-11e9-84a3-c68839286cc4"   
 }
 labels: {…}  
 logName: "projects/oss-prow/logs/crier"  
 receiveTimestamp: "2019-10-11T20:33:26.109823851Z"  
 resource: {
  labels: {
   cluster_name: "prow"    
   container_name: "crier"    
   instance_id: "6234379657828418500"    
   namespace_id: "default"    
   pod_id: "crier-7bf477fddd-jfkgn"    
   project_id: "oss-prow"    
   zone: "us-west1-a"    
  }
  type: "container"   
 }
 severity: "ERROR"  
 timestamp: "2019-10-11T20:33:20Z"  
}

from oss-test-infra.

fejta avatar fejta commented on August 18, 2024

Still happening

from oss-test-infra.

fejta avatar fejta commented on August 18, 2024

I'm planning to look into this soon/next.

from oss-test-infra.

fejta avatar fejta commented on August 18, 2024

/area prow
/area prow/crier

from oss-test-infra.

google-oss-robot avatar google-oss-robot commented on August 18, 2024

@fejta: The label(s) area/prow, area/prow/crier cannot be applied. These labels are supported: ``

In response to this:

/area prow
/area prow/crier

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

from oss-test-infra.

michelle192837 avatar michelle192837 commented on August 18, 2024

Any updates? (And/or ETA?)

from oss-test-infra.

fejta avatar fejta commented on August 18, 2024

I suspect we kept creating and never reporting some prow jobs. And now we finally reported them.

from oss-test-infra.

krzyzacy avatar krzyzacy commented on August 18, 2024

another suspicious point: https://prow.gflocks.com/?type=presubmit&job=kunit-test-presubmit - we had ~2mo old prowjobs which disagrees with the age we configured: https://github.com/GoogleCloudPlatform/oss-test-infra/blob/master/prow/config.yaml#L59

from oss-test-infra.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.