Skip to content

Conversation

bmorelli25
Copy link
Member

Summary

Adds the docs directory in elastic/apm-aws-lambda as a dependency to the following repos:

  • elastic/apm-agent-java
  • elastic/apm-agent-nodejs
  • elastic/apm-agent-python

For https://github.com/elastic/observability-docs/issues/1707.

@bmorelli25
Copy link
Member Author

@gtback I might need to pick your brain on this one. As the summary says, we're trying to add one repository as a dependency of the other three. The difficult part is that all four repositories are versioned differently.

@bmorelli25 bmorelli25 requested a review from gtback March 28, 2022 19:23
Copy link
Member

@gtback gtback left a comment

Choose a reason for hiding this comment

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

The difficult part is that all four repositories are versioned differently.

This is exactly what map_branches is for. The *mapMasterToMain and *mapMainToMaster are just shortcuts for when that's the only mismatch in branch names (and since all four of the repos use main, you won't need either of those).

@bmorelli25 bmorelli25 marked this pull request as ready for review March 28, 2022 21:44
@bmorelli25 bmorelli25 requested a review from gtback March 28, 2022 21:47
Copy link
Member

@gtback gtback left a comment

Choose a reason for hiding this comment

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

LGTM (assuming tests pass and the preview looks good)

@gtback
Copy link
Member

gtback commented Mar 28, 2022

Build failed for the same reason the main build is failing. If we re-run this tomorrow, it will hopefully pass.

@bmorelli25
Copy link
Member Author

@elasticmachine, run elasticsearch-ci/docs rebuild

@bmorelli25 bmorelli25 merged commit 82b981d into elastic:master Mar 29, 2022
@bmorelli25 bmorelli25 deleted the add-aws-deps branch March 29, 2022 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants