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

feat(ootle): init Tari Ootle #1377

Draft
wants to merge 60 commits into
base: feat/tapplets
Choose a base branch
from

Conversation

karczuRF
Copy link
Collaborator

Description

Init PR that aims to merge the work done so for on TU v1 and v2. V2 was about the work on the development of the Tari L2, the Tari Ootle (especially support for running tapplets).

THIS IS STILL A WIP VERSION

Motivation and Context

More about the Tari Universe v2 can be found in the Tari Universe Overview RFC

How Has This Been Tested?

Manually.

What process can a PR reviewer use to test or verify this change?

  1. Run locally Tari Swarm Daemon. At the moment only local tests can be done. The Tari Ootle integration is in progress.
  2. Run any dev tapplet locally.
  3. Go to TU Settings -> Tari Ootle to add dev tapplet or fetch registered tapplets from the Tapplets Registry.

Breaking Changes

  • None
  • Requires data directory on base node to be deleted
  • Requires hard fork
  • Other - Please specify

BREAKING CHANGE: Description what the user should do, e.g. delete a database, resync the chain

Still a lot of work needs to be done to make it work as expected so don't merge until is well tested.

karczuRF and others added 24 commits January 3, 2025 09:29
Description
---
Resolves tari-project#1186 
Add new telemetry service which receives events and immediately sends
them.

example event sent:
```json
{
  "app_id": "1b%i31CBnblWxHYxSanG",
  "cpu_name": "11th Gen Intel(R) Core(TM) i5-1135G7 @ 2.40GHz",
  "created_at": {
    "nanos_since_epoch": 439482034,
    "secs_since_epoch": 1734106140
  },
  "event_name": "checking-latest-version-mmproxy",
  "event_value": { "percentage": 10, "service": "mmproxy" },
  "gpu_name": "Intel(R) Iris(R) Xe Graphics [0x9a49]",
  "os": "Linux",
  "user_id": "v3,FZNcifkHKvDUJ9Luwc4xBe5HiGHV,0.8.24,VWE2AdhnnMOqNDd7fcQnqVVEBAeee68KBM2+8K7iznY=",
  "version": "0.8.24"
}
```

Motivation and Context
---
This should be useful for tracking how many users are stuck during
application setup. We saw some users being stuck at 30% with now
feedback. This should help in debugging issues like that.

How Has This Been Tested?
---
Run application and watch for error message coming from telemetry
service. If no error are present that means that events were sent
correctly. Also check the telemetry dashboard to check if any messages
are present on the server.

To verify frontend command I created test button to invoke like this:
```Typescript
const sendTelemetry = () =>
    invoke('send_data_telemetry_service', {
        eventName: 'settings_opened',
        data: { service: 'frontend-test', percentage: 100 },
    });
```
And found those event present in the kafka dashboard:
```js
{
  "app_id": "CX4h8%%G$xQ0hpnZSu@N",
  "cpu_name": "11th Gen Intel(R) Core(TM) i5-1135G7 @ 2.40GHz",
  "created_at": {
    "nanos_since_epoch": 795656093,
    "secs_since_epoch": 1736248665
  },
  "event_name": "settings_opened",
  "event_value": { "percentage": 100, "service": "frontend-test" },
  "gpu_name": "Intel(R) Iris(R) Xe Graphics [0x9a49]",
  "os": "Linux",
  "user_id": "v3,FZNcifkHKvDUJ9Luwc4xBe5HiGHV,0.8.40,VWE2AdhnnMOqNDd7fcQnqVVEBAeee68KBM2+8K7iznY",
  "version": "0.8.40"
}
```

What process can a PR reviewer use to test or verify this change?
---
Same as above.

Breaking Changes
---

- [x] None
- [ ] Requires data directory on base node to be deleted
- [ ] Requires hard fork
- [ ] Other - Please specify

---------

Co-authored-by: Brian Pearce <brianp@users.noreply.github.com>
## Fixes tari-project#1203 

### [ Summary ]
- Marked wallet directory for skipping deletion when `reset_wallet` is
false
- Added extra check for omitting keychain items in deletion process when
`reset_wallet` is false

---------

Co-authored-by: Brian Pearce <brianp@users.noreply.github.com>
@karczuRF karczuRF requested a review from a team as a code owner January 14, 2025 11:27
@karczuRF karczuRF marked this pull request as draft January 14, 2025 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants