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

v0.2.0 #29

Merged
merged 28 commits into from
Aug 10, 2024
Merged

v0.2.0 #29

merged 28 commits into from
Aug 10, 2024

Conversation

kpeluso
Copy link
Collaborator

@kpeluso kpeluso commented Aug 9, 2024

Purpose of Changes and their Description

Link(s) to Ticket(s) or Issue(s) resolved by this PR

Are these changes tested and documented?

  • If tested, please describe how. If not, why tests are not needed.
  • If documented, please describe where. If not, describe why docs are not needed.
  • Added to Unreleased section of CHANGELOG.md?

Still Left Todo

Fill this out if this is a Draft PR so others can help.

kpeluso and others added 28 commits August 2, 2024 02:23
* add changelog + security (#9)

* added dockerfile

* fix bug on build

* change the wkdir to /data

* change the wkdir to /

* fix typo

* back to /node

* push to dockerhub instead

* deployment done

---------

Co-authored-by: Kenny P <17100641+kpeluso@users.noreply.github.com>
* change the wkdir to /data

* change the wkdir to /

* fix typo

* back to /node

* deploy 2 reputer and workers for topic 1

* random
* API Adapter with README

* move API adapter readme inside dir

* Config file/env var + factory adapters + minor fixes
Hello, I found some typos in README.md

"intentend" to "intended"

"be use" to "be used"

And there were also missing space in "InferenceEndpointis required if InferenceEntrypoint is defined.". This canged for "InferenceEndpoint is required if InferenceEntrypoint is defined."

Hope this helps.

Signed-off-by: osrm <90407222+osrm@users.noreply.github.com>
* cleanups

* efficient deployment updates

* fix bug

---------

Co-authored-by: Kenny <pelusoken@gmail.com>
* uniform backoff

* README update
* Update and rename start.docker to init.docker

on devnet, when running the command:

```
chmod +x start.docker
./start.docker
```

 the worker node won't register because it is not funded. The wallet for the worker node is being created and the node is attempting to register to the network in one go. However, for the user this will never work. What they will have to do is
1. run the command
2. kill the terminal
3. fund their wallet
4. rerun the chmod command

The fix proposed is:

1. run init.docker to generate address and then you can get the address funded manually
2. run docker compose --build separately to start your services

Signed-off-by: kush-alloralabs <kush@alloralabs.xyz>

* Update README.md

Signed-off-by: kush-alloralabs <kush@alloralabs.xyz>

* Update README.md

Signed-off-by: kush-alloralabs <kush@alloralabs.xyz>

---------

Signed-off-by: kush-alloralabs <kush@alloralabs.xyz>
* add changelog + security (#9)

* release v0.1.0 (#23)

* add changelog + security

* Deployment (#10)

* add changelog + security (#9)

* added dockerfile

* fix bug on build

* change the wkdir to /data

* change the wkdir to /

* fix typo

* back to /node

* push to dockerhub instead

* deployment done

---------

Co-authored-by: Kenny P <17100641+kpeluso@users.noreply.github.com>

* use internal network dns

* Deployment (#13)

* change the wkdir to /data

* change the wkdir to /

* fix typo

* back to /node

* deploy 2 reputer and workers for topic 1

* random

* API Adapter with README ORA-2066 (#12)

* API Adapter with README

* move API adapter readme inside dir

* Config file/env var + factory adapters + minor fixes

* updating readme to tell you to copy the env file and fill it out before running docker-compose (#14)

* readme changes (#15)

* chore: fix typos README.md (#17)

Hello, I found some typos in README.md

"intentend" to "intended"

"be use" to "be used"

And there were also missing space in "InferenceEndpointis required if InferenceEntrypoint is defined.". This canged for "InferenceEndpoint is required if InferenceEntrypoint is defined."

Hope this helps.

Signed-off-by: osrm <90407222+osrm@users.noreply.github.com>

* update dependencies, do not print conf (#18)

* Cleanups (#16)

* cleanups

* efficient deployment updates

* fix bug

---------

Co-authored-by: Kenny <pelusoken@gmail.com>

* fix issues with docker-compose initialize

* readme fix

* forecast should return nodeValue

* Update dependencies + reputer nonce fix (#19)

* fix deprecated docker compose command

* string interpolation

* uniform backoff (#21)

* uniform backoff

* README update

* Update and rename start.docker to init.docker (#22)

* Update and rename start.docker to init.docker

on devnet, when running the command:

```
chmod +x start.docker
./start.docker
```

 the worker node won't register because it is not funded. The wallet for the worker node is being created and the node is attempting to register to the network in one go. However, for the user this will never work. What they will have to do is
1. run the command
2. kill the terminal
3. fund their wallet
4. rerun the chmod command

The fix proposed is:

1. run init.docker to generate address and then you can get the address funded manually
2. run docker compose --build separately to start your services

Signed-off-by: kush-alloralabs <kush@alloralabs.xyz>

* Update README.md

Signed-off-by: kush-alloralabs <kush@alloralabs.xyz>

* Update README.md

Signed-off-by: kush-alloralabs <kush@alloralabs.xyz>

---------

Signed-off-by: kush-alloralabs <kush@alloralabs.xyz>

* fix the conflicting account issues

* add pull request template

* correct the wkdir for source github actions

---------

Signed-off-by: osrm <90407222+osrm@users.noreply.github.com>
Signed-off-by: kush-alloralabs <kush@alloralabs.xyz>
Co-authored-by: Kenny <pelusoken@gmail.com>
Co-authored-by: Kenny P <17100641+kpeluso@users.noreply.github.com>
Co-authored-by: Diego C <xmariachi@gmail.com>
Co-authored-by: Madison Williams <madisonjw@gmail.com>
Co-authored-by: osrm <90407222+osrm@users.noreply.github.com>
Co-authored-by: kush-alloralabs <kush@alloralabs.xyz>

* remove usage of init.docker

* fix readme conflicts

* fix readme conflicts

* use offchain-node as name

---------

Signed-off-by: osrm <90407222+osrm@users.noreply.github.com>
Signed-off-by: kush-alloralabs <kush@alloralabs.xyz>
Co-authored-by: Kenny P <17100641+kpeluso@users.noreply.github.com>
Co-authored-by: Tobi Okedeji <tobiokedeji@gmail.com>
Co-authored-by: Kenny <pelusoken@gmail.com>
Co-authored-by: Madison Williams <madisonjw@gmail.com>
Co-authored-by: osrm <90407222+osrm@users.noreply.github.com>
Co-authored-by: kush-alloralabs <kush@alloralabs.xyz>
https://upshotworld.slack.com/archives/C06TUMEP3LM/p1723139422909399

By default we should have the worker and reputer run on the same port so that both work in the simplest path.

Signed-off-by: kush-alloralabs <kush@alloralabs.xyz>
* added metrics to offchain node

* expose metrics port on 2112

* update readme

* typo

---------

Co-authored-by: Kenny <pelusoken@gmail.com>
@kpeluso kpeluso requested review from xmariachi and okedeji August 9, 2024 23:10
@okedeji okedeji merged commit 8ca515e into main Aug 10, 2024
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.

6 participants