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

kevin: migration for new build host #1268

Merged
merged 4 commits into from
May 17, 2020
Merged

Conversation

TheJJ
Copy link
Member

@TheJJ TheJJ commented May 8, 2020

Kevin builds are migrated from within a KVM VM to a podman container within a KVM VM.

@TheJJ TheJJ added the ci Related to our continuous integration systems (kevin, github-actions, ...) label May 8, 2020
@TheJJ TheJJ force-pushed the kevin-migration branch 20 times, most recently from fe304b1 to 6133467 Compare May 9, 2020 18:15
@TheJJ TheJJ added the kevin-rebuild-pl0x instruct kevin to rebuild this pull request label May 9, 2020
@SFTbot SFTbot removed the kevin-rebuild-pl0x instruct kevin to rebuild this pull request label May 9, 2020
kevinfile Show resolved Hide resolved
@simonsan
Copy link
Contributor

2020-05-10 05_57_46- WIP  Buildsystem update and documentation by simonsan · Pull Request #1266 · SF-Screenshot

I think Kevin is doing something wrong there It seems it's not updating the remote state and links somehow? If I click on the link to the check in the downwards panel I come to an old check. I need to click on the result of the commit and go from there.

@TheJJ
Copy link
Member Author

TheJJ commented May 10, 2020

API-wise it's the same, kevin updates the status for a commit hash. I'd try reloading the page, maybe that helps?

@simonsan
Copy link
Contributor

simonsan commented May 10, 2020

API-wise it's the same, kevin updates the status for a commit hash. I'd try reloading the page, maybe that helps?

No :-( but that's what it was like before. Reloading the page was updating the checks, when it was hanging. Now not anymore. :-(

EDIT: To be more precise, it's not even showing the check of the last commit but it's stuck at a check, probably waiting for me to manually ask kevin to check? You can see it in #1266, it's not updating the state in the boxes after a push.

@TheJJ TheJJ force-pushed the kevin-migration branch from 1d1a089 to 454a085 Compare May 17, 2020 15:38
@TheJJ TheJJ mentioned this pull request May 17, 2020
@TheJJ TheJJ merged commit 50e411a into SFTtech:master May 17, 2020
@TheJJ
Copy link
Member Author

TheJJ commented May 17, 2020

Should be good for now, I've removed the werror and we'll add it (just for the CI!) once I resolved the cython-c++-warnings.

@TheJJ TheJJ deleted the kevin-migration branch May 17, 2020 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Related to our continuous integration systems (kevin, github-actions, ...)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants