diff --git a/database/scripts/get_closed_known_issues.sql b/database/scripts/get_closed_known_issues.sql new file mode 100644 index 0000000..a4a428a --- /dev/null +++ b/database/scripts/get_closed_known_issues.sql @@ -0,0 +1,9 @@ +SELECT error_name, + github_issue +FROM test_fail_issues +WHERE status IN ( + "COMPLETED", + "OBLIVIATED", + "DISABLED" +) +GROUP BY github_issue; \ No newline at end of file diff --git a/database/scripts/lib/buildfarm_tools.rb b/database/scripts/lib/buildfarm_tools.rb index 015d64a..fa4a03e 100644 --- a/database/scripts/lib/buildfarm_tools.rb +++ b/database/scripts/lib/buildfarm_tools.rb @@ -27,7 +27,13 @@ def self.build_regressions_today(filter_known: false) def self.known_issues(status: '%%') # Default is all issues. This is not the best approach, but better than none # Keys: error_name, job_name, github_issue, status - run_command("./sql_run.sh get_known_issues.sql", args: [status.upcase]) + if status == 'open' + run_command("./sql_run.sh get_opened_known_issues.sql") + elsif status == 'closed' + run_command("./sql_run.sh get_closed_known_issues.sql") + else + run_command("./sql_run.sh get_known_issues.sql", args: [status.upcase]) + end end def self.error_appearances_in_job(test_name, job_name) @@ -41,7 +47,6 @@ def self.test_regressions_all(filter_known: false, include_reports: true) if filter_known known_errors = known_issues(status: 'open') known_errors.concat known_issues(status: 'disabled') - known_errors.concat known_issues(status: 'wontfix') known_error_names = Set.new(known_errors.map { |e| [e['error_name'], e['job_name']] }) out.filter! { |e| !known_error_names.include? [e['error_name'], e['job_name']] } end