-
luigi1111w
travis says "You have used 11370 of 10000 credits"
-
luigi1111w
I turned off the setting to use credits for OSS projects but it doesn't seem to do anything
-
selsta
-
selsta
looks like we have to manually request more credits
-
selsta
that’s annoying
-
selsta
"We will be offering an allotment of OSS minutes that will be reviewed and allocated on a case by case basis. Should you want to apply for these credits please open a request with Travis CI support stating that you’d like to be considered for the OSS allotment."
-
selsta
looks like Travis stopped supporting OSS completely
news.ycombinator.com/item?id=25338983
-
Snipa
Well, that's annoying.
-
locustlord
selsta, what is this?
-
locustlord
credit allocation?
-
locustlord
what are credits in this context?
-
selsta
CI for testing development builds (e.g. does it compile?)
-
locustlord
isn't it best to compile on real hardware anyway?
-
locustlord
surely someone has a spare laptop that can be used
-
Snipa
Travis CI runs on every merge, and runs on "real" hardware for all that "real" matters in the CI processes with replicable builds.
-
selsta
it runs on every PR and every merge
-
Snipa
Up until now, they happened to support OSS pretty well, so easier than relying on someone to have 24/7 abilable hardware.
-
selsta
yep
-
locustlord
I see
-
locustlord
so now what?
-
Snipa
We'll figure something out, couple people in that HN thread moved to Circle CI, IIRC we use GH actions, which might be a solution as well.
-
iDunk
GH Actions for depends targets ?
-
selsta
Should not be too difficult to port to GH Actions, but we will end up with a lot of jobs per PR.
-
selsta
CircleCI looks nice too
-
selsta
GH Actions + caching will be fine
-
selsta
will look into porting it
-
TheCharlatan
should be easy enough. Ping me if you need help selsta. I can't seem to find information on what the actual parallel job limit is on GH actions.
-
selsta
20 concurrent builds
-
TheCharlatan
nice, should be plenty.
-
selsta
mining and p2p functional tests still occasionally fail, and I was not able to figure out which timeout to bump yet
-
selsta
(on CI)
-
selsta
AssertionError: Failed to mine successor to block 13 (initial block = 11)
-
moneromooo
Not sure I want to care. We'd really need a way to ask it "are you still working on it", and there's no such thing.
-
moneromooo
No sane one anyway :)
-
moneromooo
Oh, or bump all timeouts to silly values, but test every couple seconds whether we're in the state we want.
-
moneromooo
That's better.
-
Lyza
morning y'all
-
Lyza
working on doing deterministic builds for the new version. seems like everything went right, but the .zip files for the Windows versions can't be opened, as if the archives are corrupt. all of the tar.bz2 files are fine
-
Lyza
hashes seem to match though
-
Lyza
mmmm never mind, unzip in Linux will unpack them but Windows can't handle them and neither can 7zip. so.... that's weird. but not a huge deal I guess?
-
Lyza
but I mean, they're Windows releases, so maybe it is an issue. if I were to unpack and repack it in Windows or something the hashes would stop matching I presume
-
fluffypony
that's weird
-
fluffypony
7zip is pretty robust
-
Lyza
7zip just gives some generic "cannot open as archive" while windows gives me a real interesting "Please inset the last disk of the Multi-Volume set"
-
webname888
Why lie about something that can be easily disproven?
-
webname888
-
webname888
-
webname888
Why steal from your community and then laugh at them?
-
webname888
-
webname888
Reason is the same - to laugh at morons that are gullible enough to believe you and repeat your lies.
-
Lyza
update: I am dumb
-
Lyza
like 4 of the files got corrupted copying them over to the Windows machine somehow, everything is good now
-
sethsimmons
Thats the struggle — report something that could be serious and just turns out to be bad luck 😅
-
aubergine
hi
-
aubergine
is monero nodejs library still maintained?
-
aubergine
?
-
gingeropolous
much less cpu load in 17.1.6. thanks devs
-
sethsimmons
<gingeropolous "much less cpu load in 17.1.6. th"> Oh, really? Any specific idea why?
-
moneromooo
Dropping some cunt's spam.
-
gingeropolous
i saw some conversation about a race condition. and what mooo said
-
kinghat[m]
its a technical term
-
sethsimmons
Ah the spam reduction, didn’t even think about the effect that will have on load and bandwidth
-
aubergine
sethsimmons I always get 401 error upon connection
-
aubergine
despite setting the rpc login values correctly
-
sethsimmons
I’m not sure why you mentioned me I know nothing about that library.
-
sethsimmons
You may want to reach out to the author if possible or in #monero
-
sethsimmons
That is managed by a third party AFAIK and so isn’t likely to be a good topic for this room
-
aubergine
ok thank you
-
aubergine
monero should start to be more dev-friendly btw
-
aubergine
tons of third party libraries outdated or poorly supported
-
moneromooo
What could monero do about it ?
-
sethsimmons
Monero is a decentralized community.
-
sethsimmons
It’s up to the devs of the repos to keep it up to date or not, or newcomers to take them over as needed.
-
sethsimmons
There’s no company or anything to be more dev-friendly.
-
sethsimmons
Hundreds of devs work on Monero out of their own choice and seem to enjoy it!
-
aubergine
redirect part of the funding from developing the core to the ecosystem
-
Lyza
people can request funding for things
-
gingeropolous
what funding?
-
aubergine
otherwise monero will be a nice expensive toy that nobody can fully use
-
Lyza
funding for doing work, I should say
-
gingeropolous
looks like someone wants to open a ccs!
-
sethsimmons
There is no central funding, aubergine. Each dev is funded individually if they request it and if the Monero community decides to donate to it.
-
sethsimmons
I don’t think you understand how Monero works, this isn’t some corporate coin with vast treasuries siphoned from the block reward.
-
sethsimmons
The community funds devs of their own free will.
-
gingeropolous
and if u see a problem, fix it :)
-
gingeropolous
if you can
-
aubergine
I did, for free
-
aubergine
but it is a tiresome process and with the older libs the maintainer is no longer interested in helping
-
kinghat[m]
how can you tell an ecosystem how to operate if you dont understand how it operates?
-
aubergine
sethsimmons I know, it's just apparently the community has been educated to support core development while neglecting the surrounding ecosystem
-
sethsimmons
I don’t think we’re neglecting the surrounding ecosystem — if the dev wanted to be funded and showed why it was a good idea the community could fund continued development
-
Lyza
folks just filled a huge CCS for an atomic swap implementation so I wouldn't say the ecosystem is being ignored. there's just, limited manpower. limited people who are both interested in and capable of the work. it'll change as the ecosystem grows but it is a bit of a chick and egg thing
-
sethsimmons
Libraries are normally maintained by those using them for their own products — as more products get built on Monero more will be consistently maintained
-
aubergine
it's like selling a car engine and expecting people to build around it
-
scoobybejesus
dude, just talk to woodser
-
selsta
last commit was 20 hours ago, what do you expect from "actively maintained"?
-
binaryFate
New binaries (CLI and GUI) for v0.17.1.6 are available on getmonero.org
-
kayabaNerve_
Aw :( aubergine already left. I was going to bring up the Rust and Python libraries
-
sethsimmons
We may want to add a note to the release notes about needing to specify —restricted-rpc-bind-ip in order to keep exposing restricted rpc to non-localhost IPs
-
sethsimmons
The default behavior changed and merely updating a node with —restricted-rpc-bind-port in use breaks outside accessibility.
-
selsta
it should only be necessary if you have both restricted-rpc-bind-port and rpc-bind-port
-
hyc
hm, it shouldn't have broken backward compay
-
hyc
compat
-
sethsimmons
<selsta "it should only be necessary if y"> This is my setup, so I guess that’s it — but that seems like the normal way users would operate with a distinct restricted port
-
sethsimmons
gingeropolous noticed the same behavior
-
sethsimmons
The new version drops restricted-rpc to localhost only unless you add the new flag and specificy IP/0.0.0.0
-
selsta
yea, I did not have this setup so I did not notice it during testing
-
selsta
I usually do restricted only with tmux
-
hyc
ok, I see. the code in src/rpc/rpc_args.cpp sets config.restricted_bind_ip unconditionally; if the arg wasn't provided it defaults to localhost
-
hyc
and it sounds like you would have expected it to default to <current rpc_bind_ip> instead
-
selsta
I think current behaviour makes more sense, only issue is backwards compatibility.
-
hyc
agreed
-
sethsimmons
Yes the current behavior is safest and correct, just needs to be clear to those updating that they now have to specify that flag if they’re previously used the same config as me and many others.
-
selsta
do you have a suggestion for the release notes?
-
sethsimmons
Maybe something similar to this:
-
sethsimmons
“A note for public RPC node operators upgrading to v0.17.1.6 — if you’re using the —restricted-rpc-bind-port option along with —rpc-bind-ip currently you will need to add the new —restricted-rpc-bind-ip arg or else the daemon will bind restricted RPC only to localhost.”
-
needmoney90
deb
-
needmoney90
oops
-
needmoney90
debruyne can we repin?
-
needmoney90
and add that note
-
selsta
sethsimmons: ok will make sure to add it to release notes
-
hyc
sounds good
-
Inge-
did BTC see similar kinds of shenanigans they had to harden against?
-
sethsimmons
Not that I’ve ever heard of but I would love to know more if so.
-
selsta
BTC does not have privacy to begin with so a lot of the attacks don't apply
-
selsta
there is software to sybil attack BTC
-
selsta
would be interesting to know how bitcoin handles target height
-
Inge-
I know it has some logic to ban misbehaving nodes
-
Inge-
anyway, good job on being light-footed in getting mitigations in.
-
dEBRUYNE
needmoney90: Can just edit the post?
-
needmoney90
considering the title is > v0.17.1.5 as soon as possible and the latest version is 17.1.6
-
needmoney90
we prolly need a new title to avoid confusion
-
needmoney90
and reddit does not allow editing that
-
needmoney90
debruyne
-
needmoney90
I'm speaking specifically about our current pin #2
-
needmoney90
not seth's post
-
rehrar
dEBRUYNE selsta if we think that this has all of the mitigations that are necessary, perhaps it might have been best to make it 17.2 so it seems like a very important thing to update to rather than just a .6
-
selsta
Ledger version locks their software
-
rehrar
as a whole the third digit isn't used as much as it could be
-
moneromooo
Unlikely it's the last of it.
-
selsta
then we would have to reach out to Ledger again, tell all users to update the Ledger monero app and it always results in huge amount of support threads
-
needmoney90
Yeah but if that happens selsta will deal with it and I dont have to think about it 👀
-
rehrar
something I've discussed with selsta before that maybe I can get some other opinions on, is the Whonix guy who's doing packaging for us recommends some sort or ESR for Monero, but it doesn't seem that this is feasible for us right?
-
selsta
no it is not feasible
-
selsta
IMO
-
needmoney90
ESR?
-
rehrar
extended support release
-
needmoney90
ah. Give it two years.
-
selsta
We already mostly only add bug fixes in point releases
-
rehrar
I totally understand that nothing about the past few months of releases was typical.
-
selsta
Afaik even bitcoin does not have ESR / LTS releases and bitcoin is way more stable
-
selsta
luigi1111w: can you add "A note for public RPC node operators upgrading to v0.17.1.6 - if you’re using the --restricted-rpc-bind-port option along with --rpc-bind-ip currently you will need to add the new --restricted-rpc-bind-ip arg or else the daemon will bind restricted RPC only to localhost." to CLI release notes? similar to
monero-project/monero-site #1363
-
luigi1111w
like that?
-
selsta
yes, thanks
-
dEBRUYNE
<needmoney90> I'm speaking specifically about our current pin #2 <= Oh I see
-
dEBRUYNE
We can depin that I guess
-
selsta
luigi1111w: also please also update hashes.txt.sig sometime later, not urgent
-
moneromooo
Fingerprinting.
-
lza_menace
seems like it would be useful to be available to crawl the network and determine what versions nodes are
-
lza_menace
anywho - thanks contributors, great work on the attack mitigations