Jira (PUP-11723) Remove dependency on private class Concurrent::RubyThreadLocalVar

22 views
Skip to first unread message

Josh Cooper (Jira)

unread,
Apr 14, 2023, 5:31:03 PM4/14/23
to puppe...@googlegroups.com
Josh Cooper updated an issue
 
Puppet / Bug PUP-11723
Remove dependency on private class Concurrent::RubyThreadLocalVar
Change By: Josh Cooper
Summary: Revisit Remove dependency on private class Concurrent::RubyThreadLocalVar
Add Comment Add Comment
 
This message was sent by Atlassian Jira (v8.20.11#820011-sha1:0629dd8)
Atlassian logo

Josh Cooper (Jira)

unread,
May 4, 2023, 1:58:02 PM5/4/23
to puppe...@googlegroups.com
Josh Cooper updated an issue
Puppet relies on the Concurrent::RubyThreadLocalVar class which was removed in 1.2.0, see https://github.com/ruby-concurrency/concurrent-ruby/issues/986#issuecomment-1400997040

We explicitly rely on the Ruby implementation when running on JRuby because of a bug, see PUP-9931 and https://github.com/puppetlabs/puppet/commit/9182bc3dd2576f409a6d01fb5c08d392670e90a2

For now we've pinned back to < 1.2 (PUP-11722) but we should look into not relying on a private class and remove our concurrent-ruby pin.


Grooming:
* Replace removed class with Concurrent::ThreadLocalVar, see commit where RubyThreadLocalVar was removed https://github.com/ruby-concurrency/concurrent-ruby/commit/30465396

Josh Cooper (Jira)

unread,
May 10, 2023, 12:51:03 PM5/10/23
to puppe...@googlegroups.com
Josh Cooper updated an issue
Puppet relies on the Concurrent::RubyThreadLocalVar class which was removed in 1.2.0, see https://github.com/ruby-concurrency/concurrent-ruby/issues/986#issuecomment-1400997040

We explicitly rely on the Ruby implementation when running on JRuby because of a bug, see PUP-9931 and https://github.com/puppetlabs/puppet/commit/9182bc3dd2576f409a6d01fb5c08d392670e90a2

For now we've pinned back to < 1.2 (PUP-11722) but we should look into not relying on a private class and remove our concurrent-ruby pin.

Grooming:
* Replace removed class with Concurrent::ThreadLocalVar, see commit where RubyThreadLocalVar was removed https://github.com/ruby-concurrency/concurrent-ruby/commit/30465396

* Spec tests should pass on MRI and JRuby
* Change should be made in 7.x and main

Josh Cooper (Jira)

unread,
May 10, 2023, 12:51:03 PM5/10/23
to puppe...@googlegroups.com

Josh Cooper (Jira)

unread,
May 10, 2023, 12:53:03 PM5/10/23
to puppe...@googlegroups.com
Josh Cooper updated an issue
Puppet relies on the Concurrent::RubyThreadLocalVar class which was removed in 1.2.0, see https://github.com/ruby-concurrency/concurrent-ruby/issues/986#issuecomment-1400997040

We explicitly rely on the Ruby implementation when running on JRuby because of a bug, see PUP-9931 and https://github.com/puppetlabs/puppet/commit/9182bc3dd2576f409a6d01fb5c08d392670e90a2

For now we've pinned back to < 1.2 (PUP-11722) but we should look into not relying on a private class and remove our concurrent-ruby pin.

Grooming:
* Replace removed class with Concurrent::ThreadLocalVar, see commit where RubyThreadLocalVar was removed https://github.com/ruby-concurrency/concurrent-ruby/commit/30465396
* Spec tests should pass on MRI and JRuby
* Change should be made in both 7.x and main
* Restore concurrent-ruby gem dependency to {{~> 1.0}} in both .gemspec and ext/project_data.yaml The change must be made in two places due to PA-5400.

Josh Cooper (Jira)

unread,
May 10, 2023, 12:53:03 PM5/10/23
to puppe...@googlegroups.com
Josh Cooper updated an issue
Change By: Josh Cooper
Fix Version/s: PUP 8.1.0
Fix Version/s: PUP 7.25.0

Tony Vu (Jira)

unread,
May 10, 2023, 1:48:03 PM5/10/23
to puppe...@googlegroups.com
Tony Vu updated an issue
Change By: Tony Vu
Sprint: Phoenix 2023-05-24

Michael Hashizume (Jira)

unread,
May 10, 2023, 2:03:03 PM5/10/23
to puppe...@googlegroups.com

Michael Hashizume (Jira)

unread,
May 24, 2023, 1:08:03 PM5/24/23
to puppe...@googlegroups.com
Michael Hashizume updated an issue
Change By: Michael Hashizume
Sprint: Phoenix 2023-05-24 , Phoenix 2023-06-07
This message was sent by Atlassian Jira (v8.20.21#820021-sha1:38274c8)
Atlassian logo

Michael Hashizume (Jira)

unread,
Jun 8, 2023, 7:40:02 PM6/8/23
to puppe...@googlegroups.com
Michael Hashizume updated an issue
Change By: Michael Hashizume
Release Notes: Bug Fix
Release Notes Summary: The Puppet::ThreadLocal class no longer relies on concurrent-ruby's private Concurrent::RubyThreadLocalVar class and instead uses Concurrent::ThreadLocalVar.

Parker Leach (Jira)

unread,
Jun 12, 2023, 6:02:01 PM6/12/23
to puppe...@googlegroups.com
Reply all
Reply to author
Forward
0 new messages