Comments (3)
What do you think we should do in this situation? Blindly overwrite the hook id stored in the database with the new one?
@nolman what do you think about this?
from kochiku.
Ya we could update GithubPostReceiveHook#subscribe! to always use the #synchronize_or_create! strategy and overwrite. The downside of doing this as the @interested_events change we will leave behind hooks. So we would need to clean up those leftover hooks.
We could also update RepositoryObserver#after_save to handle errors and save the error message when installing a hook fails.
@jdavisp3 would you be able to provide more context on how the GH instance and kochiku got into this state? As well as what hooks currently exist (as well as your kochiku hostname)? Just trying to understand the what happened from a end users perspective.
from kochiku.
I'm not exactly sure how I got into this state, but it happened a few times.
I think one time it happened I moved the repo URL to point to a different
repo -- I guess the hook it was expecting wasn't there maybe?
The other time I think I was debugging a hook problem on my (networking)
and I was editing the hooks by hand to find one that worked, so I probably
deleted the original one?
from kochiku.
Related Issues (20)
- Buttons on build page do not render correctly in Firefox HOT 1
- Should send an email after first build part failure HOT 1
- Create a official Docker in https://registry.hub.docker.com/ HOT 1
- log_file_globs setting inside of targets does not work
- Automerge and success email should be combined.
- Kochiku does not display an error if rebuild is clicked after the commit has been deleted
- Drop the deprecated on_success_script column
- Build failure email does not include part name. HOT 1
- "Create Pull Request" assumes GitHub
- First-try completion percentage incorrectly counts incomplete first-attempts against metric. HOT 1
- Wiki incorrect file name for github oauth secret HOT 1
- TEST_RUNNER env var doesn't quote value
- Error with git submodules ? HOT 5
- Merge on Success uses Kochiku Robot as the author HOT 1
- [Feature Request] Allow setting an inactivity duration after which kochiku considers jobs abandoned HOT 1
- GitHub authentication for private repos HOT 2
- Support reversing an abort action HOT 2
- Support Postgresql? HOT 2
- Doc missing background jobs setup steps / examples HOT 1
- Refresh checkbox does not actually work HOT 3
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
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.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from kochiku.