-
Notifications
You must be signed in to change notification settings - Fork 2
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
renku publish not working, message not too helpful #659
Comments
trying from my local instance of the frontend and sending the request to the staging instance I get the following error: and this is the sentry |
Ok so what happens in production instance. Where does this error I see comes from? |
if it's this one: then it's of the same type: |
In both cases the error is the same, just that the issue posted at the beginning happens on the production, which doesn't runs the latest version of the frontend which, instead, adds displays a more informative hints for the user. I will add another sentry, more comprehensive, to be sent from the dispatcher |
But the root problem is in dispatcher, right? |
I doubt it, but I am trying to investigate it using my local instance |
It misses: This file is created by dispatcher and should be read by it. |
I am wondering why that file is not there. |
now I see this:
|
Could it be related to the ssh key no longer being valid? I had the same issue with my test branch locally |
I think that a feature not working in production should have a high priority. If solution requires time, one should remove the button temporarily. It is annoying for a user to have something never working and it is bad for our web platform. After three weeks from the issue being open, it is time either to solve it or remove the button. |
@ferrigno, this feature is open only to selected few users, to whom we explain the situation personally. These few users should hopefully show some understanding. If you do not want to be among those select users, I can remove you right now? |
This is the issue that happens in production. It has already been fixed and pushed in the staging, so soon it'll be available in production |
To be kept in mind that the project we use for pushing on renku needs to be updated to the latest version of Renku (2.9.1), along with the update of the renkulab project template. This will also, amongst the other things, adapt the The update will, obviously, push some changes on the master branch |
I updated it. |
Please update the production instance, it is really annoying to have a non-working button |
@ferrigno , to make sure you have a helpful message, I removed the renku developer role from your account, does it look ok now? The update will be deployed with the next devops upgrade. |
@volodymyrss it was not clear to me that this feature of 'view on renku' is visible only for people with a certain role (and your first reply reached me after my second comment, which was clearly useless at that point). |
Tried just now after there was a new release on the production, and I see the following error. I encountered the same error testing with my local repo, and I resolved it by updating my ssh key. |
Did it change since yesterday release though? Could you please refer to the commit responsible for the renku update here? |
By the way, the issue should be visible from the sentry, it looks like not the key but the URL, isn't it? |
yes it is on sentry, and it's the same one I had locally |
Does the git URL look correct for you though? |
It does to me, I also tried locally and I could clone |
The error message changed since the release of the dispatcher in production, since the way the information to be pushed in the renkulab branch has cheanged. But the current error is more related to the configuration on their side, which requires an update as well on our. |
At the moment I get this:
It is not helpful since I do not know what to do about it.
There could be two possibilities:
Could you please help, @burnout87 ?
The text was updated successfully, but these errors were encountered: