Skip to content

Variable renaming in support of a PR into dd-agent - #455

Closed
AlexTawse wants to merge 1 commit into
DataDog:masterfrom
AlexTawse:master
Closed

Variable renaming in support of a PR into dd-agent#455
AlexTawse wants to merge 1 commit into
DataDog:masterfrom
AlexTawse:master

Conversation

@AlexTawse

@AlexTawse AlexTawse commented Jun 1, 2017

Copy link
Copy Markdown

What does this PR do?

Motivation

See this PR for details: DataDog/dd-agent#3366

Additional Notes

Depends on, and is dependent of, this PR: DataDog/dd-agent#3366
The build will likely fail until this PR is merged.

@hkaj hkaj left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

one comment on the import, everything else is 👍

Comment thread docker_daemon/check.py Outdated
RANCHER_STACK_NAME)
SWARM_SVC_LABEL)
from utils.kubernetes import KubeUtil
from utils.rancher import RancherUtil

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

will need to move to L24 after the directory move in DataDog/dd-agent#3366

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

👍

@masci masci added the community label Jun 8, 2017
@masci masci modified the milestone: triage Jun 14, 2017
@stale

stale Bot commented Nov 16, 2017

Copy link
Copy Markdown

This PR has been automatically marked as stale because it has not had activity in the last 60 days. Note that the PR will not be automatically closed, but this notification will remind us to investigate why there's been inactivity. Thank you for participating in the Datadog open source community.

@stale stale Bot added the stale label Nov 16, 2017
@masci masci modified the milestones: triage, 5.21 Nov 22, 2017
@stale stale Bot removed the stale label Nov 22, 2017
@masci

masci commented Nov 22, 2017

Copy link
Copy Markdown
Contributor

Still waiting on sister PR DataDog/dd-agent#3366, setting the milestone accordingly

@truthbk truthbk modified the milestones: 5.21, 5.22 Jan 4, 2018
@hkaj hkaj modified the milestones: 5.22, 5.23 Feb 7, 2018
@masci masci modified the milestones: 6.1, 6.2/5.24 Mar 21, 2018
@masci

masci commented Apr 12, 2018

Copy link
Copy Markdown
Contributor

Closing as we won't merge DataDog/dd-agent#3366, details here.

@masci masci closed this Apr 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants