c6fc / npk

A mostly-serverless distributed hash cracking platform
522 stars 62 forks source link

Add link to campaign from dashboard #47

Closed bzekanovic closed 2 years ago

bzekanovic commented 2 years ago
  1. Any way this message can be removed or updated when campaign is running "No status reports received. Nodes may have errored out" with something more accurate? image

  2. Manage Campaigns page should also include "Start" function since right now you can only execute "Cancel / Delete" functions from that page.

  3. Any way to remove hyperlink to AWS EC2 instance that is shown on dashboard and add hyperlink to related campaign? image

c6fc commented 2 years ago
  1. That's precisely the state of the campaign - I can hide the 'Nodes may have errored out' piece for a bit, but if you have more specific suggestions, I'm all ears.
  2. This is intentional.
  3. I plan to link this to the instance log. I can add a separate link to the campaign though, that's a good idea.
bzekanovic commented 2 years ago
  1. Maybe replace it with something that would encourage users to check output logs to determine state of each instance. You already gave them all info they need to determine if spot is rejected / hashcat job failed via spot fleet request history and output logs. I would even leverage that "Campaign Manifest and Outputs" to link to individual instance output logs, cracked hashes files etc.... so its all in one place. See what works for you on your end.
  2. Got it
  3. Sounds good.
c6fc commented 2 years ago

Added in dev

bzekanovic commented 2 years ago

I did notice message below at first, but it did go away after I refreshed my browser page and nicer UI status metrics showed up.

image

c6fc commented 2 years ago

Ahh, I'd forgotten to remove a debug string. It's gone now though.

This is why dev exists :joy:

bzekanovic commented 2 years ago

I'll start testing this.

Also, it would not be bad idea to show end users how to run dev branch directly from CloudShell via source <(curl https://npkproject.io/cloudshell_install_dev.sh) in your docs. I had to dig it up in your prod script, but something to mention in docs.

bzekanovic commented 2 years ago

You can merge this one whenever you are ready and close. thx

c6fc commented 2 years ago

Merged