Delivery: Project Aurras MVP - Phase 1 - Milestone 2 & Milestone 3

[MuhammedIrfan]

Milestone Delivery Checklist

• The milestone-delivery-template.md has been copied and updated.
• The invoice form 📝 has been filled out for this milestone.
• This pull request is being made by the same account as the accepted application.
• In case of acceptance, the payment will be transferred to the BTC/ETH payment address provided in the application.
• The delivery is according to the Guidelines for Milestone Deliverables.

Link to the application pull request: w3f/Grants-Program#163

[Noc2]

Let us know once it’s ready for review.

[MuhammedIrfan]

Let us know once it’s ready for review.

Sure @Noc2 . We need to update the payment address. Can we raise a new pull request in applications repo?

[alxs]

@MuhammedIrfan that's the correct procedure 👍

[MuhammedIrfan]

@MuhammedIrfan that's the correct procedure 👍

Thank you @alxs

[MuhammedIrfan]

@Noc2 Kindly look into the delivery. The Article will be published to medium post approval from the grants team and Will be updated in the delivery document with the link to the medium article.

Thank you

[semuelle]

Thanks for the reminder, @MuhammedIrfan. We have a bit of a backlog, but someone will look into it very soon.

[MuhammedIrfan]

Thanks for the reminder, @MuhammedIrfan. We have a bit of a backlog, but someone will look into it very soon.

We understand @semuelle . Thank you.

[mmagician]

@MuhammedIrfan I've had a first go at your milestone 2 & 3. I've ran into a few issues, you can check these directly in my evaluation notes for M2 & M3. Please let me know once you've addressed the questions there, so I could proceed with the process. Thanks for all the work so far!

[MuhammedIrfan]

@MuhammedIrfan I've had a first go at your milestone 2 & 3. I've ran into a few issues, you can check these directly in my evaluation notes for M2 & M3. Please let me know once you've addressed the questions there, so I could proceed with the process. Thanks for all the work so far!

@mmagician Thank you very much. We will soon fix the issues and update you as soon as possible.

[MuhammedIrfan]

@mmagician We have fixed the issues you have pointed. Kindly find the issues and respective PR

Issue HugoByte/aurras#14
PR HugoByte/aurras#15

Issue HugoByte/aurras#12
PR HugoByte/aurras#16

Issue HugoByte/aurras#13
Commits 1, 2

With Regards to Automated Integration Testing, We tried to implement but we stumbled upon many issues, We will implement it in the course of further development.

Kindly review the changes

[MuhammedIrfan]

@mmagician Please find below the link to article that has been published as part of the milestone.

https://medium.com/hugobyte/completion-of-aurras-mvp-a-middleware-for-the-web3-ecosystem-ee4f666be6aa

[mmagician]

@MuhammedIrfan Thanks for the changes, they look good.

However now I'm struggling to deploy your actions with the deploy.sh script. Currently it is failing with:

error: Invalid argument(s). Unable to get action 'push-notification': The connection failed, or timed out. (HTTP status code 502)

It seems that all wsk commands are failing for me now.
I have just updated all the containers with docker-compose pull and started openwhisk seemingly successfully.
docker ps output:

...
22da17f33b94   openwhisk/apigateway:nightly           "/usr/bin/dumb-init …"   2 minutes ago    Up 2 minutes                    0.0.0.0:80->80/tcp, 0.0.0.0:9000->9000/tcp, 0.0.0.0:31001->31001/tcp, 8423/tcp, 0.0.0.0:9090->8080/tcp   openwhisk-apigateway-1
...

---

Also, when trying to test the push-notification action with cargo test --all-features, it's failing because of missing TEST_DEVICE_TOKEN, but I actually have no clue what this relates to or how to get it. Could you add appropriate instructions please?

---

Lastly, I have just updated my local docker setup to the latest engine and uncovered some incompatibility with the env variables, I opened a PR.

[MuhammedIrfan]

@mmagician Thank you for reviewing our delivery and apologies for the obstacles faced while testing.

However now I'm struggling to deploy your actions with the deploy.sh script. Currently it is failing with:

error: Invalid argument(s). Unable to get action 'push-notification': The connection failed, or timed out. (HTTP status code 502)

It seems that all wsk commands are failing for me now. I have just updated all the containers with docker-compose pull and started openwhisk seemingly successfully. docker ps output:

...
22da17f33b94   openwhisk/apigateway:nightly           "/usr/bin/dumb-init …"   2 minutes ago    Up 2 minutes                    0.0.0.0:80->80/tcp, 0.0.0.0:9000->9000/tcp, 0.0.0.0:31001->31001/tcp, 8423/tcp, 0.0.0.0:9090->8080/tcp   openwhisk-apigateway-1
...

This possibly can be due to openwhisk not yet started functioning completely. After we run docker-compose the controller, invoker and Kafka provider will wait for couchdb-setup to stop as the couchdb-setup would be migrating the db for controller and invoker. Thank you for pointing it out, It is usually not noticeable, but due to some reason if the container takes time to start we will encounter this issue. I will add this to the docker-compose deployment repo.

Please keep an eye on the logs folder, We can be sure if the openwhisk started only when we find the below logs

• logs/controller.log
• logs/invoker.log
• logs/kafka-provider.log

Issue: HugoByte/aurras#18

Also, when trying to test the push-notification action with cargo test --all-features, it's failing because of missing TEST_DEVICE_TOKEN, but I actually have no clue what this relates to or how to get it. Could you add appropriate instructions please?

TEST_DEVICE_TOKEN refers to push notification token, the token can be generated from the client for which the client is registered with the FIREBASE_API_KEY, ie in our case client is Substrate Push Notification. Once you start Substrate Push notification and upon opening the Register Balance Notification modal, you can get the Push Notification Token from the input box. It has been kept read only as it is a generated token and to avoid mistyping it. We will add instructions for it finding the token.

Issue: HugoByte/aurras#17

Lastly, I have just updated my local docker setup to the latest engine and uncovered some incompatibility with the env variables, I opened a PR.

We highly appreciate your contribution and Thank you for supporting our project, We are looking forward to more contributions and to make this a community-driven project.

[mmagician]

Great, I'll wait for you to let me know once you're done.

[MuhammedIrfan]

@mmagician Kindly review

Issue: HugoByte/aurras#18
PR: HugoByte/aurras-deployment-docker-compose#3

Issue: HugoByte/aurras#17
PR: HugoByte/aurras#19

[mmagician]

@MuhammedIrfan Apologies for a delayed response. I have finished your evaluation and can happily say that both milestones are accepted now.
You can find the full evaluations here: M2 & M3.

[mmagician]

I've noticed that the numbers on your invoice don't quite match: you had 0.58 for M2 and 0.33 for M3, but have written 0.95 BTC as a total. Please amend that and send us a new, corrected invoice.

[MuhammedIrfan]

@MuhammedIrfan Apologies for a delayed response. I have finished your evaluation and can happily say that both milestones are accepted now. You can find the full evaluations here: M2 & M3.

Thank you very much @mmagician.

I've noticed that the numbers on your invoice don't quite match: you had 0.58 for M2 and 0.33 for M3, but have written 0.95 BTC as a total. Please amend that and send us a new, corrected invoice.

I sincerely apologize for the mistake. We have sent forward the corrected invoice. Please check.

[Noc2]

Great. I just forwarded the new invoice internally.

[MuhammedIrfan]

Great. I just forwarded the new invoice internally.

Thank you @Noc2

[RouvenP]

hi @MuhammedIrfan we transferred the payment today.

[MuhammedIrfan]

hi @MuhammedIrfan we transferred the payment today.

Hi @RouvenP,

You meant about the test payment?

We have received 0.01 BTC @ https://www.blockchain.com/btc/tx/23eec7c9e145ed5fa2a154336035e418fabad0f6e0cb661a3de9fb93f7040e4c

[RouvenP]

hi @MuhammedIrfan exactly - thanks for confirming. We'll send the remainder next week.

[MuhammedIrfan]

hi @MuhammedIrfan exactly - thanks for confirming. We'll send the remainder next week.

@RouvenP Sure

[RouvenP]

hi @MuhammedIrfan we transferred the remainder today.

[MuhammedIrfan]

hi @MuhammedIrfan we transferred the remainder today.

@RouvenP I confirm that we have received. Thank you.