Skip to content
This repository has been archived by the owner on Jan 27, 2023. It is now read-only.

[apmpackage] add apm 7.16 #2357

Merged
merged 7 commits into from
Oct 14, 2021
Merged

Conversation

stuartnelson3
Copy link

add apmpackage 7.16.0 to snapshot

@stuartnelson3 stuartnelson3 requested a review from a team October 11, 2021 12:54
@elasticmachine
Copy link
Collaborator

elasticmachine commented Oct 11, 2021

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2021-10-14T09:14:04.055+0000

  • Duration: 25 min 39 sec

  • Commit: ca99b13

Test stats 🧪

Test Results
Failed 0
Passed 71
Skipped 0
Total 71

@stuartnelson3
Copy link
Author

/test

@stuartnelson3
Copy link
Author

Not sure what's going on with the nil pointer deref, mage -debug check appears to be running without issue on my local machine

@stuartnelson3
Copy link
Author

/test

@axw
Copy link
Member

axw commented Oct 12, 2021

We haven't bumped the version in the apm-server repo yet -- we normally do that and copy across. Any reason to diverge?

Never mind, I just saw elastic/apm-server#6319

# change type can be one of: enhancement, bugfix, breaking-change
- version: "7.16.0"
changes:
- description: remove {transaction,span}.message.routing_key
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Left a comment on the linked APM server PR about this entry

@stuartnelson3
Copy link
Author

ok, going to wait for #2365 to be merged

@jsoriano
Copy link
Member

ok, going to wait for #2365 to be merged

Change merged 🙂 but not sure how this would affect this PR. Maybe it was this one: #2392, try to update the branch.

@elasticmachine
Copy link
Collaborator

💔 Build Failed

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2021-10-13T09:04:01.904+0000

  • Duration: 6 min 34 sec

  • Commit: cf41ad2

Steps errors 1

Expand to view the steps failures

Checks formatting / linting
  • Took 0 min 4 sec . View more details here
  • Description: mage -debug check

1 similar comment
@elasticmachine
Copy link
Collaborator

💔 Build Failed

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2021-10-13T09:04:01.904+0000

  • Duration: 6 min 34 sec

  • Commit: cf41ad2

Steps errors 1

Expand to view the steps failures

Checks formatting / linting
  • Took 0 min 4 sec . View more details here
  • Description: mage -debug check

@stuartnelson3
Copy link
Author

@jsoriano @mtojek do either of you know what's going wrong with this build?

[2021-10-13T09:09:30.231Z] + mage -debug check
...
[2021-10-13T09:09:33.722Z] DEBUG: 09:09:33.443186 running magefile with mage vars:
[2021-10-13T09:09:33.722Z] MAGEFILE_DEBUG=1
[2021-10-13T09:09:33.722Z] MAGEFILE_GOCMD=go
[2021-10-13T09:09:34.105Z] Error: runtime error: invalid memory address or nil pointer dereference
script returned exit code 1

@mtojek
Copy link
Contributor

mtojek commented Oct 13, 2021

Not really, I'm afraid you have to try reproducing it locally.

@mtojek mtojek self-requested a review October 13, 2021 09:58
Copy link
Contributor

@mtojek mtojek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This package is versioned as 7.16.0, which is a bit odd comparing to other packages and considering that we don't bind packages with Elastic release trains.

@simitt @ruflin Are we sure about this approach?

@ruflin
Copy link
Contributor

ruflin commented Oct 13, 2021

The idea is to bundle the apm, synthetics, endpoint package soon directly with Kibana which indirectly means these will become part of the release train. This does not mean, the packages should not still be available in the registry but that is where the versioning comes from.

@mtojek
Copy link
Contributor

mtojek commented Oct 13, 2021

This may cause troubles for our users in the future if they start mixing versions. For example: Kibana 8.3.0 and APM package 7.16.0, because the version constraint allows for it.

I would like to highlight here that our initial principle was to not bind package releases with stack trains.

@ruflin
Copy link
Contributor

ruflin commented Oct 13, 2021

If I remember correctly we want to go so far that also the constraint would not allow this. Also these packages are "auto upgraded", basically the user will not have an option to opt out in the future. This applies to only a very small list of packages.

@mtojek
Copy link
Contributor

mtojek commented Oct 13, 2021

If I remember correctly we want to go so far that also the constraint would not allow this.

Hm.. interesting. Could you please elaborate more on this?

Also these packages are "auto upgraded", basically the user will not have an option to opt out in the future. This applies to only a very small list of packages.

TIL! Is this list defined in Kibana or in other place? I'd like to learn more about this, e.g. upgrading rules, rollback, etc.

@axw
Copy link
Member

axw commented Oct 14, 2021

If I remember correctly we want to go so far that also the constraint would not allow this.

Hm.. interesting. Could you please elaborate more on this?

APM is a stack-aligned product, and we want to keep the number of variables down by fixing the package version exactly to the version of Kibana.

(Eventually?) we should update the package's constraint to be an exact version match. I think we want to wait for the package to be bundled in Kibana by release-manager before doing this, as otherwise we'll have to manually release patch releases of the package.

Also these packages are "auto upgraded", basically the user will not have an option to opt out in the future. This applies to only a very small list of packages.

TIL! Is this list defined in Kibana or in other place? I'd like to learn more about this, e.g. upgrading rules, rollback, etc.

Yes, in Kibana. Here's the recent PR that added APM and Synthetics to the list: elastic/kibana#114641

@stuartnelson3 stuartnelson3 merged commit e39c188 into elastic:snapshot Oct 14, 2021
@stuartnelson3 stuartnelson3 deleted the apm-7.16 branch October 14, 2021 09:46
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants