Ticket #4539 (closed defect: fixed)

Opened 7 years ago

Last modified 6 years ago

Use Tubes for Etoys collaboration

Reported by: bert Owned by: bert
Priority: high Milestone: 8.2.0 (was Update.2)
Component: etoys-activity Version:
Keywords: joyride-2233:+ Cc: gdesmott, gregorio
Action Needed: test in build Verified: no
Deployments affected: Blocked By:
Blocking: #6300, #7417

Description

working on tubification.

Change History

Changed 7 years ago by bert

  • milestone changed from Update.1 to Update.2

Changed 6 years ago by bert

  • blocking 6300 added

(In #6300) Etoys still uses direct IPv4 connections obtained from buddy properties (#4539). The plan was to switch to tubes by update.1 but we did not quite finish it.

Changed 6 years ago by gdesmott

Do you plan to use D-Bus or Stream tubes ?

Changed 6 years ago by gdesmott

  • cc gdesmott added

Changed 6 years ago by bert

Stream tubes - this allows us to reuse much of the existing code.

Changed 6 years ago by gdesmott

Any progress on this? We'd like to stop to expose IPv4 address (#6300) in the PS but are blocked on Etoys tube support.

Changed 6 years ago by bert

Still working on it, had to rewrite the Squeak D-Bus bindings first ... should be ready for update.2.

Changed 6 years ago by bert

  • next_action set to code
  • summary changed from Etoys still uses TCP to Use Tubes for Etoys collaboration

The rewritten D-Bus bindings are in now, but it turned out the Etoys collab stuff depends on unique IP addresses to identify peers. When switching to stream tubes, all peers appear as localhost. I'm changing it to use ip:port pairs instead. Should still be ready for 8.2.0.

Changed 6 years ago by bert

  • blocking 7417 added

Changed 6 years ago by gregorio

  • cc gregorio added

Bert,

Is this coded?

Is it important for 8.2.0?

Thanks,

Greg S

Changed 6 years ago by bert

Yes, it is important for reliable collaboration. It is almost fully coded - the tubes work, the p2p can use ip:port now, I just need to put both together.

Changed 6 years ago by bert

  • next_action changed from code to package

Fixed by 2058tubes-bf and preceeding changesets.

Changed 6 years ago by bert

  • keywords joyride-2233:+ added
  • next_action changed from package to test in build

Tested working in joyride-2233.

Changed 6 years ago by bert

  • status changed from new to closed
  • resolution set to fixed
Note: See TracTickets for help on using tickets.