Bug #18247

[arv] faraday gem warnings on Ruby 2.7+

Added by Ward Vandewege about 2 months ago. Updated about 2 months ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
-
Target version:
Start date:
10/05/2021
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
-
Release relationship:
Auto

Description

e.g. on Ubuntu 20.04 (Focal Fossa) with system Ruby packages:

$ arv
/var/lib/gems/2.7.0/gems/faraday-0.15.4/lib/faraday/options.rb:166: warning: Capturing the given block using Proc.new is deprecated; use `&block` instead
/var/lib/gems/2.7.0/gems/faraday-0.15.4/lib/faraday/options.rb:166: warning: Capturing the given block using Proc.new is deprecated; use `&block` instead
/var/lib/gems/2.7.0/gems/faraday-0.15.4/lib/faraday/options.rb:166: warning: Capturing the given block using Proc.new is deprecated; use `&block` instead
/var/lib/gems/2.7.0/gems/faraday-0.15.4/lib/faraday/options.rb:166: warning: Capturing the given block using Proc.new is deprecated; use `&block` instead
/var/lib/gems/2.7.0/gems/faraday-0.15.4/lib/faraday/options.rb:166: warning: Capturing the given block using Proc.new is deprecated; use `&block` instead
ARVADOS_API_HOST and ARVADOS_API_TOKEN need to be defined as environment variables.
$ ruby -v
ruby 2.7.0p0 (2019-12-25 revision 647ee6f091) [x86_64-linux-gnu]
$ gem list |grep arv
arvados (2.2.2)
arvados-cli (2.2.2)
arvados-google-api-client (0.8.7.4)

Subtasks

Task #18250: review 18247-faraday-versionResolvedWard Vandewege


Related issues

Related to Arvados - Bug #15673: [SDKs][Ruby] Update faraday usageResolved

Related to Arvados - Bug #18309: [gem] faraday dependency difference between arvados and arvados-login-syncResolved10/29/2021

Associated revisions

Revision 2462bc7f
Added by Ward Vandewege about 2 months ago

Merge branch '18247-faraday-version'

refs #18247

Arvados-DCO-1.1-Signed-off-by: Ward Vandewege <>

History

#1 Updated by Ward Vandewege about 2 months ago

  • Status changed from New to In Progress

#2 Updated by Ward Vandewege about 2 months ago

  • Description updated (diff)

#3 Updated by Ward Vandewege about 2 months ago

  • Release set to 42

#4 Updated by Ward Vandewege about 2 months ago

  • Description updated (diff)

#5 Updated by Ward Vandewege about 2 months ago

Hmm. We have faraday 0.15.4 because of the old arvados-google-api-client gem we use, according to a comment in arvados-cli.gemspec:

  # arvados-google-api-client 0.8.7.2 is incompatible with faraday 0.16.2
  s.add_dependency('faraday', '< 0.16')

This was done in #15673, where we also pushed 0.8.7.3 for arvados-google-api-client. So.... the logical solution then may be to depend on that version of arvados-google-api-client and remove this faraday pin. Maybe. Looking into that...

#6 Updated by Ward Vandewege about 2 months ago

  • Related to Bug #15673: [SDKs][Ruby] Update faraday usage added

#7 Updated by Ward Vandewege about 2 months ago

Ready for review in ebf9bc1e9ac39c4c0b4ea36fd3bd5d9411c73a6b on branch 18247-faraday-version

#9 Updated by Lucas Di Pentima about 2 months ago

Test passed - LGTM!

#10 Updated by Ward Vandewege about 2 months ago

  • Status changed from In Progress to Resolved

#11 Updated by Ward Vandewege 30 days ago

  • Related to Bug #18309: [gem] faraday dependency difference between arvados and arvados-login-sync added

Also available in: Atom PDF