axfelix
3 Mayo, 2022 15:34
1
Falló una reconstrucción automática, parece que hay variaciones en este problema:
2022-05-03 15:30:54.370 UTC [1262] discourse@discourse LOG: duration: 1707.909 ms statement: INSERT INTO post_hotlinked_media (post_id, url, status, upload_id, created_at, updated_at)
SELECT
post_id,
obj.key AS url,
'downloaded',
obj.value::bigint AS upload_id,
pcf.created_at,
pcf.updated_at
FROM post_custom_fields pcf
JOIN json_each_text(pcf.value::json) obj ON true
JOIN uploads ON obj.value::bigint = uploads.id
WHERE name='downloaded_images'
ERROR: duplicate key value violates unique constraint "index_post_hotlinked_media_on_post_id_and_url_md5"
Había reconstruido hace menos de un mes sin errores, así que lo que sea que introdujo esto parece haber sido reciente.
4 Me gusta
axfelix
3 Mayo, 2022 16:09
3
¿parece tener algo que ver con esta confirmación reciente?
# frozen_string_literal: true
class CreatePostHotlinkedMedia < ActiveRecord::Migration[6.1]
def change
reversible do |dir|
dir.up do
execute <<~SQL
CREATE TYPE hotlinked_media_status AS ENUM('downloaded', 'too_large', 'download_failed', 'upload_create_failed')
SQL
end
dir.down do
execute <<~SQL
DROP TYPE hotlinked_media_status
SQL
end
end
create_table :post_hotlinked_media do |t|
t.bigint :post_id, null: false
t.string :url, null: false
show original
@david
2 Me gusta
david
3 Mayo, 2022 16:15
4
Sí, eso es todo. Tengo una solución en proceso, debería fusionarse en los próximos minutos. Gracias por informar @axfelix
main ← hotlinked-conflict
opened 03:54PM - 03 May 22 UTC
In the old custom_field-based system, it was possible for a url to be both 'down… loaded' and 'broken'. The new table enforces uniqueness, so we need to drop invalid data.
9 Me gusta
axfelix
3 Mayo, 2022 16:39
5
¡Gracias, eso fue rápido!
¿Necesito hacer algo especial para descargar esto? Todavía obtengo Causado por: ActiveRecord::RecordNotUnique: PG::UniqueViolation: ERROR: la clave duplicada viola la restricción única «index_post_hotlinked_media_on_post_id_and_url_md5» al ejecutar una reconstrucción del lanzador, incluso después de hacer un Docker prune.
david
3 Mayo, 2022 16:48
6
La corrección acaba de pasar las pruebas hace unos segundos. Si reconstruyes ahora, debería funcionar mucho mejor. Avísanos cómo va.
5 Me gusta
david
3 Mayo, 2022 18:07
7
Encontramos un lugar más que podría causar este error. Así que si la primera solución no ayudó, esta debería hacerlo
committed 05:51PM - 03 May 22 UTC
custom_field data on some sites has duplicate values for a given url key in the … JSON value. This is invalid, so we should drop the extra data.
(ahora activo en tests-passed)
4 Me gusta
pfaffman
3 Mayo, 2022 19:31
8
En mi instancia de desarrollo, migra la base de datos correctamente, pero si restauro una copia de seguridad de 2022-04-27-153103-v20220407195246.tar.gz, falla:
[361/9020]
Migrating the database... EXCEPTION: /home/pfaffman/src/discourse-repos/discourse/lib/discourse.rb:126:in `exec': Failed to migrat
e database.
Ignoring /home/pfaffman/src/discourse-repos/discourse/db/schema_cache.yml because it has expired. The cu
rrent schema version is 20220407195246, but the one in the schema cache file is 20220428094027.
rake aborted!
StandardError: An error has occurred, this and all later migrations canceled:
seguido de más de eso con algunos de estos:
ActiveRecord::StatementInvalid: PG::DuplicateObject: ERROR: type \"hotlinked_media_status\" already exist
1 me gusta
axfelix
3 Mayo, 2022 21:22
9
¡Solo para informar que la primera corrección funcionó para nosotros después de pasar las pruebas!
2 Me gusta
david
3 Mayo, 2022 21:30
10
¡Genial, gracias por confirmar!
Abrí una PR con una corrección para esa:
main ← fix-restore-enums
opened 09:27PM - 03 May 22 UTC
c1db9687 introduced an postgres enum type. Our database restore logic did not ha… ndle custom types correctly, and would therefore raise a 'type already exists' error when restoring any backup.
This commit adds restore handling for enums, mirroring the similar logic for tables and views.
---
(This code is not tested by rspec, but it is tested by a regular internal job at CDCK, which correctly picked up on this issue)
4 Me gusta
pfaffman
4 Mayo, 2022 14:05
11
¡Terminado!
[ÉXITO]
Restauración completada.
¡Gracias!
2 Me gusta
