Backfill StatusPage::PublishedIncidents and add /publish quick action
What does this MR do?
- Backfills table for
::StatusPage::PublishedIncident
with entries for all public issues for which status page is enabled & are not confidential (idempotent). This is a new feature with low usage, so this will be fairly few rows. - Updates
PublishService
to determine whether to publish an issue based on whether an entry in::StatusPage::PublishedIncident
exists (was & still is idempotent) - Adds a
/publish
quick action which only appears for developers+ on non-published issues. - Not changed:
confidential
issues still cannot be published.
Related issue: #209849 (closed)
MRs for full feature:
-
!29994 (merged) - Adds
status_page_published_incidents
table - !30880 (merged) - Adds helpers for system note create for publish events
-
!30893 (merged) - Adds helpers for creating/deleting
::StatusPage::PublishedIncident
s -
!31380 (merged) - Rename
include_confidential
option -
!31381 (merged) - Move
PublishService
publish vs unpublish check into reusable method - !31395 (merged) - Add flow to publish an issue directly
- !31480 (merged) - Documentation
Migration output:
$ rake db:migrate
== 20200421195234 BackfillStatusPagePublishedIncidents: migrating =============
== 20200421195234 BackfillStatusPagePublishedIncidents: migrated (0.0630s) ====
$ rake db:migrate:down VERSION=20200421195234
== 20200421195234 BackfillStatusPagePublishedIncidents: reverting =============
== 20200421195234 BackfillStatusPagePublishedIncidents: reverted (0.0000s) ====
Migration query execution plan:
SELECT * FROM issues
INNER JOIN status_page_settings
ON status_page_settings.project_id = issues.project_id
WHERE status_page_settings.enabled = true
AND issues.confidential = false
Nested Loop (cost=0.56..263.10 rows=137 width=3572) (actual time=9.835..24.239 rows=6 loops=1)
Buffers: shared hit=5 read=13
I/O Timings: read=24.054
-> Seq Scan on public.status_page_settings (cost=0.00..4.02 rows=1 width=2265) (actual time=0.889..0.891 rows=2 loops=1)
Filter: status_page_settings.enabled
Rows Removed by Filter: 0
Buffers: shared read=1
I/O Timings: read=0.878
-> Index Scan using index_issues_on_project_id_and_iid on public.issues (cost=0.56..257.71 rows=137 width=1307) (actual time=7.107..11.666 rows=3 loops=2)
Index Cond: (issues.project_id = status_page_settings.project_id)
Filter: (NOT issues.confidential)
Rows Removed by Filter: 0
Buffers: shared hit=5 read=12
I/O Timings: read=23.176
Time: 24.774 ms
- planning: 0.476 ms
- execution: 24.298 ms
- I/O read: 24.054 ms
- I/O write: 0.000 ms
Shared buffers:
- hits: 5 (~40.00 KiB) from the buffer pool
- reads: 13 (~104.00 KiB) from the OS file cache, including disk I/O
- dirtied: 0
- writes: 0
Screenshots
On issue creation: | Upon adding a note: | Failure message (not reproducible outside of specs; only unforeseen issues): |
---|---|---|
Does this MR meet the acceptance criteria?
Conformity
-
Changelog entry - [-] Documentation (if required) - follow-up !31480 (merged)
-
Code review guidelines -
Merge request performance guidelines -
Style guides - [-] Database guides
-
Separation of EE specific content
Availability and Testing
-
Review and add/update tests for this feature/bug. Consider all test levels. See the Test Planning Process. - [-] Tested in all supported browsers
- [-] Informed Infrastructure department of a default or new setting change, if applicable per definition of done
Security
If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:
- [-] Label as security and @ mention
@gitlab-com/gl-security/appsec
- [-] The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
- [-] Security reports checked/validated by a reviewer from the AppSec team
Edited by Peter Leitzen