admin: details of stratums instances per algo #228

phm87 · 2018-02-10T22:02:24Z

Thank you for the help.
Layout of display is still to enhance

Update to keep up to date

if needed (already mining) you can do that to update the pool estimate graph values: UPDATE hashrate SET price=price*1000.0, rent=rent*1000 WHERE algo='lbry' AND price < 0.01;

Implemented PROXY v2 protocol (https://www.haproxy.org/download/1.8/doc/proxy-protocol.txt) so that stratum can be run behind an haproxy instance. iptables/ipset still work if haproxy is run on local machine. if run on remote machine switch to using client_block_ipset() and periodically sync the ipset to the machine running haproxy. haproxy must be configured with send-proxy-v2, version 1 does not work

Avoid the override of global variables g_stratum_max_cons, g_max_shares, g_stratum_segwit by excluded coinds by checking the include/exclude filters earlier.

beware: this algo may have a variable hashrate, unlike timetravel its ignored from benchs for this reason

to do like bitstamp...

Greetings, I am head dev for Magnet masternode and noticed the change made in coinbase.cpp for recent masternode RPC as default. This change has disabled masternode payments on pools using MAG client. So, we would like to add it to oldmasternodes list with this pull request.

and restore bitstamp & cexio behavior to act like other trading exchanges todo: kraken if possible... real pain, need to parse open orders..

Algo yescryptR16 for Yenten [YTN] https://bitcointalk.org/?topic=2329470.0 stratum changes from https://github.com/bubasik/yiimp

wallets without getinfo will be handled in a later commit... ps: remember to restart memcached service after the db change..

manual alignment with master 5468b84#diff-bee428e5bcff8c748847aa3bf76a3d8d

align with master

url stratum-g_tcp_server

tpruvot · 2018-02-11T08:05:12Z

web/yaamp/modules/site/common_results.php

+                $urls = $urls . "\n" . $stra['url'];
+        }
+        echo '<td style="background-color: '.$algo_color.'"><b title="'.$urls.'">';
+	echo CHtml::link($algo, '/site/Gostratums?algo='.$algo);


This cap should be removed here.. like "gomining"... the actionGostratum cap is only a way to make action functions more readable.

tpruvot · 2018-02-11T08:06:53Z

web/yaamp/modules/site/common_results.php

+
+	$stratum_instances = dboscalar("SELECT COUNT(*) FROM stratums WHERE algo='$algo'");
+        $stratum_urls = dbolist("SELECT url FROM stratums WHERE algo='$algo'"); // AND NOT(url = NULL)");
+


indentation spaces to fix (please keep tabs)

@tpruvot

Thank you for you help @tpruvot - replace tabs by spaces - Remove uppercase G in Gostratums - HTML enhancement: stratumstable Remark: If anything else should be enhanced, please tell me. I'll conduct tests and write a comment if tests are fine (ex: textExtraction fields of stratumstable).

tpruvot · 2018-02-11T09:08:35Z

web/yaamp/modules/site/common_results.php

+		</thead>
+		<tbody>
+		END;
+		$stratums_details_list = dbolist("SELECT * FROM stratums WHERE algo:=algo='$algo_selected'");


algo:=algo='...' ?

Change to " to ' on th align

because single quotes were used and not double quotes

tpruvot · 2018-02-11T09:12:17Z

web/yaamp/modules/site/common_results.php

+		</tr>
+		</thead>
+		<tbody>
+		END;
 		$stratums_details_list = dbolist("SELECT * FROM stratums WHERE algo:=algo='$algo_selected'");
 		foreach ($stratums_details_list as $stratums_details)	{
 			echo "<tr>";


Yep, you need to fill data="$time" in the time td... and use the datetoa2() function for the content

tpruvot · 2018-02-11T09:13:25Z

web/yaamp/modules/site/common_results.php

+		<th data-sorter='numeric' align='left">Started</th>
+		<th data-sorter='numeric' align='left">Workers</th>
+		<th data-sorter='numeric' align='left">Port</th>
+		<th data-sorter='numeric' align='left">Symbol</th>


symbol and "url" are not numeric...

tpruvot · 2018-02-11T09:13:43Z

web/yaamp/modules/site/common_results.php

+			widgets: ["Storage","saveSort"],
+			textExtraction: {
+				1: function(node, table, cellIndex) { return $(node).attr("data"); },
+				5: function(node, table, cellIndex) { return $(node).attr("data"); }


5th column doesnt need that

Thank you again for the review :-) - Remove :=algo in SQL - Remove 5 in stratumstable

tpruvot · 2018-02-11T09:52:56Z

web/yaamp/modules/site/common_results.php

@@ -195,7 +195,6 @@ function cmp($a, $b)
 			widgets: ["Storage","saveSort"],
 			textExtraction: {
 				1: function(node, table, cellIndex) { return $(node).attr("data"); },


you need to remove the comma, some browsers dont like that ;)

tpruvot · 2018-02-11T09:54:39Z

web/yaamp/modules/site/common_results.php

+		<th data-sorter='numeric' align='left'>Port</th>
+		<th data-sorter='numeric' align='left'>Symbol</th>
+		<th data-sorter='numeric' align='left'>url</th>
+		<th data-sorter='numeric' align='left'>fds</th>


html single quotes are not standard use... and cant work echo '<td colspan='13'>';

Thank you again for your help and support during the weekend. - Double quotes instead of single quotes - data-sorter : text instead of numeric for url and Symbol - fill data for time and started column I've made the changes discussed, I'll test on a pooltest and write a comment about tests results later.

tpruvot · 2018-02-11T13:39:18Z

web/yaamp/modules/site/SiteController.php

+		if(!$this->admin) return;
+		$algo = substr(getparam('algo'), 0, 32);
+		if ($algo == 'all') {
+			return;


no need to prevent "all" if its for the admin

tpruvot · 2018-02-11T13:43:01Z

web/yaamp/modules/site/common_results.php

+				saveSort: true
+			}}');
+		echo <<<END
+		<table>


showTableSorter() php func creates a <table class="dataGrid">... this one is double/useless so

As suggested by Epsy, no need to prevent "all" if its for the admin

Thank you Epsy

meant to detect socket leaks before its too late...

phm87 and others added 27 commits January 7, 2018 01:31

GoaCoin stratum support

900e5fa

Update coinbase.cpp

bc7fa41

Merge pull request #1 from tpruvot/next

815c955

Update to keep up to date

stratum: initialize segwit flag

cdd72bf

lbry: change estimates in mBTC/GH like other sha algos

7f7d533

if needed (already mining) you can do that to update the pool estimate graph values: UPDATE hashrate SET price=price*1000.0, rent=rent*1000 WHERE algo='lbry' AND price < 0.01;

backend: drop old blocks from the db after 60 days

e0e7c14

stratum: pushing more old masternodes (#210)

8509aae

decred: listtransactions order is finally like others

a3a3ba8

stratum: check coin filters early (#211)

762fbdb

Avoid the override of global variables g_stratum_max_cons, g_max_shares, g_stratum_segwit by excluded coinds by checking the include/exclude filters earlier.

new x16r algo, with stratum factor 256, like timetravel algos

2d4348e

beware: this algo may have a variable hashrate, unlike timetravel its ignored from benchs for this reason

benchs: ignore some weird device names and x16r algo

885b448

stratum: fix altminer commit

461f788

cexio: include btc in orders for the balance

ac42aab

to do like bitstamp...

balances: fill the btc 'onsell' field too (locked)

95aa7fc

and restore bitstamp & cexio behavior to act like other trading exchanges todo: kraken if possible... real pain, need to parse open orders..

stratum: avoid the use of getinfo

f022346

yescryptR16 algo (#217)

5c5abe5

Algo yescryptR16 for Yenten [YTN] https://bitcointalk.org/?topic=2329470.0 stratum changes from https://github.com/bubasik/yiimp

sql: add hasgetinfo and no_explorer fields to coins table

73ac94f

wallets without getinfo will be handled in a later commit... ps: remember to restart memcached service after the db change..

Fill url in stratum table with g_tcp_server

5d464ff

Display g_tcp_server of stratums (url column)

0af4987

stratum: set PBS as old masternode #220

682a293

manual alignment with master 5468b84#diff-bee428e5bcff8c748847aa3bf76a3d8d

Merge pull request #5 from tpruvot/next

e2ad0b5

align with master

Merge pull request #6 from phm87/phm87-url-stratum-g_tcp_server

18104f2

url stratum-g_tcp_server

actionGostratums & actionAdminStratums

e451b12

Update SiteController.php

a4a3ac9

Update common_results.php

2f591bc

phm87 mentioned this pull request Feb 10, 2018

url stratum - g_tcp_server #226

Closed

tpruvot reviewed Feb 11, 2018

View reviewed changes

single quotes

0169fa3

because single quotes were used and not double quotes

tpruvot reviewed Feb 11, 2018

View reviewed changes

single quote

19e7933

tpruvot reviewed Feb 11, 2018

View reviewed changes

SQL fix + stratumstable fix

594dda0

Thank you again for the review :-) - Remove :=algo in SQL - Remove 5 in stratumstable

tpruvot reviewed Feb 11, 2018

View reviewed changes

phm87 added 3 commits February 11, 2018 16:34

admin: no need to prevent all

eb56e26

As suggested by Epsy, no need to prevent "all" if its for the admin

Remove double/useless <table>

7f6bf1f

Thank you Epsy

display stratums instances when algo_selected=all

d4bbbb9

phm87 referenced this pull request Mar 1, 2018

stratum: fill fds and server name/url in stratums table

befb105

meant to detect socket leaks before its too late...

tpruvot force-pushed the next branch from a2b8301 to 669ab6b Compare March 12, 2018 18:01

tpruvot added the not proper label Mar 23, 2018

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

admin: details of stratums instances per algo #228

admin: details of stratums instances per algo #228

phm87 commented Feb 10, 2018

tpruvot Feb 11, 2018

tpruvot Feb 11, 2018

tpruvot Feb 11, 2018

mman07 Apr 12, 2018 •

edited

Loading

tpruvot Feb 11, 2018

tpruvot Feb 11, 2018

tpruvot Feb 11, 2018

tpruvot Feb 11, 2018

tpruvot Feb 11, 2018 •

edited

Loading

tpruvot Feb 11, 2018

tpruvot Feb 11, 2018


		$stratum_instances = dboscalar("SELECT COUNT(*) FROM stratums WHERE algo='$algo'");
		$stratum_urls = dbolist("SELECT url FROM stratums WHERE algo='$algo'"); // AND NOT(url = NULL)");

admin: details of stratums instances per algo #228

Are you sure you want to change the base?

admin: details of stratums instances per algo #228

Conversation

phm87 commented Feb 10, 2018

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

mman07 Apr 12, 2018 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

tpruvot Feb 11, 2018 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

mman07 Apr 12, 2018 •

edited

Loading

tpruvot Feb 11, 2018 •

edited

Loading