Project

General

Profile

Enhancement #1551

Opdater eksporteret konfiguration

Added by Kasper Garnæs over 3 years ago. Updated about 1 year ago.

Status:
Ready for development
Priority:
Urgent
Target version:
Estimated time:
URL med eksempel:
Kategorier:
Driftsvedligehold - Refaktorering (Opdatering af kodebasen)

Description

Martin Cording, Inlead har gjort opmærksom på at modulopdateringer har medført at vores konfiguration skal opdateres, så den er opdateret tilsvarende. Dermed bør pull requests med ændringer ad konfiguration fremadrettet kun indeholde de relevante ændringer.

Dette bør gøres i forbindelse med næste release.

History

#2 Updated by Kasper Garnæs over 3 years ago

Det er vigtigt at bemærke at denne opgave bør laves på en vanilla installation af DDB CMS - ikke ud fra et kørende produktionssite.

#3 Updated by Rolf Madsen over 3 years ago

  • Status changed from New to Ready for development
  • Priority changed from Normal to Urgent
  • Target version set to DDB CMS 2016 1. opgradering

#4 Updated by Martin Cording over 3 years ago

Og den burde laves umiddelbart efter næste release :)

#5 Updated by Rolf Madsen over 3 years ago

  • Target version changed from DDB CMS 2016 1. opgradering to 42

#6 Updated by Rolf Madsen over 3 years ago

  • Assignee changed from Rolf Madsen to Laura Holm

#7 Updated by Rolf Madsen over 3 years ago

  • Assignee deleted (Laura Holm)

#8 Updated by Rolf Madsen over 3 years ago

  • Target version changed from 42 to DDB CMS 2016 1. opgradering

#9 Updated by Rolf Madsen over 3 years ago

  • Target version changed from DDB CMS 2016 1. opgradering to DDB CMS 2016 2. opgradering (Inlead sprintbacklog)

#10 Updated by Martin Cording over 3 years ago

@Rolf: Denne opgave burde ligge "udenfor" de oprindelige udviklingsrunder og laves lige efter 2.4.x (alternativt 2.5.x) er released.

#11 Updated by Martin Cording over 3 years ago

  • Target version changed from DDB CMS 2016 2. opgradering (Inlead sprintbacklog) to DDB CMS 2016 2. opgradering

#12 Updated by Rolf Madsen over 3 years ago

  • Target version changed from DDB CMS 2016 2. opgradering to DDB CMS 2016 2. opgradering (DBC sprintbacklog)

#13 Updated by Rolf Madsen over 3 years ago

  • Kategorier Driftsvedligehold - Refaktorering (Opdatering af kodebasen) added

#14 Updated by per johansen over 3 years ago

  • Status changed from Ready for development to Development
  • Assignee set to per johansen

planen er helt bevidstløst at genskabe features fra en frisk installation.

#15 Updated by per johansen over 3 years ago

  • Status changed from Development to Needs code review
  • Assignee changed from per johansen to Kasper Garnæs

Når i siger konfiguration går jeg udfra at i mener indstillinger i features. Hvis jeg kigger på et vanilla-site (https://vanilla-alma.ddbcms.dk/) er disse features "tilsidesat";
bpi
ding_content
ding_frontend
ding_groups

ding_content er tilsidesat pga den hook der bruges i ding_content/ding_content.file_default_display.inc se fx dether issue:https://www.drupal.org/node/2104193, der er en patch til features, der gør at den ikke vises som "tilsidesat", men den anbefalede løsning (som er brugt i Media modulet) er at flytte konfigurationen til install

ændringerne i ding_frontend og ding_groups er små og ligegyldige

summa summarum har jeg lavet et pull-request på ændringerne i bpi
https://github.com/ding2/ding2/pull/205

#16 Updated by Kasper Garnæs about 3 years ago

  • Status changed from Needs code review to Reviewed - Needs info/rework
  • Assignee changed from Kasper Garnæs to per johansen

Pull requestet indholder ingen kodeændringer pt. så der er intet at reviewe.

Generelt kan jeg ikke få det til at stemme at Martin Cording melder at pull requests med konfigurationsændringer indholder irrelevante ændringer pga. modulopdateringer, mens der ifølge Per nærmest ikke er ændringer.

> ding_content er tilsidesat pga den hook der bruges i ding_content/ding_content.file_default_display.inc se fx dether issue:https://www.drupal.org/node/2104193, der er en patch til features, der gør at den ikke vises som "tilsidesat", men den anbefalede løsning (som er brugt i Media modulet) er at flytte konfigurationen til install

Burde vi i så fald ikke flytte vores konfiguration til install også?

> ændringerne i ding_frontend og ding_groups er små og ligegyldige

Hvilke ændringer er der tale om? Generelt handler dette issue jo om at undgå at features er unødvendigt overriden, så derfor tænker jeg som udgangspunkt at vi skal have disse ændringer med hvor små de så end måtte være.

#17 Updated by per johansen about 3 years ago

  • Assignee changed from per johansen to Martin Cording

Måske Martin kan være mere specifik på hvad der er for ændringer der er tale om - som sagt kan jeg ikke rigtig finde nogen

#18 Updated by Martin Cording about 3 years ago

  • Assignee changed from Martin Cording to per johansen

Jeg har ikke lige noget konkret vedr. dette, men det gælder primært Views og Panels som er eksporteret - hvorefter versionen af disse moduler er opdateret.
Efter opdatering af contrib modulet, er der ændrede måder at strukturere den eksporterede kode på.

Det vil altså sige at laver man en lille ændring til f.eks. et View, så er diff'en kæmpe stor fordi at det er ønsket at man anvender den nye/opdaterede formatering/struktur.
Du kunne grep'e efter features i kodebasen, eksportere alt hvad der ligger og se diffen.

#19 Updated by Rolf Madsen almost 3 years ago

  • Assignee changed from per johansen to Laura Holm

#20 Updated by per johansen almost 3 years ago

  • Status changed from Reviewed - Needs info/rework to Need more info
  • Assignee changed from Laura Holm to Martin Cording

jamen altså -

Ville det ikke være bedre at tage dem som de kommer.

Hvis der bliver lavet en ændring i én feature må core jo reviewe den éne ændring - Istedet for som du foreslår, hvor diffen potentielt bliver gigantisk og HELT umulig at reviewe på fornuftig vis ...

og hvad hvis der opstår fejl - hvordan ville vi kunne debugge den gigantiske diff - det ville være nemmere med én ad gangen, også hvis vi evt. skal rulle tilbage.

#21 Updated by Martin Cording almost 3 years ago

  • Assignee changed from Martin Cording to Kasper Garnæs

#22 Updated by Gitte Barlach over 2 years ago

Kan denne vente til næste release?

#23 Updated by Kasper Garnæs over 2 years ago

  • Status changed from Need more info to Needs analysis
  • Assignee changed from Kasper Garnæs to Gitte Barlach

Jo. Det må den.

#24 Updated by Kasper Garnæs over 2 years ago

  • Target version changed from DDB CMS 2016 2. opgradering (DBC sprintbacklog) to DDB CMS 2017 1. opgradering (7.x-4.0.2)

#25 Updated by Kasper Garnæs about 2 years ago

Jeg har netop kørt features update og får stadig en række ændringer, så jeg mener stadig denne er relevant.

Samtidig har issues så som #2304 og projekter der vil benytte Features Override i nyeste version behov for nyeste version af features.

Derfor vil jeg foreslå at man i samme forbindelse med denne ticket også opdaterer Features til seneste version, pt. 2.10.

#26 Updated by Rolf Madsen about 2 years ago

  • Status changed from Needs analysis to Ready for development

#27 Updated by Kasper Garnæs about 2 years ago

Ændringerne i #1551 kunne medtages i denne opgave.

#28 Updated by Rolf Madsen about 2 years ago

  • Target version changed from DDB CMS 2017 1. opgradering (7.x-4.0.2) to Release 27 - Bugfixes (2017 2. opgradering) (7.x-4.2.1)

#29 Updated by Gitte Barlach over 1 year ago

  • Description updated (diff)
  • Assignee changed from Gitte Barlach to Martin Cording

Hej Martin 

Har I mulighed for at kigge på denne?

#30 Updated by Martin Cording over 1 year ago

  • Assignee changed from Martin Cording to Christel Krabbenhøft

Jeg er usikker på hvad der skal ske her.

Skal alle features gen-eksporteres, eller skal features modulet "blot" opgraderes til 2.10?

#31 Updated by Rolf Madsen over 1 year ago

  • Status changed from Ready for development to Need more info
  • Assignee changed from Christel Krabbenhøft to Kasper Garnæs

@Kasper & @Per har I nogen input her?

Det er et meget teknisk issue så jeg kan ikke byde ind med noget ...

#32 Updated by Rolf Madsen over 1 year ago

  • Target version changed from Release 27 - Bugfixes (2017 2. opgradering) (7.x-4.2.1) to Release 29-2 - Bugfixes (7.x-4.5.0)

#33 Updated by Rolf Madsen over 1 year ago

  • Status changed from Need more info to Needs analysis
  • Assignee changed from Kasper Garnæs to Gitte Barlach

@Gitte, vil du ikke booke Core team til at tage et kig på den her?

#34 Updated by Rolf Madsen over 1 year ago

  • Assignee changed from Gitte Barlach to Kasper Garnæs
  • Target version changed from Release 29-2 - Bugfixes (7.x-4.5.0) to Release 29-2 - Bugfixes (Inlead)

#35 Updated by Kasper Garnæs about 1 year ago

  • Status changed from Needs analysis to Ready for development
  • Assignee changed from Kasper Garnæs to Martin Cording

Skal alle features gen-eksporteres, eller skal features modulet "blot" opgraderes til 2.10?

I mine øjne begge dele.

#36 Updated by Rolf Madsen about 1 year ago

  • Target version changed from Release 29-2 - Bugfixes (Inlead) to Release 33 - Bugfixes

Also available in: Atom PDF