Fork me on GitHub
#tools-deps
<
2021-11-02
>
kenny16:11:45

Accidentally ran into https://clojure.atlassian.net/browse/TDEPS-207 again. Is there a workaround to allow us to update to the latest cli version? ATM, we're stuck on 1.10.3.933.

Dimitar Uzunov18:11:52

@U083D6HK9 One thing that worked for me is that I removed unaccessible maven repos from the config file, I had the same symptoms otherwise

Alex Miller (Clojure team)19:12:22

@U083D6HK9 latest CLI prerelease includes a speculative fix for this - would love to have any feedback if you want to try 1.10.3.1053

Alex Miller (Clojure team)19:12:46

sorry for the delay in looking at this

kenny19:12:40

Thank you for taking the time Alex. I will take a look later today and let you know. Happy holidays.

kenny03:12:48

Ran out of time. Will try to get to it tomorrow morning. If not, sometime next week.

kenny16:12:10

Agh, holiday delay... Just tested it out. Still getting a hang.

clojure -version                                                                                                                                                                                                                         08:53:05
Clojure CLI version 1.10.3.1053
Then I run:
clojure -A... -Sforce -Stree
DEPRECATED: Libs must be qualified, change deps-deploy => deps-deploy/deps-deploy (/Users/kenny/.gitlibs/libs/com.ardoq/azure-api/4fbac2ddb9eedf0779890ae2c5bb2d1f3dcfdd7f/deps.edn)
DEPRECATED: Libs must be qualified, change deps-deploy => deps-deploy/deps-deploy (/Users/kenny/.gitlibs/libs/com.ardoq/azure-api/4fbac2ddb9eedf0779890ae2c5bb2d1f3dcfdd7f/deps.edn)
DEPRECATED: Libs must be qualified, change clj-http => clj-http/clj-http (/Users/kenny/.gitlibs/libs/com.ardoq/azure-api/4fbac2ddb9eedf0779890ae2c5bb2d1f3dcfdd7f/deps.edn)
DEPRECATED: Libs must be qualified, change clj-time => clj-time/clj-time (/Users/kenny/.gitlibs/libs/com.ardoq/azure-api/4fbac2ddb9eedf0779890ae2c5bb2d1f3dcfdd7f/deps.edn)
Downloading: org/clojure/clojure/maven-metadata.xml from datomic-cloud
And then it hangs. Would another thread dump be useful?

kenny18:12:22

This is such a weird issue. Something about it is non-deterministic. ~1 out of every 10 runs will succeed.

kenny18:12:47

We have 2 s3 maven repos in our :mvn/repos. I could see that affecting things for sure.

Alex Miller (Clojure team)15:01:53

I think I have a bit more insight on this and s3 definitely is required to trigger the condition. still thinking about how to fix it but I think I understand it now at least

kenny16:01:28

Nice. Let me know if there's any info I can provide.

Alex Miller (Clojure team)02:01:15

@U083D6HK9 with those clues, I was able to make a repro of my own, and there's a new prerelease that you can test with a fix 1.10.3.1058

kenny19:01:20

Awesome! Just tested it and it works. Thank you so much for taking the time to fix this.

Alex Miller (Clojure team)19:01:54

thx for the good reports and retests that helped me fix!

✔️ 1
Alex Miller (Clojure team)16:11:11

not sure what you're asking for? not fixed yet

Alex Miller (Clojure team)16:11:56

-207 is relatively rare, are you actually seeing that?

kenny16:11:54

IIRC, I reported it 🙂 Yes, it occurs on every CLI run on any version greater than 1.10.3.933 both locally and on CI.

kenny16:11:17

Several of the recent tdeps improvements are big QoL improvements for us, so I was hoping to find a way for us to upgrade.

Alex Miller (Clojure team)16:11:41

I can look at it on Friday

kenny16:11:39

Oh, awesome. Thank you.