Opened 7 years ago

Closed 6 years ago

#4539 closed defect (fixed)

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
Blocked By: Blocking: #6300, #7417
Deployments affected: Action Needed: test in build
Verified: no

Description

working on tubification.

Change History (14)

comment:1 Changed 7 years ago by bert

  • Milestone changed from Update.1 to Update.2

comment:2 Changed 7 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.

comment:3 Changed 7 years ago by gdesmott

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

comment:4 Changed 7 years ago by gdesmott

  • Cc gdesmott added

comment:5 Changed 7 years ago by bert

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

comment:6 Changed 7 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.

comment:7 Changed 7 years ago by bert

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

comment:8 Changed 6 years ago by bert

  • Action Needed 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.

comment:9 Changed 6 years ago by bert

  • Blocking 7417 added

comment:10 Changed 6 years ago by gregorio

  • Cc gregorio added

Bert,

Is this coded?

Is it important for 8.2.0?

Thanks,

Greg S

comment:11 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.

comment:12 Changed 6 years ago by bert

  • Action Needed changed from code to package

Fixed by 2058tubes-bf and preceeding changesets.

comment:13 Changed 6 years ago by bert

  • Action Needed changed from package to test in build
  • Keywords joyride-2233:+ added

Tested working in joyride-2233.

comment:14 Changed 6 years ago by bert

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