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

[JJBB] fix credentials with a service account #20535

Merged
merged 1 commit into from
Aug 11, 2020

Conversation

v1v
Copy link
Member

@v1v v1v commented Aug 11, 2020

What does this PR do?

Revert some changes partially of #19852 to fix some misconfiguration with the GH accounts in the golang-crossbuild project

Why is it important?

To fix

No changes detected: master (still at 7fe9c7c0bcde40f473473271f8b807557d74c683)

  1 branches were processed

  Checking pull-requests...

    Checking pull request elastic/beats#65

  Error while processing pull request 65

  Reason: org.kohsuke.github.GHFileNotFoundException: https://api.github.com/repos/elastic/golang-crossbuild/collaborators/kuisathaverat/permission {"message":"Resource not accessible by integration","documentation_url":"https://docs.github.com/rest/reference/repos#get-repository-permissions-for-a-user"}

    Checking pull request elastic/beats#62

  Error while processing pull request 62

  Reason: org.kohsuke.github.GHFileNotFoundException: https://api.github.com/repos/elastic/golang-crossbuild/collaborators/kuisathaverat/permission {"message":"Resource not accessible by integration","documentation_url":"https://docs.github.com/rest/reference/repos#get-repository-permissions-for-a-user"}

    Checking pull request elastic/beats#43

  Error while processing pull request 43

  Reason: org.kohsuke.github.GHFileNotFoundException: https://api.github.com/repos/elastic/golang-crossbuild/collaborators/null/permission {"message":"Resource not accessible by integration","documentation_url":"https://docs.github.com/rest/reference/repos#get-repository-permissions-for-a-user"}

  3 pull requests were processed

  3 pull requests encountered errors and were orphaned.

  Checking tags...

  Getting remote tags...

  0 tags were processed

Finished examining elastic/golang-crossbuild

[Tue Aug 11 06:56:46 UTC 2020] Finished branch indexing. Indexing took 3.3 sec
Finished: SUCCESS

Further details

I did add the elasticmachine and apmmachine as maintainers of the project with the same outcome. I don't know if the issue could related to some cache inconsistency in the CI side, some plugin misbehaving, or something internal with GH. To workaround this particular issue I've decided to revert partially the changes I did to use the GH app credentials.

@v1v v1v added automation ci Team:Automation Label for the Observability productivity team labels Aug 11, 2020
@v1v v1v requested a review from a team August 11, 2020 07:04
@v1v v1v self-assigned this Aug 11, 2020
@botelastic botelastic bot added needs_team Indicates that the issue/PR needs a Team:* label and removed needs_team Indicates that the issue/PR needs a Team:* label labels Aug 11, 2020
@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: [Pull request #20535 opened]

  • Start Time: 2020-08-11T07:05:39.150+0000

  • Duration: 22 min 16 sec

@v1v v1v merged commit a58ec5a into elastic:master Aug 11, 2020
@v1v v1v deleted the fix/use-credentials-rather-than-apps branch August 11, 2020 07:40
v1v added a commit to v1v/beats that referenced this pull request Aug 11, 2020
v1v added a commit to v1v/beats that referenced this pull request Aug 12, 2020
…ne-2.0

* upstream/master: (39 commits)
  [ITs] Revert healthcheck for elasticsearchssl service to the previous behaviour (elastic#20558)
  [Heartbeat] Fix packaging (elastic#20566)
  [Heartbeat] Add Magefile to X-Pack (elastic#20549)
  [Packetbeat] Add "network" to event.category (elastic#20392)
  fix typo in docs (elastic#20541)
  Add service resource in k8s cluster role (elastic#20546)
  Update Golang version to 1.14.7 (elastic#20508)
  Add missing inputs to filebeat spec (elastic#20388)
  add warning log in aws and googlecloud module for API cost (elastic#20523)
  Fix fortinet.firewall.mem value to be interpreted as integer (elastic#19335)
  [CI] add more resilience (elastic#20505)
  [JJBB] fix credentials with a service account for golang-crossbuild (elastic#20537)
  [ITs] change healthcheck for elasticsearch (elastic#20514)
  [JJBB] fix credentials with a service account (elastic#20535)
  chore(ci): use build step for checking if is PR (elastic#20536)
  [CI] runbld project name (elastic#20466)
  Add panw.panos.endreason field (elastic#18705)
  [Filebeat] Fix PANW field spelling "veredict" to "verdict" (elastic#18808)
  Fix typo in netflow module docs (elastic#18992)
  Modified auditd ingest pipeline to handle node=hostname (elastic#19659)
  ...
melchiormoulin pushed a commit to melchiormoulin/beats that referenced this pull request Oct 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automation ci Team:Automation Label for the Observability productivity team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants