[COMPASS-6200] Project Evergreen YAML is dependent on a bug in YAML parsing which will soon be removed Created: 13/Oct/22 Updated: 29/Oct/23 Resolved: 12/Dec/22 |
|
| Status: | Closed |
| Project: | Compass |
| Component/s: | None |
| Affects Version/s: | None |
| Fix Version/s: | No version |
| Type: | Task | Priority: | Major - P3 |
| Reporter: | Kim Tao | Assignee: | Sergey Petushkov |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | None | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Issue Links: |
|
||||
| Documentation Changes: | Not Needed | ||||
| Sprint: | Iteration Porpoise, Iteration Quahog | ||||
| Description |
|
Parts of the project YAML are relying on a bug in Evergreen's YAML parsing library with regards to how nested fields are merged when using aliases. For example, in this snippet of the YAML, the current YAML parser will interpret this as:
This is relying on the fact that the YAML parser does a deep nested merge of the keys for the params map between the save-artifacts alias and the in-line defined params. However, the YAML spec states that in this case, it's supposed to actually do a shallow merge, which would result in this interpretation:
This bug has been fixed in the latest YAML parser version, and EVG-17291 will upgrade the YAML parser soon, so reliance on the deep nested merge behavior should be removed. |
| Comments |
| Comment by Githook User [ 07/Dec/22 ] |
|
Author: {'name': 'Sergey Petushkov', 'email': 'petushkov.sergey@gmail.com', 'username': 'gribnoysup'}Message: chore(ci): fix yaml anchor usage in evergreen config chore(ci): fix yaml anchor usage in evergreen config |
| Comment by Githook User [ 06/Dec/22 ] |
|
Author: {'name': 'Sergey Petushkov', 'email': 'petushkov.sergey@gmail.com', 'username': 'gribnoysup'}Message: chore(ci): fix yaml anchor usage in evergreen config chore(ci): fix yaml anchor usage in evergreen config |
| Comment by Githook User [ 05/Dec/22 ] |
|
Author: {'name': 'Sergey Petushkov', 'email': 'petushkov.sergey@gmail.com', 'username': 'gribnoysup'}Message: chore(ci): fix yaml anchor usage in evergreen config chore(ci): fix yaml anchor usage in evergreen config |
| Comment by Githook User [ 02/Dec/22 ] |
|
Author: {'name': 'Sergey Petushkov', 'email': 'petushkov.sergey@gmail.com', 'username': 'gribnoysup'}Message: chore(ci): fix yaml anchor usage in evergreen config chore(ci): fix yaml anchor usage in evergreen config |
| Comment by Githook User [ 02/Dec/22 ] |
|
Author: {'name': 'Sergey Petushkov', 'email': 'petushkov.sergey@gmail.com', 'username': 'gribnoysup'}Message: chore(ci): fix yaml anchor usage in evergreen config chore(ci): fix yaml anchor usage in evergreen config |
| Comment by Githook User [ 01/Dec/22 ] |
|
Author: {'name': 'Sergey Petushkov', 'email': 'petushkov.sergey@gmail.com', 'username': 'gribnoysup'}Message: chore(ci): fix yaml anchor usage in evergreen config chore(ci): fix yaml anchor usage in evergreen config |
| Comment by Githook User [ 01/Dec/22 ] |
|
Author: {'name': 'Sergey Petushkov', 'email': 'petushkov.sergey@gmail.com', 'username': 'gribnoysup'}Message: chore(ci): fix yaml anchor usage in evergreen config chore(ci): fix yaml anchor usage in evergreen config |
| Comment by Githook User [ 01/Dec/22 ] |
|
Author: {'name': 'Sergey Petushkov', 'email': 'petushkov.sergey@gmail.com', 'username': 'gribnoysup'}Message: chore(ci): fix yaml anchor usage in evergreen config chore(ci): fix yaml anchor usage in evergreen config |