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

Add dummy test for self hosted GitHub runners #5521

Merged

Conversation

tomassebestik
Copy link
Member

This is simple test for newly installed self hosted RPI Github Runners in Espressif Brno site.
Before accepting this PR some selfhosted GH runner must be assigned into project.

Details in https://jira.espressif.com:8443/browse/RDT-43

@CLAassistant
Copy link

CLAassistant commented Aug 11, 2021

CLA assistant check
All committers have signed the CLA.


on:
push:
branches: [pull_request]
Copy link
Member

Choose a reason for hiding this comment

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

Shouldn't this be

on:
  push:
    branches:
    - master
  pull_request:

or

on:
  pull_request:

Copy link
Member Author

Choose a reason for hiding this comment

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

@me-no-dev Thanks! I've changed that as proposal(1).

Copy link
Member

Choose a reason for hiding this comment

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

Now the test ran :) https://github.com/espressif/arduino-esp32/pull/5521/checks?check_run_id=3310070479

Could you sign the CLA too please? GH Checks are complaining

Copy link
Member Author

Choose a reason for hiding this comment

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

@me-no-dev Done, thank you. PLS merge it.

Copy link

@hippyaki hippyaki left a comment

Choose a reason for hiding this comment

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

What good does it do? Can you help me understand?

@me-no-dev
Copy link
Member

What good does it do? Can you help me understand?

this is a test for future use.

@me-no-dev me-no-dev merged commit 6dfaf6c into espressif:master Sep 15, 2021
@jepler
Copy link
Contributor

jepler commented Dec 5, 2021

This causes CI in forks to fail, because they do not have such a runner.

jepler added a commit to adafruit/arduino-esp32 that referenced this pull request Dec 5, 2021
This was added upstream at espressif#5521
and does nothing practical.  However, it DOES cause CI to wait for a
long time for a matching self-hosted runner.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants