- 10 Aug, 2012 1 commit
-
-
ssh://git.bind10.isc.org/var/bind10/git/bind10Naoki Kambe authored
Conflicts: ChangeLog src/lib/config/tests/testdata/spec40.spec
-
- 09 Aug, 2012 11 commits
-
-
Naoki Kambe authored
-
Naoki Kambe authored
-
Naoki Kambe authored
If a target module returns data whose key name is such a identifier type, the stats module can identify statistics data. The stats module checks the validation of data even if such a identifier is specified. But when the stats shows statistics data e.g. via bindctl, this type of identifier is not shown for now.
-
Naoki Kambe authored
A named_set data type is supported in a spec file in each module for statistics.
-
Naoki Kambe authored
Stats updates corresponding statistics data which each module returns. For that, the method "update_statistics_data()" was updated to merge recursively old value and new value each other. As for this merging, a new method "merge_oldnew" was introduced into stats.py. This method is intended to be internally used so far.
-
Naoki Kambe authored
When the set command was used for collecting statistics data, the action was randomly. But in the case that periodical polling is used for collecting statistics data, update_modules doesn't need to be invoked so many times every time update_statistics_data is invoked. Invoking update_module() causes an IO wait for receiving statistics spec from Cfgmgr.
-
Naoki Kambe authored
Not self.stats.mccs.specification but self.stats.modules['Stats'] should be set the invalid spec_module into. And it should check whether StatsError is raised by setting the owner to the self module name in the argument of command_show.
-
Naoki Kambe authored
Because do_polling() is done when the show command is invoked, the do_polling method didn't need to be done when the stats is starting. Related to this change, the b10-stats_test.py was changed. do_polling() is inserted before the assertion of statistics_data.
-
Naoki Kambe authored
-
Naoki Kambe authored
-
Naoki Kambe authored
- Other conditions are considered. One case is that two args are string type, another case is that either of them is None type. - _accum() was moved to outside of the Stats class for testing it individually. - revised the FIXME description in update_statistics_data() according to the above changes
-
- 08 Aug, 2012 28 commits
-
-
Mukund Sivaraman authored
-
Mukund Sivaraman authored
-
Naoki Kambe authored
- moved b10-config_test.db from src/bin/stats/tests to src/bin/stats/tests/testdata - renamed its name to b10-config.db - changed its format to multiline These changes are due to the reviewer's comment.
-
Mukund Sivaraman authored
-
Mukund Sivaraman authored
-
Mukund Sivaraman authored
-
Mukund Sivaraman authored
-
Mukund Sivaraman authored
-
Mukund Sivaraman authored
-
Mukund Sivaraman authored
-
Mukund Sivaraman authored
-
Mukund Sivaraman authored
-
Naoki Kambe authored
A list of instances for stats to request is obtained from the show_processes command of Boss instead of asking the components config of Boss.
-
Naoki Kambe authored
This information is used for other module e.g. stats to count running instances of same module. The address information is seen in the third index of the array.
-
Naoki Kambe authored
It returns the "_address" attribute of the classes. This is used for the "show_processes" command of Boss to provide address information.
-
Naoki Kambe authored
The stats module decides if polling should be done by the the last time of polling. If more than one seconds past since the last request to each module, the stats module requests each module statistics data and then shows the latest result. Otherwise, the stats module just shows statistics data which it has.
-
Naoki Kambe authored
In the updating process, it communicates with the cfgmgr module for updating statistics specification of each module. This fix is for separating such mixed communication process.
-
Naoki Kambe authored
- Fixed the incorrect previous fix. There should not be the inconsistency on number of mock Auth instances on both test scripts: b10-stats_test.py and b10-stats-httpd_test.py. In the previous change, there was the inconsistency in MockBoss class. Actually in b10-stats-httpd_test.py, the number of auth instances which Cfgmgr answers was four, but the number of actually invoked auth instances was one. - Reduce the number of invoked mock Auth instances In theses test scripts, multiple Auth instances need to be invoked. But if four instances are actually invoked in these scripts, open files(ulimit -n) exceeds 256. In the environment where ulimit -n was set to 256, the b10-stats-httpd_test.py test failed actually. In that case, b10-stats-httpd_test.py needs open files more than 256. So the number of the auth instances in mock modules was reduced into 2 in the scripts. This fix is for the comment at #2136.
-
Naoki Kambe authored
The extra MockAuth objects didn't shut down in BaseModules.shutdown(). But these weren't needed for unittest of stats httpd. So setting up and shutting up of them were moved to b10-stats_test.py from test_utils.py.
-
JINMEI Tatuya authored
-
Naoki Kambe authored
-
Naoki Kambe authored
-
Naoki Kambe authored
integer
-
Naoki Kambe authored
polling
-
Naoki Kambe authored
-
Naoki Kambe authored
-
Naoki Kambe authored
do_polling for efficiency Also added setting timeout value to one second for precise and refactoring timeout handling
-
Naoki Kambe authored
module
-