Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remote changes to notebooks not being reflected in real-time #5562

Closed
2 of 7 tasks
akhenry opened this issue Jul 25, 2022 · 6 comments · Fixed by #5563
Closed
2 of 7 tasks

Remote changes to notebooks not being reflected in real-time #5562

akhenry opened this issue Jul 25, 2022 · 6 comments · Fixed by #5563
Labels
bug:regression It used to work. Now it doesn't :( needs:e2e Needs an e2e test severity:critical type:bug

Comments

@akhenry
Copy link
Contributor

akhenry commented Jul 25, 2022

Summary

AKA "multi-user notebook doesn't work any more".

Expected vs Current Behavior

When multiple users are viewing a notebook, changes by one users should be automatically reflected in other user's views. For example, when one user adds a new entry, all users viewing that page should see that entry appear automatically. This used to work but it appears that a recent code change has introduced a regression here.

Steps to Reproduce

  1. Using an instance of Open MCT with Couch DB as the persistence backend
  2. Open a notebook in Firefox (or Safari)
  3. Open the same notebook in Chrome. It's important to use two different browsers as the Couch synchronization uses a shared worker to reduce the number of server connections we are holding open.
  4. Add a new entry in Firefox.
  5. Observe that the new entry does not automatically appear in Chrome.

Environment

  • Open MCT Version: c10e9ad
  • Deployment Type: VIPER test and local dev with Couch DB enabled

Impact Check List

  • Data loss or misrepresented data?
  • Regression? Did this used to work or has it always been broken?
  • Is there a workaround available?
  • Does this impact a critical component?
  • Is this just a visual bug with no functional impact?
  • Does this block the execution of e2e tests?
  • Does this have an impact on Performance?

Additional Information

@akhenry akhenry added type:bug severity:critical bug:regression It used to work. Now it doesn't :( labels Jul 25, 2022
@akhenry akhenry mentioned this issue Jul 26, 2022
15 tasks
@akhenry
Copy link
Contributor Author

akhenry commented Jul 26, 2022

Test Instructions

  1. Execute reproduction steps above and verify that remote changes to notebook entries are automatically reflect
  2. Add and remove tags and verify that changes are automatically reflected for other users (using the same two browser method from step 1.

@jvigliotta
Copy link
Contributor

Verified Fixed: Testathon 8/2/22

@ozyx
Copy link
Member

ozyx commented Aug 2, 2022

Verified Testathon 8/2/22:

I was able to create entries on Chrome and Firefox instances and they appeared in the other, synchronized. Edits, and deletes were also reflected across browser instances.

@akhenry
Copy link
Contributor Author

akhenry commented Aug 8, 2022

Reopening, this regression was again observed in recent demo of 2.0.7

@akhenry akhenry reopened this Aug 8, 2022
@akhenry akhenry self-assigned this Aug 8, 2022
@akhenry akhenry removed their assignment Aug 23, 2022
@jvigliotta jvigliotta self-assigned this Aug 25, 2022
@unlikelyzero unlikelyzero added the needs:e2e Needs an e2e test label Sep 1, 2022
@unlikelyzero unlikelyzero added this to To triage in Improve Test Coverage via automation Sep 1, 2022
@jvigliotta
Copy link
Contributor

Was unable to replicate this issue again, needs retesting.

@jvigliotta
Copy link
Contributor

Verified Fixed - Testathon 9/12/22 (Without Tags, will update, with tags)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug:regression It used to work. Now it doesn't :( needs:e2e Needs an e2e test severity:critical type:bug
Projects
Development

Successfully merging a pull request may close this issue.

5 participants