plowsof · June 16, 2025 14:11
diff --git a/log.txt b/log.txt
 at, Jun 14, 2025, 12:00:33 - plowsof: napoly you realise that this person leaving AI slop issues on your repo has a net negative on your project? https://github.com/orgs/btcpay-monero/discussions/31 
 Sat, Jun 14, 2025, 12:00:55 - plowsof: i noticed you gave it a heart 
 Sat, Jun 14, 2025, 12:02:26 - plowsof: now lets figure out how to make sure that wallet rpc is ready to receive/respond to messages (quite important) https://github.com/btcpay-monero/btcpayserver-monero-plugin/issues/14
 Sat, Jun 14, 2025, 12:25:30 - ofrnxmr: <@plowsof:matrix.org "now lets figure out how to make ..."> Interesting 
 Sat, Jun 14, 2025, 12:26:03 - ofrnxmr: also, are the monero libs managed by btcpayserver? they are outdated
 Sat, Jun 14, 2025, 13:29:20 - plowsof: if the rpc wallet run time args for these tests are here - https://github.com/btcpay-monero/btcpayserver-monero-plugin/blob/e9ecf24f96ca2a5fbbf06c9c6225d84663ef070b/BTCPayServer.Plugins.IntegrationTests/docker-compose.yml#L105 
 Sat, Jun 14, 2025, 13:33:10 - plowsof: it appears to be lack of --no-inital-sync flag and programatically starting the wallet rpc / or at least polling until its responsive before beginning 
 Sat, Jun 14, 2025, 13:33:45 - deverickapollo: <@plowsof:matrix.org "napoly you realise that this per..."> I’m going to come at this differently. Regardless of that persons comment, as the ccs coordinator, I expect you to do better. If you ever slander a community member that is a potential donors, I will ban you from commenting on our repo. That was distasteful and not a position you should be taking. 
 Sat, Jun 14, 2025, 13:33:54 - deverickapollo: And if you got a problem. Come see me
 Sat, Jun 14, 2025, 13:34:48 - plowsof: point to the slander? Not AI slop which contained [ Your Name ] place holder?
 Sat, Jun 14, 2025, 13:35:09 - deverickapollo: I ain’t wasting my time. You got my point
 Sat, Jun 14, 2025, 13:36:14 - plowsof: " If you ever slander a community member that is a potential donors," again, how have i slandered? 
 Sat, Jun 14, 2025, 13:36:21 - deverickapollo: <@ofrnxmr:xmr.mx "also, are the monero libs manage..."> Not yet! Will be coming in soon
 Sat, Jun 14, 2025, 13:38:14 - deverickapollo: <@plowsof:matrix.org "if the rpc wallet run time args ..."> Much better focus
 Sat, Jun 14, 2025, 13:38:30 - ofrnxmr: <@deverickapollo:matrix.org "Not yet! Will be coming in soon"> I mean, who manages them?
 Sat, Jun 14, 2025, 13:38:58 - ofrnxmr: Where is the docker pulled from
 Sat, Jun 14, 2025, 13:39:09 - deverickapollo: BTCPay owns the repo - they manage the updates. It’s from their repo. One sec, can link you
 Sat, Jun 14, 2025, 13:40:34 - ofrnxmr: <@deverickapollo:matrix.org "I’m going to come at this differ..."> Ai generated praise is trash
 Sat, Jun 14, 2025, 13:40:55 - ofrnxmr: didnt even write his own name
 Sat, Jun 14, 2025, 13:41:16 - deverickapollo: https://github.com/btcpayserver/dockerfile-deps
 Sat, Jun 14, 2025, 13:41:21 - plowsof: [ Your name ] was input maunally later, so technically he did 
 Sat, Jun 14, 2025, 13:41:38 - plowsof: 😆
 Sat, Jun 14, 2025, 13:42:33 - ofrnxmr: <@deverickapollo:matrix.org "https://github.com/btcpayserver/..."> Ok 18.4.0 is available, and 18.4.1 coming in a week or so
 Sat, Jun 14, 2025, 13:45:17 - ofrnxmr: <@plowsof:matrix.org "it appears to be lack of --no-in..."> +1. Try adding the --no-initial-sync
 Sat, Jun 14, 2025, 13:45:54 - ofrnxmr: Once 9950 and 9935 (if merged to release branch) are merged upstream, it should be more reliable
 Sat, Jun 14, 2025, 13:47:08 - ofrnxmr: wallet-rpc pulls hashes from genesis, and takes a minute or 2 of unresponsiveness - fixed by 9935. 

 when syncing, wallet rpc will ignore any/all commands sent to it, including sigint etc. 9950 fixes this
 Sat, Jun 14, 2025, 13:47:53 - ofrnxmr: --no-initial-sync will start the rpc server before calling refresh, allowing you to send commands to it before it starts pulling hashes
 Sat, Jun 14, 2025, 13:48:50 - ofrnxmr: So it should be used if you require responses _at_ startup
 Sat, Jun 14, 2025, 13:54:42 - deverickapollo: napoly: should be able to bump version used on the integration test for now. 
 Sat, Jun 14, 2025, 13:58:05 - plowsof: where do i see you? im in the btcpay monero room right?
 Sat, Jun 14, 2025, 14:08:13 - deverickapollo: I’m in Lincoln Nebraska but happy to meet you anywhere 
 Sat, Jun 14, 2025, 14:08:49 - ofrnxmr: https://github.com/btcpayserver/dockerfile-deps/pull/114
 Sat, Jun 14, 2025, 14:09:20 - ofrnxmr: "cash me ousside how bow daht" vibes
 Sat, Jun 14, 2025, 14:09:45 - deverickapollo: I ain’t no keyboard warrior - come correct 
 Sat, Jun 14, 2025, 14:10:15 - deverickapollo: I show respect and work hard. No time for simple and def not kids looking for direction
 Sat, Jun 14, 2025, 14:17:23 - plowsof: you've just learned something vital about wallet-rpc, you're welcome. 
 Sat, Jun 14, 2025, 14:18:04 - plowsof: i don't know what "No time for simple and def not kids looking for direction" means in relation to this all neither 
 Sat, Jun 14, 2025, 14:18:53 - ofrnxmr: tbh, I don't know what it means at all.. so i just pretended like i didn't see it 🫠
 Sat, Jun 14, 2025, 14:19:04 - plowsof: if i needed to meet you in person to tell you about --no-initial-sync there would be a huge delay 
 Sat, Jun 14, 2025, 14:20:08 - ofrnxmr: who are the kids looking for direction? And what does "no time for simple" mean? All this and more, on the next exciting episode of dragon ball zeeeee
 Sat, Jun 14, 2025, 14:20:49 - ofrnxmr: Anyway, signing off
 [ your name ]
 Market capital investment group
 Sun, Jun 15, 2025, 09:04:59 - napoly: feels like u guys are overreacting.. i just gave dude an emoji and moved on.. wasn't really analyzing it any further.. did not receive the [your name] email since i'm not following btcpay-monero lol
 Sun, Jun 15, 2025, 09:06:30 - napoly: you will need to ping seth or someone from btcpay for review/approval.. fyi
 Sun, Jun 15, 2025, 09:21:15 - napoly: it’s understandable that you guys at CCS are triggered about AI.. but this dude here was just showing appreciation.. not trying to lock in or gain funds
 Sun, Jun 15, 2025, 09:50:52 - ofrnxmr: <@napoly:matrix.org "you will need to ping seth or so..."> Why would i need to ping seth? why can't you approve it?
 Sun, Jun 15, 2025, 10:15:12 - ofrnxmr: Also, i missed https://github.com/btcpayserver/dockerfile-deps/pull/113
 Sun, Jun 15, 2025, 10:15:23 - ofrnxmr: Seth already sent a pr
 Sun, Jun 15, 2025, 10:15:33 - napoly: i don't have rights there.. we scratched the surface of having them under btcpay-monero.. but the outcome for now is to leave them there.. as they are used to create a compose yml
 Sun, Jun 15, 2025, 10:19:06 - ofrnxmr: You don't need rights to approve (i just approved seths. Closing mine)
 Sun, Jun 15, 2025, 12:51:09 - plowsof: deverickapollo: have you had time to think about your behaviour yesterday? 
 Sun, Jun 15, 2025, 12:53:17 - ofrnxmr: <@ofrnxmr:xmr.mx "Also, i missed https://github.co..."> Or anyone have time to approve seth's 113?
 Sun, Jun 15, 2025, 12:54:30 - ofrnxmr: Or (i assume you guys are in contact) to reach out to ask them to merge it
 Sun, Jun 15, 2025, 13:31:02 - ofrnxmr: <@ofrnxmr:xmr.mx "Or anyone have time to approve s..."> Or maybe time 2 migrate it to btcpay-monero
 Sun, Jun 15, 2025, 13:38:22 - ofrnxmr: I seem to be using the the word "or" a lot today
 Sun, Jun 15, 2025, 13:56:33 - plowsof: napoly: so adding a health check to xmrwallet (looping until the rpc server is online/responsive) doesn't seem to fix the issue, --no-initial-sync can also be ignored as its starting with a wallet dir. so the docker-compose file doesnt actually open a wallet, and im wondering where this happens for the test?
 Sun, Jun 15, 2025, 13:58:07 - plowsof: alot of warnings in the logs giving me a headache too 
 Sun, Jun 15, 2025, 14:55:58 - saeedab joined the room
 Sun, Jun 15, 2025, 14:58:32 - plowsof: similar timeout error on this one https://github.com/btcpay-monero/btcpayserver-monero-plugin/actions/runs/15640176398/job/44065403739#step:9:2150 
 Sun, Jun 15, 2025, 15:02:27 - plowsof: is there simply no account label because there is no wallet created / opened / synced? 🤷
 Sun, Jun 15, 2025, 15:15:08 - napoly: would be cool if u could investigate this.. i did not dig into this direction tbh.. or at least creating a documentation for this process.. it's always like someone from the comm has to go to their repos to update it.. an emergency update could give us a headache to push through as seems even seth is waiting for a merge
 Sun, Jun 15, 2025, 15:16:51 - napoly: yes!.. if you try locally and your containers are running the test will go green afaik
 Sun, Jun 15, 2025, 15:22:48 - plowsof: you mentioned it running the 2nd time? or do you mean 2nd time after commenting out the 'wait for wallet label'? 
 Sun, Jun 15, 2025, 15:23:16 - plowsof: so the actual problem is something/somewhere is trying to load info from a wallet-rpc which the dockerfile started, that has --wallet-dir , and no open wallet?
 Sun, Jun 15, 2025, 15:30:22 - plowsof: napoly: pls clarify what you mean about it working the 2nd time, so i can go further 
 Sun, Jun 15, 2025, 15:32:21 - napoly: 2nd time means having those containers up and running already (heated up)
 Sun, Jun 15, 2025, 15:33:35 - plowsof: ok but with wallets opened, synced? - done outside of the docker-compose.yml which only starts the wallet-rpc? and thats definitely the reason why this isnt working? i ask because i know absolutely nothing currently 
 Sun, Jun 15, 2025, 15:34:41 - plowsof: we can add healthchecks (service_healthy) to pause/wait for things where necessary in this docker-compose, but the tests need rpc open _with a wallet open_? 
 Sun, Jun 15, 2025, 16:54:23 - napoly: i donno either.. will need to dig it tbh
 Sun, Jun 15, 2025, 19:14:30 - plowsof: napoly i got something to run finally, what a headache ehhhhh ill gather a PR for you
 Sun, Jun 15, 2025, 19:53:29 - plowsof: napoly would you like me to look into your code coverage report failing to upload artifacts? or why the lint errors without saying why?
 Sun, Jun 15, 2025, 19:55:13 - napoly: got link? u should create a pr..
 Sun, Jun 15, 2025, 19:56:43 - plowsof: PR here https://github.com/btcpay-monero/btcpayserver-monero-plugin/pull/33 
 Sun, Jun 15, 2025, 19:58:41 - plowsof: likely a space after .. will remove and re-run to allow for the lint to fail without saying (i hope)
 Sun, Jun 15, 2025, 20:00:04 - napoly: in readme: To enforce formatting for the whole project, run `dotnet format btcpay-monero-plugin.sln --exclude submodules/\* --verbosity diagnostic`
 Sun, Jun 15, 2025, 20:00:49 - plowsof: ah yes, i do remember glancing at a blog too for dev env?
 Sun, Jun 15, 2025, 20:01:43 - plowsof: ideally the lint would be verbose in the workflow , todo?
 Sun, Jun 15, 2025, 20:02:33 - napoly: it is: --verbosity diagnostic
 Sun, Jun 15, 2025, 20:02:33 - plowsof: this one https://btcpay.host/developer/2025/06/09/btcpay-monero-developer-setup.html
 Sun, Jun 15, 2025, 20:02:56 - napoly: yeah.. copy of readme
 Sun, Jun 15, 2025, 20:03:35 - napoly: C:\a\btcpayserver-monero-plugin\btcpayserver-monero-plugin\BTCPayServer.Plugins.IntegrationTests\Monero\MoneroPluginUITest.cs(18,76): error WHITESPACE: Fix whitespace formatting. Replace 11 characters with '\r\n\s\s\s\s\s\s\s\s'. [C:\a\btcpayserver-monero-plugin\btcpayserver-monero-plugin\BTCPayServer.Plugins.IntegrationTests\BTCPayServer.Plugins.IntegrationTests.csproj]
 Sun, Jun 15, 2025, 20:03:38 - plowsof: am i missing something @ https://github.com/plowsof/btcpayserver-monero-plugin/actions/runs/15666251510/job/44130728201#step:5:1
 Sun, Jun 15, 2025, 20:04:55 - napoly: tabs.. instead of spaces ? smthng like that
 Sun, Jun 15, 2025, 20:05:18 - plowsof: thanks, im blind, will fix
 Sun, Jun 15, 2025, 20:08:52 - plowsof: ok good, it was the expected space, lint working fine
 Sun, Jun 15, 2025, 20:12:48 - plowsof: do the workflows randomly fail? xD
 Sun, Jun 15, 2025, 20:13:17 - napoly: yeah.. need to stabilize them a bit
 Sun, Jun 15, 2025, 20:13:43 - plowsof: ok. i was able to get a successful run with this PR earlier 
 Sun, Jun 15, 2025, 20:13:56 - napoly: re-running it
 Sun, Jun 15, 2025, 20:14:22 - plowsof: i dont trust re-running on github, never works for me lol
 Sun, Jun 15, 2025, 20:15:06 - plowsof: ok sorry for spamming everyone
 Sun, Jun 15, 2025, 20:16:20 - napoly: even if green.. it will take me some time to look through it fyi.. as bitcoind doesn't need this health check
 Sun, Jun 15, 2025, 20:18:17 - plowsof: health checks, to poll get_version which tests will 'depends_on'
 https://medium.com/@cbaah123/docker-compose-health-checks-made-easy-a-practical-guide-3a340571b88e
 service_healthy is defined by the healthcheck
 https://docs.docker.com/compose/how-tos/startup-order/
 Sun, Jun 15, 2025, 20:19:07 - plowsof: had to add condition: to all because you cant mix - this with this: 🤷
 Sun, Jun 15, 2025, 20:19:39 - plowsof: essentially it loops get_version curl , and waits for it to return 
 Sun, Jun 15, 2025, 20:19:49 - napoly: do we know why bitcoind doesn't need this?
 Sun, Jun 15, 2025, 20:20:23 - plowsof: from my exp bitcoind starts its own http server (we use wallet rpc binary that kinda sucks sometimes?)
 Sun, Jun 15, 2025, 20:20:59 - plowsof: ah nvm, bitcoind (the daemon) not the 'wallet-rpc', i do not know 
 Sun, Jun 15, 2025, 20:21:41 - napoly: ok.. thnx.. will need to find time to look at it
 Sun, Jun 15, 2025, 20:21:50 - plowsof: i made a typo , thanks ofrnxmr 
 Sun, Jun 15, 2025, 20:22:06 - plowsof: create xmr_wallet.. open xnr_wallet 🧠
 Sun, Jun 15, 2025, 20:23:00 - plowsof: actually napoly, this probably doesnt need --no-initial-sync because we start rpc without opening a wallet (and syncing) 
 Sun, Jun 15, 2025, 20:23:17 - plowsof: and the monerod its connecting to is --offline, so will never sync 
 Sun, Jun 15, 2025, 20:24:12 - napoly: ya clean it pls up from unnecessary stuff.. would be definitely better and easier to check later on
 Mon, Jun 16, 2025, 07:16:54 - ajs_ joined the room
 Mon, Jun 16, 2025, 12:20:05 - plowsof: deverickapollo: have you had a chance to think about your behaviour yet? not sure if you caught the last time i sent this question. 
 Mon, Jun 16, 2025, 12:30:00 - deverickapollo: I said my part and left it. Let’s keep convo focused. You did good yesterday. Keep it up
 Mon, Jun 16, 2025, 12:30:58 - deverickapollo: https://btcpay.host/ccs.html
 Mon, Jun 16, 2025, 12:38:13 - plowsof: there is a problem with the repo for linux users, it will need .gitattributes fixes but i dont have a windows machine to go back and forth, for now linux contributors must ignore 2 files , the method ive found working is `git update-index --assume-unchanged` <file> , currently this is img/Checkout.png and another .sln file cc ofrnxmr
 Mon, Jun 16, 2025, 12:39:10 - plowsof: so is my interpretation of your words an attempt at physical intimidation? a perceived physical threat to a staff member is quite important to clarify
	at, Jun 14, 2025, 12:00:33 - plowsof: napoly you realise that this person leaving AI slop issues on your repo has a net negative on your project? https://github.com/orgs/btcpay-monero/discussions/31
	Sat, Jun 14, 2025, 12:00:55 - plowsof: i noticed you gave it a heart
	Sat, Jun 14, 2025, 12:02:26 - plowsof: now lets figure out how to make sure that wallet rpc is ready to receive/respond to messages (quite important) https://github.com/btcpay-monero/btcpayserver-monero-plugin/issues/14
	Sat, Jun 14, 2025, 12:25:30 - ofrnxmr: <@plowsof:matrix.org "now lets figure out how to make ..."> Interesting
	Sat, Jun 14, 2025, 12:26:03 - ofrnxmr: also, are the monero libs managed by btcpayserver? they are outdated
	Sat, Jun 14, 2025, 13:29:20 - plowsof: if the rpc wallet run time args for these tests are here - https://github.com/btcpay-monero/btcpayserver-monero-plugin/blob/e9ecf24f96ca2a5fbbf06c9c6225d84663ef070b/BTCPayServer.Plugins.IntegrationTests/docker-compose.yml#L105
	Sat, Jun 14, 2025, 13:33:10 - plowsof: it appears to be lack of --no-inital-sync flag and programatically starting the wallet rpc / or at least polling until its responsive before beginning
	Sat, Jun 14, 2025, 13:33:45 - deverickapollo: <@plowsof:matrix.org "napoly you realise that this per..."> I’m going to come at this differently. Regardless of that persons comment, as the ccs coordinator, I expect you to do better. If you ever slander a community member that is a potential donors, I will ban you from commenting on our repo. That was distasteful and not a position you should be taking.
	Sat, Jun 14, 2025, 13:33:54 - deverickapollo: And if you got a problem. Come see me
	Sat, Jun 14, 2025, 13:34:48 - plowsof: point to the slander? Not AI slop which contained [ Your Name ] place holder?
	Sat, Jun 14, 2025, 13:35:09 - deverickapollo: I ain’t wasting my time. You got my point
	Sat, Jun 14, 2025, 13:36:14 - plowsof: " If you ever slander a community member that is a potential donors," again, how have i slandered?
	Sat, Jun 14, 2025, 13:36:21 - deverickapollo: <@ofrnxmr:xmr.mx "also, are the monero libs manage..."> Not yet! Will be coming in soon
	Sat, Jun 14, 2025, 13:38:14 - deverickapollo: <@plowsof:matrix.org "if the rpc wallet run time args ..."> Much better focus
	Sat, Jun 14, 2025, 13:38:30 - ofrnxmr: <@deverickapollo:matrix.org "Not yet! Will be coming in soon"> I mean, who manages them?
	Sat, Jun 14, 2025, 13:38:58 - ofrnxmr: Where is the docker pulled from
	Sat, Jun 14, 2025, 13:39:09 - deverickapollo: BTCPay owns the repo - they manage the updates. It’s from their repo. One sec, can link you
	Sat, Jun 14, 2025, 13:40:34 - ofrnxmr: <@deverickapollo:matrix.org "I’m going to come at this differ..."> Ai generated praise is trash
	Sat, Jun 14, 2025, 13:40:55 - ofrnxmr: didnt even write his own name
	Sat, Jun 14, 2025, 13:41:16 - deverickapollo: https://github.com/btcpayserver/dockerfile-deps
	Sat, Jun 14, 2025, 13:41:21 - plowsof: [ Your name ] was input maunally later, so technically he did
	Sat, Jun 14, 2025, 13:41:38 - plowsof: 😆
	Sat, Jun 14, 2025, 13:42:33 - ofrnxmr: <@deverickapollo:matrix.org "https://github.com/btcpayserver/..."> Ok 18.4.0 is available, and 18.4.1 coming in a week or so
	Sat, Jun 14, 2025, 13:45:17 - ofrnxmr: <@plowsof:matrix.org "it appears to be lack of --no-in..."> +1. Try adding the --no-initial-sync
	Sat, Jun 14, 2025, 13:45:54 - ofrnxmr: Once 9950 and 9935 (if merged to release branch) are merged upstream, it should be more reliable
	Sat, Jun 14, 2025, 13:47:08 - ofrnxmr: wallet-rpc pulls hashes from genesis, and takes a minute or 2 of unresponsiveness - fixed by 9935.

	when syncing, wallet rpc will ignore any/all commands sent to it, including sigint etc. 9950 fixes this
	Sat, Jun 14, 2025, 13:47:53 - ofrnxmr: --no-initial-sync will start the rpc server before calling refresh, allowing you to send commands to it before it starts pulling hashes
	Sat, Jun 14, 2025, 13:48:50 - ofrnxmr: So it should be used if you require responses _at_ startup
	Sat, Jun 14, 2025, 13:54:42 - deverickapollo: napoly: should be able to bump version used on the integration test for now.
	Sat, Jun 14, 2025, 13:58:05 - plowsof: where do i see you? im in the btcpay monero room right?
	Sat, Jun 14, 2025, 14:08:13 - deverickapollo: I’m in Lincoln Nebraska but happy to meet you anywhere
	Sat, Jun 14, 2025, 14:08:49 - ofrnxmr: https://github.com/btcpayserver/dockerfile-deps/pull/114
	Sat, Jun 14, 2025, 14:09:20 - ofrnxmr: "cash me ousside how bow daht" vibes
	Sat, Jun 14, 2025, 14:09:45 - deverickapollo: I ain’t no keyboard warrior - come correct
	Sat, Jun 14, 2025, 14:10:15 - deverickapollo: I show respect and work hard. No time for simple and def not kids looking for direction
	Sat, Jun 14, 2025, 14:17:23 - plowsof: you've just learned something vital about wallet-rpc, you're welcome.
	Sat, Jun 14, 2025, 14:18:04 - plowsof: i don't know what "No time for simple and def not kids looking for direction" means in relation to this all neither
	Sat, Jun 14, 2025, 14:18:53 - ofrnxmr: tbh, I don't know what it means at all.. so i just pretended like i didn't see it 🫠
	Sat, Jun 14, 2025, 14:19:04 - plowsof: if i needed to meet you in person to tell you about --no-initial-sync there would be a huge delay
	Sat, Jun 14, 2025, 14:20:08 - ofrnxmr: who are the kids looking for direction? And what does "no time for simple" mean? All this and more, on the next exciting episode of dragon ball zeeeee
	Sat, Jun 14, 2025, 14:20:49 - ofrnxmr: Anyway, signing off
	[ your name ]
	Market capital investment group
	Sun, Jun 15, 2025, 09:04:59 - napoly: feels like u guys are overreacting.. i just gave dude an emoji and moved on.. wasn't really analyzing it any further.. did not receive the [your name] email since i'm not following btcpay-monero lol
	Sun, Jun 15, 2025, 09:06:30 - napoly: you will need to ping seth or someone from btcpay for review/approval.. fyi
	Sun, Jun 15, 2025, 09:21:15 - napoly: it’s understandable that you guys at CCS are triggered about AI.. but this dude here was just showing appreciation.. not trying to lock in or gain funds
	Sun, Jun 15, 2025, 09:50:52 - ofrnxmr: <@napoly:matrix.org "you will need to ping seth or so..."> Why would i need to ping seth? why can't you approve it?
	Sun, Jun 15, 2025, 10:15:12 - ofrnxmr: Also, i missed https://github.com/btcpayserver/dockerfile-deps/pull/113
	Sun, Jun 15, 2025, 10:15:23 - ofrnxmr: Seth already sent a pr
	Sun, Jun 15, 2025, 10:15:33 - napoly: i don't have rights there.. we scratched the surface of having them under btcpay-monero.. but the outcome for now is to leave them there.. as they are used to create a compose yml
	Sun, Jun 15, 2025, 10:19:06 - ofrnxmr: You don't need rights to approve (i just approved seths. Closing mine)
	Sun, Jun 15, 2025, 12:51:09 - plowsof: deverickapollo: have you had time to think about your behaviour yesterday?
	Sun, Jun 15, 2025, 12:53:17 - ofrnxmr: <@ofrnxmr:xmr.mx "Also, i missed https://github.co..."> Or anyone have time to approve seth's 113?
	Sun, Jun 15, 2025, 12:54:30 - ofrnxmr: Or (i assume you guys are in contact) to reach out to ask them to merge it
	Sun, Jun 15, 2025, 13:31:02 - ofrnxmr: <@ofrnxmr:xmr.mx "Or anyone have time to approve s..."> Or maybe time 2 migrate it to btcpay-monero
	Sun, Jun 15, 2025, 13:38:22 - ofrnxmr: I seem to be using the the word "or" a lot today
	Sun, Jun 15, 2025, 13:56:33 - plowsof: napoly: so adding a health check to xmrwallet (looping until the rpc server is online/responsive) doesn't seem to fix the issue, --no-initial-sync can also be ignored as its starting with a wallet dir. so the docker-compose file doesnt actually open a wallet, and im wondering where this happens for the test?
	Sun, Jun 15, 2025, 13:58:07 - plowsof: alot of warnings in the logs giving me a headache too
	Sun, Jun 15, 2025, 14:55:58 - saeedab joined the room
	Sun, Jun 15, 2025, 14:58:32 - plowsof: similar timeout error on this one https://github.com/btcpay-monero/btcpayserver-monero-plugin/actions/runs/15640176398/job/44065403739#step:9:2150
	Sun, Jun 15, 2025, 15:02:27 - plowsof: is there simply no account label because there is no wallet created / opened / synced? 🤷
	Sun, Jun 15, 2025, 15:15:08 - napoly: would be cool if u could investigate this.. i did not dig into this direction tbh.. or at least creating a documentation for this process.. it's always like someone from the comm has to go to their repos to update it.. an emergency update could give us a headache to push through as seems even seth is waiting for a merge
	Sun, Jun 15, 2025, 15:16:51 - napoly: yes!.. if you try locally and your containers are running the test will go green afaik
	Sun, Jun 15, 2025, 15:22:48 - plowsof: you mentioned it running the 2nd time? or do you mean 2nd time after commenting out the 'wait for wallet label'?
	Sun, Jun 15, 2025, 15:23:16 - plowsof: so the actual problem is something/somewhere is trying to load info from a wallet-rpc which the dockerfile started, that has --wallet-dir , and no open wallet?
	Sun, Jun 15, 2025, 15:30:22 - plowsof: napoly: pls clarify what you mean about it working the 2nd time, so i can go further
	Sun, Jun 15, 2025, 15:32:21 - napoly: 2nd time means having those containers up and running already (heated up)
	Sun, Jun 15, 2025, 15:33:35 - plowsof: ok but with wallets opened, synced? - done outside of the docker-compose.yml which only starts the wallet-rpc? and thats definitely the reason why this isnt working? i ask because i know absolutely nothing currently
	Sun, Jun 15, 2025, 15:34:41 - plowsof: we can add healthchecks (service_healthy) to pause/wait for things where necessary in this docker-compose, but the tests need rpc open _with a wallet open_?
	Sun, Jun 15, 2025, 16:54:23 - napoly: i donno either.. will need to dig it tbh
	Sun, Jun 15, 2025, 19:14:30 - plowsof: napoly i got something to run finally, what a headache ehhhhh ill gather a PR for you
	Sun, Jun 15, 2025, 19:53:29 - plowsof: napoly would you like me to look into your code coverage report failing to upload artifacts? or why the lint errors without saying why?
	Sun, Jun 15, 2025, 19:55:13 - napoly: got link? u should create a pr..
	Sun, Jun 15, 2025, 19:56:43 - plowsof: PR here https://github.com/btcpay-monero/btcpayserver-monero-plugin/pull/33
	Sun, Jun 15, 2025, 19:58:41 - plowsof: likely a space after .. will remove and re-run to allow for the lint to fail without saying (i hope)
	Sun, Jun 15, 2025, 20:00:04 - napoly: in readme: To enforce formatting for the whole project, run `dotnet format btcpay-monero-plugin.sln --exclude submodules/\* --verbosity diagnostic`
	Sun, Jun 15, 2025, 20:00:49 - plowsof: ah yes, i do remember glancing at a blog too for dev env?
	Sun, Jun 15, 2025, 20:01:43 - plowsof: ideally the lint would be verbose in the workflow , todo?
	Sun, Jun 15, 2025, 20:02:33 - napoly: it is: --verbosity diagnostic
	Sun, Jun 15, 2025, 20:02:33 - plowsof: this one https://btcpay.host/developer/2025/06/09/btcpay-monero-developer-setup.html
	Sun, Jun 15, 2025, 20:02:56 - napoly: yeah.. copy of readme
	Sun, Jun 15, 2025, 20:03:35 - napoly: C:\a\btcpayserver-monero-plugin\btcpayserver-monero-plugin\BTCPayServer.Plugins.IntegrationTests\Monero\MoneroPluginUITest.cs(18,76): error WHITESPACE: Fix whitespace formatting. Replace 11 characters with '\r\n\s\s\s\s\s\s\s\s'. [C:\a\btcpayserver-monero-plugin\btcpayserver-monero-plugin\BTCPayServer.Plugins.IntegrationTests\BTCPayServer.Plugins.IntegrationTests.csproj]
	Sun, Jun 15, 2025, 20:03:38 - plowsof: am i missing something @ https://github.com/plowsof/btcpayserver-monero-plugin/actions/runs/15666251510/job/44130728201#step:5:1
	Sun, Jun 15, 2025, 20:04:55 - napoly: tabs.. instead of spaces ? smthng like that
	Sun, Jun 15, 2025, 20:05:18 - plowsof: thanks, im blind, will fix
	Sun, Jun 15, 2025, 20:08:52 - plowsof: ok good, it was the expected space, lint working fine
	Sun, Jun 15, 2025, 20:12:48 - plowsof: do the workflows randomly fail? xD
	Sun, Jun 15, 2025, 20:13:17 - napoly: yeah.. need to stabilize them a bit
	Sun, Jun 15, 2025, 20:13:43 - plowsof: ok. i was able to get a successful run with this PR earlier
	Sun, Jun 15, 2025, 20:13:56 - napoly: re-running it
	Sun, Jun 15, 2025, 20:14:22 - plowsof: i dont trust re-running on github, never works for me lol
	Sun, Jun 15, 2025, 20:15:06 - plowsof: ok sorry for spamming everyone
	Sun, Jun 15, 2025, 20:16:20 - napoly: even if green.. it will take me some time to look through it fyi.. as bitcoind doesn't need this health check
	Sun, Jun 15, 2025, 20:18:17 - plowsof: health checks, to poll get_version which tests will 'depends_on'
	https://medium.com/@cbaah123/docker-compose-health-checks-made-easy-a-practical-guide-3a340571b88e
	service_healthy is defined by the healthcheck
	https://docs.docker.com/compose/how-tos/startup-order/
	Sun, Jun 15, 2025, 20:19:07 - plowsof: had to add condition: to all because you cant mix - this with this: 🤷
	Sun, Jun 15, 2025, 20:19:39 - plowsof: essentially it loops get_version curl , and waits for it to return
	Sun, Jun 15, 2025, 20:19:49 - napoly: do we know why bitcoind doesn't need this?
	Sun, Jun 15, 2025, 20:20:23 - plowsof: from my exp bitcoind starts its own http server (we use wallet rpc binary that kinda sucks sometimes?)
	Sun, Jun 15, 2025, 20:20:59 - plowsof: ah nvm, bitcoind (the daemon) not the 'wallet-rpc', i do not know
	Sun, Jun 15, 2025, 20:21:41 - napoly: ok.. thnx.. will need to find time to look at it
	Sun, Jun 15, 2025, 20:21:50 - plowsof: i made a typo , thanks ofrnxmr
	Sun, Jun 15, 2025, 20:22:06 - plowsof: create xmr_wallet.. open xnr_wallet 🧠
	Sun, Jun 15, 2025, 20:23:00 - plowsof: actually napoly, this probably doesnt need --no-initial-sync because we start rpc without opening a wallet (and syncing)
	Sun, Jun 15, 2025, 20:23:17 - plowsof: and the monerod its connecting to is --offline, so will never sync
	Sun, Jun 15, 2025, 20:24:12 - napoly: ya clean it pls up from unnecessary stuff.. would be definitely better and easier to check later on
	Mon, Jun 16, 2025, 07:16:54 - ajs_ joined the room
	Mon, Jun 16, 2025, 12:20:05 - plowsof: deverickapollo: have you had a chance to think about your behaviour yet? not sure if you caught the last time i sent this question.
	Mon, Jun 16, 2025, 12:30:00 - deverickapollo: I said my part and left it. Let’s keep convo focused. You did good yesterday. Keep it up
	Mon, Jun 16, 2025, 12:30:58 - deverickapollo: https://btcpay.host/ccs.html
	Mon, Jun 16, 2025, 12:38:13 - plowsof: there is a problem with the repo for linux users, it will need .gitattributes fixes but i dont have a windows machine to go back and forth, for now linux contributors must ignore 2 files , the method ive found working is `git update-index --assume-unchanged` <file> , currently this is img/Checkout.png and another .sln file cc ofrnxmr
	Mon, Jun 16, 2025, 12:39:10 - plowsof: so is my interpretation of your words an attempt at physical intimidation? a perceived physical threat to a staff member is quite important to clarify