CodeFixthisList
Below is a table of FIXTHIS lines in the source code for tbwiki. This table is created using data_scan and table modules.
FIXTHIS lines in master branch [edit section]
| file ^ | line_no ^ | description ^ |
| Benchmark.bonnie/fuego_test.sh | 2 | upgrade bonnie++ to version 1.97.3 (or 1.04) |
| Benchmark.dd/fuego_test.sh | 10 | need to handle time as shell builtin case here |
| Benchmark.iperf3/fuego_test.sh | 14 | validate the server and client ips, and make sure they can communicate |
| Benchmark.iperf3/fuego_test.sh | 15 | add functionality to setup the server using a board file |
| Benchmark.iperf3/parser.py | 2 | support parsing streams |
| Benchmark.sysbench/fuego_test.sh | 3 | should check for libaio in test_pre_check |
| Benchmark.sysbench/fuego_test.sh | 18 | would be nice to support mysql and other tests as well |
| Benchmark.vuls/fuego_test.sh | 12 | support other distributions (e.g. CentOS requires yum-utils) |
| Functional.LTP/fuego_test.sh | 98 | do not skip tests by kernel version if the kernel is an LTS kernel |
| Functional.LTP/fuego_test.sh | 188 | use regex for selecting tests to skip once merged on LTP upstream |
| Functional.LTP/fuego_test.sh | 211 | grep tst_kvercmp on LTP to check for other tests that require specific versions |
| Functional.LTP/fuego_test.sh | 218 | only check the necessary ones |
| Functional.LTP/ltp_target_run.sh | 19 | add -t option for limiting the duration of each test group execution |
| Functional.LTP/parser.py | 166 | show errtype |
| Functional.LTP/parser.py | 167 | add sub-test data |
| Functional.aiostress/fuego_test.sh | 8 | this package-check is Debian-specific |
| Functional.fuego_check_tables/parser.py | 12 | could do better TAP processing here |
| Functional.fuego_compare_reports/fuego_test.sh | 43 | function override from fuego_test.sh didn't work |
| Functional.fuego_compare_reports/fuego_test.sh | 75 | should retrieve data based on batch_id instead of time?? |
| Functional.fuego_transport/fuego_test.sh | 40 | should try some file encodings that would be problematic for some transports |
| Functional.kernel_build/fuego_test.sh | 77 | add support for device trees |
| Functional.mesa_demos/fuego_test.sh | 1 | no test definition! |
| Functional.year2038/fuego_test.sh | 2 | the test program requires perl (it should be converted to C) |
| base/base-board.fuegoclass | 86 | ov_transport_get: 'ttc cp' doesn't support recursion |
| base/base-board.fuegoclass | 123 | ov_transport_put: 'ttc cp' doesn't support multiple sources |
| base/base-board.fuegoclass | 124 | ov_transport_put: 'ttc cp' doesn't support recursion (-r argument) |
| base/base-distrib.fuegoclass | 36 | it would be nice to remove "Name:" from grep in ov_rootfs_kill |
| flot/mod.js | 253 | remove tail? |
| flot/mod.js | 263 | make sure that test_set has no - in it |
| parser/common.py | 51 | remove this at version 1.6 |
| parser/common.py | 94 | use an ENV_ prefix and capital letters for all environment variables |
| parser/common.py | 584 | |
| parser/common.py | 662 | add JSON schema validation |
| parser/prepare_chart_data.py | 43 | could remove testname from flat_plot_data.txt |
| parser/prepare_chart_data.py | 111 | old chart_config files should be updated with proper chart_types and tguids. |
| parser/prepare_chart_data.py | 146 | declare more config info later (such as groupings, etc.) |
| parser/prepare_chart_data.py | 148 | could do some chart_config sanity checking in get_chart_config |
| parser/prepare_chart_data.py | 153 | read all run.json files and re-generate the flat_plot_data.txt file |
| parser/prepare_chart_data.py | 307 | put measures into board groups, if chart_config specifies so |
| parser/prepare_chart_data.py | 440 | use list of test sets in chart_config, if present |
| parser/prepare_chart_data.py | 512 | should read col_limit from chart_config |
| parser/prepare_chart_data.py | 636 | use list of boards in chart_config, if present |
| scripts/common.sh | 242 | fuego versions should get set at run time |
| scripts/common.sh | 255 | if the user started this test with a non-standard config |
| scripts/ftc | 697 | board_class should have methods to read board and dist file |
| scripts/ftc | 763 | remove deprecated routine get_run_data_from_buildxml() |
| scripts/ftc | 806 | parse causeBag from build.xml for run_class |
| scripts/ftc | 816 | in run_class, convert build.xml startTime from seconds to timestamp |
| scripts/ftc | 869 | run_class is not up-to-date with fuego-schema.json |
| scripts/ftc | 898 | hold warning for now - reenable when base script writes the run.json file |
| scripts/ftc | 908 | should load more instance data here |
| scripts/ftc | 919 | add timestamp to run.json file |
| scripts/ftc | 1092 | use glob |
| scripts/ftc | 1609 | have add-jobs support wildcards in the test name |
| scripts/ftc | 1728 | fix broken update of results output files |
| scripts/ftc | 1817 | not enough error handling (parameter checking) in do_add_view |
| scripts/ftc | 1832 | validate that job_name is valid in do_add_view |
| scripts/ftc | 2028 | should read host from run.json file in get_runs |
| scripts/ftc | 2059 | print value in nicer format |
| scripts/ftc | 2193 | add support for 'or' to 'where' processing |
| scripts/ftc | 2392 | use col_widths instead of just hardcoding 20 in gen_report |
| scripts/ftc | 2828 | support wildcard removal in do_rm_request |
| scripts/ftc | 2885 | run_request: should process Reboot, Rebuild and Pre/Post cleanup flags |
| scripts/ftc | 2889 | in run_request, optionally send run results to server |
| scripts/ftc | 2898 | in run_request, optionally remove the request from the server |
| scripts/ftc | 3067 | description and descriptionSetter data are for the success case only |
| scripts/ftc | 3173 | try graceful termination of test |
| scripts/ftc | 3202 | options for put_request should be same as run-test |
| scripts/ftc | 3287 | could consolidate do_put_test and do_put_run into do_put_item |
| scripts/ftc | 3445 | have run-test support wildcards in the test name |
| scripts/ftc | 3480 | eliminate these local variables, if not needed |
| scripts/ftc | 3534 | do_run_test: set job description |
| scripts/ftc | 3653 | use a more pythonic way |
| scripts/ftc | 3683 | what about "UNKNOWN" for the abort case, or something for |
| scripts/ftc | 3776 | if called as root, we may need to become root again before moving here |
| scripts/ftc | 3779 | do something to make the build/run appear in Jenkins |
| scripts/ftc | 3899 | don't handle overrides yet |
| scripts/ftc | 4012 | should report: |
| scripts/ftc | 4080 | add TTC to prompt |
| scripts/ftc | 4086 | should get user's preferred shell from /etc/passwd |
| scripts/ftc | 4147 | remove unused routine do_set_config() |
| scripts/ftc | 4184 | split_filepath doesn't handle ':' in file names or paths |
| scripts/ftc | 4367 | do more sanity checking of the package data here |
| scripts/ftc | 4422 | if upgrading, remove old test |
| scripts/ftc | 4424 | should delete old test first") |
| scripts/ftc | 4437 | should add config.xml for test to Jenkins ###" |
| scripts/ftc | 4438 | should now do some sanity checking of the installed |
| scripts/ftc | 4573 | we could get tricky here and try to find fuego.conf by |
| scripts/ftc | 4641 | remove the following hack for FUEGO_CORE location |
| scripts/functions.sh | 53 | add commit id information to the json output |
| scripts/functions.sh | 176 | Add descriptions for parameters in every function |
| scripts/functions.sh | 301 | locking here only prevents concurrency during the build phase |
| scripts/functions.sh | 609 | should review this order for failure evaluation |
| scripts/functions.sh | 846 | log_compare is not using $1 |
| scripts/functions.sh | 979 | race condition |
| scripts/gen-page.py | 27 | this doesn't handle multi-part testsets correctly |
| scripts/gen-page.py | 77 | run ftc or something to generate real data |
| scripts/main.sh | 58 | do not require board connection too early |
| scripts/make_cache.sh | 45 | need to mark this as NO_BOARD_CONTACT |
| scripts/mod.js | 102 | this line is completely bogus - need to convert from measurements to 'data' |
| scripts/mod.js | 103 | what is 'j'??? |
| scripts/need_check.sh | 197 | get_full_kconfig: if no config.gz, check /lib/modules for configs |
| scripts/need_check.sh | 241 | check kernel package build area? |
| scripts/need_check.sh | 272 | allow NEED_CONFIG=CONFIG_PRINTK (to be 'y' or 'm') |
| scripts/need_check.sh | 273 | allow NEED_CONFIG=CONFIG_SOMETHING=8 (allow ints and strings) |
| scripts/overlays.sh | 85 | automatically discover the best option |
| scripts/overlays.sh | 110 | add BUILD_TIMESTAMP and other variables to prolog.sh for --replay |
| scripts/ovgen.py | 175 | add case for single-line functions |
| scripts/ovgen.py | 435 | get fuego-core from env |
| scripts/ovgen.py | 445 | in Fuego 1.6, replace this with: |
FIXTHIS lines in next branch [edit section]
| file ^ | line_no ^ | priority ^ | description ^ |
| Benchmark.bonnie/fuego_test.sh | 2 | None | upgrade bonnie++ to version 1.97.3 (or 1.04) |
| Benchmark.dd/fuego_test.sh | 10 | None | need to handle time as shell builtin case here |
| Benchmark.iperf3/fuego_test.sh | 14 | None | validate the server and client ips, and make sure they can communicate |
| Benchmark.iperf3/fuego_test.sh | 15 | None | add functionality to setup the server using a board file |
| Benchmark.iperf3/parser.py | 2 | None | support parsing streams |
| Benchmark.sysbench/fuego_test.sh | 3 | None | should check for libaio in test_pre_check |
| Benchmark.sysbench/fuego_test.sh | 18 | None | would be nice to support mysql and other tests as well |
| Benchmark.vuls/fuego_test.sh | 12 | None | support other distributions (e.g. CentOS requires yum-utils) |
| Functional.LTP/fuego_test.sh | 98 | None | do not skip tests by kernel version if the kernel is an LTS kernel |
| Functional.LTP/fuego_test.sh | 188 | None | use regex for selecting tests to skip once merged on LTP upstream |
| Functional.LTP/fuego_test.sh | 211 | None | grep tst_kvercmp on LTP to check for other tests that require specific versions |
| Functional.LTP/fuego_test.sh | 218 | None | only check the necessary ones |
| Functional.LTP/ltp_target_run.sh | 19 | None | add -t option for limiting the duration of each test group execution |
| Functional.LTP/parser.py | 166 | None | show errtype |
| Functional.LTP/parser.py | 167 | None | add sub-test data |
| Functional.aiostress/fuego_test.sh | 8 | None | this package-check is Debian-specific |
| Functional.fuego_check_tables/parser.py | 12 | None | could do better TAP processing here |
| Functional.fuego_compare_reports/fuego_test.sh | 43 | None | function override from fuego_test.sh didn't work |
| Functional.fuego_compare_reports/fuego_test.sh | 75 | None | should retrieve data based on batch_id instead of time?? |
| Functional.fuego_transport/fuego_test.sh | 40 | None | should try some file encodings that would be problematic for some transports |
| Functional.kernel_build/fuego_test.sh | 77 | None | add support for device trees |
| Functional.mesa_demos/fuego_test.sh | 1 | None | no test definition! |
| Functional.year2038/fuego_test.sh | 2 | None | the test program requires perl (it should be converted to C) |
| base/base-board.fuegoclass | 86 | None | ov_transport_get: 'ttc cp' doesn't support recursion |
| base/base-board.fuegoclass | 123 | None | ov_transport_put: 'ttc cp' doesn't support multiple sources |
| base/base-board.fuegoclass | 124 | None | ov_transport_put: 'ttc cp' doesn't support recursion (-r argument) |
| base/base-distrib.fuegoclass | 36 | None | it would be nice to remove "Name:" from grep in ov_rootfs_kill |
| flot/mod.js | 253 | None | remove tail? |
| flot/mod.js | 263 | None | make sure that test_set has no - in it |
| parser/common.py | 51 | None | remove this at version 1.6 |
| parser/common.py | 94 | None | use an ENV_ prefix and capital letters for all environment variables |
| parser/common.py | 584 | None | |
| parser/common.py | 662 | None | add JSON schema validation |
| parser/prepare_chart_data.py | 43 | None | could remove testname from flat_plot_data.txt |
| parser/prepare_chart_data.py | 111 | None | old chart_config files should be updated with proper chart_types and tguids. |
| parser/prepare_chart_data.py | 146 | None | declare more config info later (such as groupings, etc.) |
| parser/prepare_chart_data.py | 148 | None | could do some chart_config sanity checking in get_chart_config |
| parser/prepare_chart_data.py | 153 | None | read all run.json files and re-generate the flat_plot_data.txt file |
| parser/prepare_chart_data.py | 307 | None | put measures into board groups, if chart_config specifies so |
| parser/prepare_chart_data.py | 440 | None | use list of test sets in chart_config, if present |
| parser/prepare_chart_data.py | 512 | None | should read col_limit from chart_config |
| parser/prepare_chart_data.py | 636 | None | use list of boards in chart_config, if present |
| scripts/common.sh | 242 | None | fuego versions should get set at run time |
| scripts/common.sh | 255 | None | if the user started this test with a non-standard config |
| scripts/ftc | 697 | None | board_class should have methods to read board and dist file |
| scripts/ftc | 763 | None | remove deprecated routine get_run_data_from_buildxml() |
| scripts/ftc | 806 | None | parse causeBag from build.xml for run_class |
| scripts/ftc | 816 | None | in run_class, convert build.xml startTime from seconds to timestamp |
| scripts/ftc | 869 | None | run_class is not up-to-date with fuego-schema.json |
| scripts/ftc | 898 | None | hold warning for now - reenable when base script writes the run.json file |
| scripts/ftc | 908 | None | should load more instance data here |
| scripts/ftc | 919 | None | add timestamp to run.json file |
| scripts/ftc | 1092 | None | use glob |
| scripts/ftc | 1609 | None | have add-jobs support wildcards in the test name |
| scripts/ftc | 1728 | None | fix broken update of results output files |
| scripts/ftc | 1817 | None | not enough error handling (parameter checking) in do_add_view |
| scripts/ftc | 1832 | None | validate that job_name is valid in do_add_view |
| scripts/ftc | 2028 | None | should read host from run.json file in get_runs |
| scripts/ftc | 2059 | None | print value in nicer format |
| scripts/ftc | 2193 | None | add support for 'or' to 'where' processing |
| scripts/ftc | 2392 | None | use col_widths instead of just hardcoding 20 in gen_report |
| scripts/ftc | 2828 | None | support wildcard removal in do_rm_request |
| scripts/ftc | 2885 | None | run_request: should process Reboot, Rebuild and Pre/Post cleanup flags |
| scripts/ftc | 2889 | None | in run_request, optionally send run results to server |
| scripts/ftc | 2898 | None | in run_request, optionally remove the request from the server |
| scripts/ftc | 3067 | None | description and descriptionSetter data are for the success case only |
| scripts/ftc | 3173 | None | try graceful termination of test |
| scripts/ftc | 3202 | None | options for put_request should be same as run-test |
| scripts/ftc | 3287 | None | could consolidate do_put_test and do_put_run into do_put_item |
| scripts/ftc | 3445 | None | have run-test support wildcards in the test name |
| scripts/ftc | 3480 | None | eliminate these local variables, if not needed |
| scripts/ftc | 3534 | None | do_run_test: set job description |
| scripts/ftc | 3653 | None | use a more pythonic way |
| scripts/ftc | 3683 | None | what about "UNKNOWN" for the abort case, or something for |
| scripts/ftc | 3776 | None | if called as root, we may need to become root again before moving here |
| scripts/ftc | 3779 | None | do something to make the build/run appear in Jenkins |
| scripts/ftc | 3899 | None | don't handle overrides yet |
| scripts/ftc | 4012 | None | should report: |
| scripts/ftc | 4080 | None | add TTC to prompt |
| scripts/ftc | 4086 | None | should get user's preferred shell from /etc/passwd |
| scripts/ftc | 4147 | None | remove unused routine do_set_config() |
| scripts/ftc | 4184 | None | split_filepath doesn't handle ':' in file names or paths |
| scripts/ftc | 4367 | None | do more sanity checking of the package data here |
| scripts/ftc | 4422 | None | if upgrading, remove old test |
| scripts/ftc | 4424 | None | should delete old test first") |
| scripts/ftc | 4437 | None | should add config.xml for test to Jenkins ###" |
| scripts/ftc | 4438 | None | should now do some sanity checking of the installed |
| scripts/ftc | 4573 | None | we could get tricky here and try to find fuego.conf by |
| scripts/ftc | 4641 | None | remove the following hack for FUEGO_CORE location |
| scripts/functions.sh | 53 | None | add commit id information to the json output |
| scripts/functions.sh | 176 | None | Add descriptions for parameters in every function |
| scripts/functions.sh | 301 | None | locking here only prevents concurrency during the build phase |
| scripts/functions.sh | 609 | None | should review this order for failure evaluation |
| scripts/functions.sh | 846 | None | log_compare is not using $1 |
| scripts/functions.sh | 979 | None | race condition |
| scripts/gen-page.py | 27 | None | this doesn't handle multi-part testsets correctly |
| scripts/gen-page.py | 77 | None | run ftc or something to generate real data |
| scripts/main.sh | 58 | None | do not require board connection too early |
| scripts/make_cache.sh | 45 | None | need to mark this as NO_BOARD_CONTACT |
| scripts/mod.js | 102 | None | this line is completely bogus - need to convert from measurements to 'data' |
| scripts/mod.js | 103 | None | what is 'j'??? |
| scripts/need_check.sh | 197 | None | get_full_kconfig: if no config.gz, check /lib/modules for configs |
| scripts/need_check.sh | 241 | None | check kernel package build area? |
| scripts/need_check.sh | 272 | None | allow NEED_CONFIG=CONFIG_PRINTK (to be 'y' or 'm') |
| scripts/need_check.sh | 273 | None | allow NEED_CONFIG=CONFIG_SOMETHING=8 (allow ints and strings) |
| scripts/overlays.sh | 85 | None | automatically discover the best option |
| scripts/overlays.sh | 110 | None | add BUILD_TIMESTAMP and other variables to prolog.sh for --replay |
| scripts/ovgen.py | 175 | None | add case for single-line functions |
| scripts/ovgen.py | 435 | None | get fuego-core from env |
| scripts/ovgen.py | 445 | None | in Fuego 1.6, replace this with: |