Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Report generation timeout #1640

Merged
merged 2 commits into from
Aug 21, 2023
Merged

Report generation timeout #1640

merged 2 commits into from
Aug 21, 2023

Conversation

ammar92
Copy link
Contributor

@ammar92 ammar92 commented Aug 21, 2023

Changes

Implements KEIKO_REPORT_TIMEOUT configuration for Rocky

Issue link

Closes #1614


Code Checklist

  • All the commits in this PR are properly PGP-signed and verified;
  • This PR only contains functionality relevant to the issue; tickets have been created for newly discovered issues.
  • I have written unit tests for the changes or fixes I made.
  • For any non-trivial functionality, I have added integration and/or end-to-end tests.
  • I have performed a self-review of my code and refactored it to the best of my abilities.

Communication

  • I have informed others of any required .env changes files if required and changed the .env-dist accordingly.
  • I have made corresponding changes to the documentation, if necessary.

Checklist for code reviewers:

Copy-paste the checklist from the docs/source/templates folder into your comment.


Checklist for QA:

Copy-paste the checklist from the docs/source/templates folder into your comment.

@ammar92 ammar92 marked this pull request as ready for review August 21, 2023 10:29
@ammar92 ammar92 requested a review from a team as a code owner August 21, 2023 10:29
@ammar92 ammar92 changed the title Feature/report generation timeout Report generation timeout Aug 21, 2023
@dekkers dekkers merged commit b66e04e into main Aug 21, 2023
18 checks passed
@dekkers dekkers deleted the feature/report-generation-timeout branch August 21, 2023 12:23
dekkers pushed a commit that referenced this pull request Aug 21, 2023
jpbruinsslot added a commit that referenced this pull request Aug 22, 2023
* main:
  Use 127.0.0.1 for RabbitMQ in install script (#1644)
  Remove environment variables from container docs (#1645)
  Feature/report generation timeout (#1640)
  Add reverse DNS boefje (#1579)
  Add first version of new normalisers runner design (#1538)
  Fix `poetry-dependencies` target in Makefile (#1627)
  Upgrade OpenTelemetry (#1626)
  Remove finding types from rocky/OOI_database_seed.json (#1619)
  Feature: Add task detail pages and show objects yielded by normalizer (#1506)
  Update django-admin-auto-tests (#1617)
  Update GitHub Actions (#1618)
  Updated cryptography (#1615)
  Improve filter by muted findings on findings page (#1595)
  Redteamer can now acknowledge clearance level during onboarding (#1549)
  Do not add line information in `.po` files (#1616)
  Add TLS Cipher checks (#1381)
jpbruinsslot added a commit that referenced this pull request Aug 24, 2023
* main: (27 commits)
  Update install script with new env vars (#1655)
  Fix pre-commit exclude (#1660)
  Add mention to Weblate to the README (#1653)
  Fix reschedule button (#1648)
  Add new boefjes runner design (#1620)
  Pin poetry in the CI and fix deprecation warning (#1654)
  Get current language and show in html lang attribute (#1625)
  Fix translations in main  (#1649)
  Add 1.11 release notes (#1646)
  Automatically re-create queue if durable changed (#1637)
  Fix configuration issues upgrading Debian installs (#1636)
  Use 127.0.0.1 for RabbitMQ in install script (#1644)
  Remove environment variables from container docs (#1645)
  Feature/report generation timeout (#1640)
  Add reverse DNS boefje (#1579)
  Add first version of new normalisers runner design (#1538)
  Fix `poetry-dependencies` target in Makefile (#1627)
  Upgrade OpenTelemetry (#1626)
  Remove finding types from rocky/OOI_database_seed.json (#1619)
  Feature: Add task detail pages and show objects yielded by normalizer (#1506)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make report generation timeout configurable
2 participants