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

Builds are failing #27

Closed
javagl opened this issue Sep 11, 2023 · 4 comments
Closed

Builds are failing #27

javagl opened this issue Sep 11, 2023 · 4 comments

Comments

@javagl
Copy link
Contributor

javagl commented Sep 11, 2023

As mentioned in #24 (comment) (and as it can be seen in the PR list), the builds are currently failing when a new model is added. I zoomed into that, a little bit, but ... am not a GitHub actions expert. So with the usual disclaimers (I have no idea what I'm doing here. LOL), a few comments:


Starting with an aside:

The workflows are run twice in the PR that I linked to above. This is probably related to the combination of

  • the two events that trigger them, namely push and pull_request
  • @echadwick-artist working directly on the KhronosGroup repo, and not on a fork

For example, in #26 , I'm creating a PR from a fork, and there, the workflows are only run once. Maybe there's a tricky way to avoid running them twice when the PR comes from KhronosGroup, but ... it shouldn't be so much an issue. (In fact, there sometimes seem to be difficulties when trying to run such actions based on PRs from a fork, but ... maybe we can make it work both ways).


The actual failing build:

  • It does work when it is triggered by the push event
  • It does not work when it is triggered by the pull_request event

Looking at the logs:

And it cannot commit with a deached head.

Now... why is the head detached here? I don't know. I haven't looked at the implementation of the actions/checkout@v2 action. From quickly skimming the documentation, it talks about a ref property that may determine what exactly is checked out, depending on what triggered the action. This might be related (or similar) to actions/checkout#455.


I could reproduce the issue in my "Sandbox" repo (so apparently, ~"something in GitHub" had changed and caused this issue, because it worked smoothly a while ago!). And I played a bit with some some comments from the linked issue, and this change seems to resolve the problem:

javagl/sandbox@297d322

Maybe we want to try this out here as well.

@javagl
Copy link
Contributor Author

javagl commented Sep 11, 2023

I opened a PR with an attempt to fix this at #28 , maybe someone wants to have a look.

@javagl
Copy link
Contributor Author

javagl commented Oct 15, 2023

I'll just claim that it is not possible to use GitHub actions for the tasks that are required for this repository.

(It's not that I think that it is not possible. It was possible until a few weeks ago, and then someone screwed things up. I just claim that it is not possible)

We should remove the failing GitHub action, and someone should manually update the model index periodically.

@DRx3D
Copy link
Contributor

DRx3D commented Nov 17, 2023

The action has been changed to just perform the necessary consistency checks and report the results. The build of the repo indices is still a manual operation. Leaving this issue open until the build of the repo can be addressed.

@javagl
Copy link
Contributor Author

javagl commented Nov 20, 2023

Replaced by #54

@javagl javagl closed this as completed Nov 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants