Skip to content

Conversation

@mauri870
Copy link
Member

@mauri870 mauri870 commented Sep 10, 2024

Proposed commit message

I stumbled upon some warnings when trying to start the integration testing environment. The logs don't cause any harm but they pollute the output of the tool. Warnings include:

WARN[0000] /home/mauri870/git/elastic/beats/testing/environments/snapshot.yml: the attribute `version` is obsolete, it will be ignored, please remove it to avo
id potential confusion
WARN[0000] /home/mauri870/git/elastic/beats/testing/environments/local.yml: the attribute `version` is obsolete, it will be ignored, please remove it to avoid
potential confusion
WARN: MaintainerDeprecated: Maintainer instruction is deprecated in favor of using label (line 4)

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.

How to test this PR locally

  • cd testing/environments; make start ENV=snapshot.yml

I stumbled upon some warnings when trying to start the integration testing
environment. The errors don't cause any harm but they pollute the output of
the tool. Errors include:

	WARN[0000] /home/mauri870/git/elastic/beats/testing/environments/snapshot.yml: the attribute `version` is obsolete, it will be ignored, please remove it to avo
id potential confusion
	WARN[0000] /home/mauri870/git/elastic/beats/testing/environments/local.yml: the attribute `version` is obsolete, it will be ignored, please remove it to avoid
potential confusion

	WARN: MaintainerDeprecated: Maintainer instruction is deprecated in favor of using label (line 4)
@mauri870 mauri870 requested a review from a team as a code owner September 10, 2024 13:50
@mauri870 mauri870 requested review from belimawr and rdner September 10, 2024 13:50
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Sep 10, 2024
@mergify
Copy link
Contributor

mergify bot commented Sep 10, 2024

This pull request does not have a backport label.
If this is a bug or security fix, could you label this PR @mauri870? 🙏.
For such, you'll need to label your PR with:

  • The upcoming major version of the Elastic Stack
  • The upcoming minor version of the Elastic Stack (if you're not pushing a breaking change)

To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-v8./d.0 is the label to automatically backport to the 8./d branch. /d is the digit

NOTE: backport-v8.x has been added to help with the transition to the new branch 8.x.

@mauri870 mauri870 added the Team:Elastic-Agent-Data-Plane Label for the Agent Data Plane team label Sep 10, 2024
@elasticmachine
Copy link
Contributor

Pinging @elastic/elastic-agent-data-plane (Team:Elastic-Agent-Data-Plane)

@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Sep 10, 2024
@mergify
Copy link
Contributor

mergify bot commented Sep 11, 2024

backport-8.x has been added to help with the transition to the new branch 8.x.

@mergify mergify bot added the backport-8.x Automated backport to the 8.x branch with mergify label Sep 11, 2024
@v1v v1v removed the backport-v8.x label Sep 11, 2024
@mauri870 mauri870 added backport-skip Skip notification from the automated backport with mergify and removed backport-8.x Automated backport to the 8.x branch with mergify labels Sep 12, 2024
@mauri870 mauri870 merged commit 28cd825 into elastic:main Sep 13, 2024
@pierrehilbert pierrehilbert added the backport-8.x Automated backport to the 8.x branch with mergify label Sep 16, 2024
mergify bot pushed a commit that referenced this pull request Sep 16, 2024
* testing: Fix Docker deprecation warnings

I stumbled upon some warnings when trying to start the integration testing
environment. The errors don't cause any harm but they pollute the output of
the tool. Errors include:

	WARN[0000] /home/mauri870/git/elastic/beats/testing/environments/snapshot.yml: the attribute `version` is obsolete, it will be ignored, please remove it to avo
id potential confusion
	WARN[0000] /home/mauri870/git/elastic/beats/testing/environments/local.yml: the attribute `version` is obsolete, it will be ignored, please remove it to avoid
potential confusion

	WARN: MaintainerDeprecated: Maintainer instruction is deprecated in favor of using label (line 4)

(cherry picked from commit 28cd825)
mauri870 added a commit that referenced this pull request Sep 17, 2024
* testing: Fix Docker deprecation warnings

I stumbled upon some warnings when trying to start the integration testing
environment. The errors don't cause any harm but they pollute the output of
the tool. Errors include:

	WARN[0000] /home/mauri870/git/elastic/beats/testing/environments/snapshot.yml: the attribute `version` is obsolete, it will be ignored, please remove it to avo
id potential confusion
	WARN[0000] /home/mauri870/git/elastic/beats/testing/environments/local.yml: the attribute `version` is obsolete, it will be ignored, please remove it to avoid
potential confusion

	WARN: MaintainerDeprecated: Maintainer instruction is deprecated in favor of using label (line 4)

(cherry picked from commit 28cd825)

Co-authored-by: Mauri de Souza Meneguzzo <[email protected]>
Co-authored-by: Pierre HILBERT <[email protected]>
@khushijain21 khushijain21 mentioned this pull request Jun 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-8.x Automated backport to the 8.x branch with mergify backport-skip Skip notification from the automated backport with mergify cleanup Team:Elastic-Agent-Data-Plane Label for the Agent Data Plane team

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants