Skip to content
This repository has been archived by the owner on Sep 19, 2024. It is now read-only.

Clear Logging Error for broken ubiquibot-config.yml #680

Closed
2 tasks
0x4007 opened this issue Aug 27, 2023 · 15 comments · Fixed by #684
Closed
2 tasks

Clear Logging Error for broken ubiquibot-config.yml #680

0x4007 opened this issue Aug 27, 2023 · 15 comments · Fixed by #684

Comments

@0x4007
Copy link
Member

0x4007 commented Aug 27, 2023

The bot does not show any visible errors for when it has an issue loading in a new config. The bot was down for about 14 hours on this repository until we identified the fix. Apparently it was the broken config.

There were no logs to verify this, we just changed it because the bot was working on other repositories.

Context: https://t.me/c/1588400061/3250/4436

The business problem to solve is that we need to know as soon as possible if the config is valid.

  • Clearly log error while the bot tries to load the config in the Supabase.
  • We should consider some type of special event that when the config is updated, the bot immediately validates it and lets the user know via a comment directly on the commit. Similar to how we do deploy comments on: ubiquity/ubiquity-dollar@8fa34df#commitcomment-125340301
@0x4007
Copy link
Member Author

0x4007 commented Aug 27, 2023

We should consider some type of special event that when the config is updated, the bot immediately validates it and lets the user know via a comment directly on the commit.

@whilefoo you expressed interest in handling this issue. Can you provide input on the second subtask?

@whilefoo
Copy link
Collaborator

The second subtask is doable. We already handle push events on the bot, so we just need to retrieve the content of the config at specific commit hash, parse it and validate it and if it fails it will post a commit comment. This will also work for partners when they change the config.

@ubiquibot
Copy link

ubiquibot bot commented Aug 27, 2023

@whilefoo The time limit for this bounty is on Sun, 27 Aug 2023 14:03:53 UTC

@ubiquibot
Copy link

ubiquibot bot commented Sep 6, 2023

Do you have any updates @whilefoo? If you would like to release the bounty back to the DevPool, please comment /stop
Last activity time: Sat Sep 02 2023 18:15:08 GMT+0000 (Coordinated Universal Time)

@ubiquibot
Copy link

ubiquibot bot commented Sep 12, 2023

Permit generation skipped because bounty amount after penalty is 0

If you've enjoyed your experience in the DevPool, we'd appreciate your support. Follow Ubiquity on GitHub and star this repo. Your endorsement means the world to us and helps us grow!
We are excited to announce that the DevPool and UbiquiBot are now available to partners! Our ideal collaborators are globally distributed crypto-native organizations, who actively work on open source on GitHub, and excel in research & development. If you can introduce us to the repository maintainers in these types of companies, we have a special bonus in store for you!

@ubiquibot
Copy link

ubiquibot bot commented Sep 12, 2023

Task Creator Reward

pavlovcik: [ CLAIM 73.6 WXDAI ]

@ubiquibot
Copy link

ubiquibot bot commented Sep 12, 2023

@whilefoo
Copy link
Collaborator

Permit generation skipped because bounty amount after penalty is 0

If you've enjoyed your experience in the DevPool, we'd appreciate your support. Follow Ubiquity on GitHub and star this repo. Your endorsement means the world to us and helps us grow!
We are excited to announce that the DevPool and UbiquiBot are now available to partners! Our ideal collaborators are globally distributed crypto-native organizations, who actively work on open source on GitHub, and excel in research & development. If you can introduce us to the repository maintainers in these types of companies, we have a special bonus in store for you!

I don't think I have any penalties?

@0x4007
Copy link
Member Author

0x4007 commented Sep 14, 2023

Permit generation skipped because bounty amount after penalty is 0

If you've enjoyed your experience in the DevPool, we'd appreciate your support. Follow Ubiquity on GitHub and star this repo. Your endorsement means the world to us and helps us grow!
We are excited to announce that the DevPool and UbiquiBot are now available to partners! Our ideal collaborators are globally distributed crypto-native organizations, who actively work on open source on GitHub, and excel in research & development. If you can introduce us to the repository maintainers in these types of companies, we have a special bonus in store for you!

I don't think I have any penalties?

I guess it would make sense to link back to the re-opened issue that you were penalized for. I'm glad that I saw this because this is the first time I've seen a penalty work. I'm checking the database to get more info here.

I currently only see one row for $150 in ubiquity/ubiquibot repo. No further details sadly.

@ubiquibot
Copy link

ubiquibot bot commented Sep 14, 2023

@whilefoo please be sure to review this conversation and implement any necessary fixes. Unless this is closed as completed, its payment of 73.6 WXDAI will be deducted from your next bounty.

@0x4007
Copy link
Member Author

0x4007 commented Sep 14, 2023

/query @whilefoo

@ubiquibot
Copy link

ubiquibot bot commented Sep 14, 2023

@whilefoo's wallet address is 0xA0B11F474d8ECE1205d38c66d5F2bE8917675d60 and multiplier is 1

@0x4007
Copy link
Member Author

0x4007 commented Sep 14, 2023

Permit generation skipped because bounty amount after penalty is 0

If you've enjoyed your experience in the DevPool, we'd appreciate your support. Follow Ubiquity on GitHub and star this repo. Your endorsement means the world to us and helps us grow!
We are excited to announce that the DevPool and UbiquiBot are now available to partners! Our ideal collaborators are globally distributed crypto-native organizations, who actively work on open source on GitHub, and excel in research & development. If you can introduce us to the repository maintainers in these types of companies, we have a special bonus in store for you!

https://gnosisscan.io/tx/0x87d8703ab5d690bb549f6813f84bbea3b5966d27b8b7150ebcc269673735b979

@whilefoo
Copy link
Collaborator

I guess it would make sense to link back to the re-opened issue that you were penalized for. I'm glad that I saw this because this is the first time I've seen a penalty work. I'm checking the database to get more info here.

I'd guess that is hard to do in current state because penalties seem to add up from different issues and get deducted when new issues are closed so it acts like a debt balance and we can't accurately link back a PR unless we redesign the system

@0x4007
Copy link
Member Author

0x4007 commented Sep 14, 2023

unless we redesign the system

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants