Bulk package-spec v3 migration of packages for serverless #8147

Activity

1. shmsr

   self-assigned this

   on Oct 10, 2023
2. shmsr

   changed the title [-]Bulk serverless v3 migration of packages[/-] [+]Bulk package-spec v3 migration of packages for serverless[/+] on Oct 10, 2023
3. shmsr commented on Oct 16, 2023

   

   shmsr

   on Oct 16, 2023

   MemberAuthor

   More actions

   Related links:

   • https://github.com/elastic/elastic-package/blob/main/docs/howto/update_major_package_spec.md
   • Update (most) obs-cloud-monitoring owned packages to format_version 3.0.0 #8041 (sample PR)
   • Update (some) integrations team owned packages to format_version: 3.0.0 #8031 (sample PR)
4. shmsr commented on Oct 16, 2023

   

   shmsr

   on Oct 16, 2023

   MemberAuthor

   More actions

   Rough notes (challenges faced while migrating to package-spec v3):

   • Migrate obs-infraobs-integrations to package-spec v3 #1 #8170 (comment)
   • Migrate obs-infraobs-integrations to package-spec v3 #2 #8171 (comment)
   • Migrate obs-infraobs-integrations to package-spec v3 #3 #8203 (comment)
5. shmsr commented on Oct 16, 2023

   

   shmsr

   on Oct 16, 2023

   MemberAuthor

   More actions

   pinging:
   @elastic/obs-service-integrations for review
   @tommyers-elastic for review
6. shmsr commented on Oct 16, 2023

   

   shmsr

   on Oct 16, 2023

   MemberAuthor

   More actions

   Have made the changes in batches so that it is easier to review.

   In total 48 packages are owned by @elastic/obs-infraobs-integrations as per CODEOWNERS

   $ pkgs=($(cat .github/CODEOWNERS | grep '@elastic/obs-infraobs-integrations' | cut -d ' ' -f 1 | cut -d '/' -f 3 | sort | uniq)
   

   So packages are migrated in a batch of 15, 15 and 18 (15+15+18=48).
7. shmsr commented on Oct 16, 2023

   

   shmsr

   on Oct 16, 2023

   MemberAuthor

   More actions

   Please read the rough notes carefully before reviewing because some packages need a thorough review.
8. shmsr commented on Oct 16, 2023

   

   shmsr

   on Oct 16, 2023

   MemberAuthor

   More actions

   Also see #8174 before review. Related changes are there in all 3 PRs. As a side-effect of transformation that elastic-package makes, some fields needs to be quoted in test configuration files.
9. tommyers-elastic commented on Oct 16, 2023

   

   tommyers-elastic

   on Oct 16, 2023

   Contributor

   More actions

   closing this issue in favour of the tracking issue here
10. tommyers-elastic

    closed this as completedon Oct 16, 2023