FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

feat: add influx-cli image by alespour · Pull Request #554 · influxdata/influxdata-docker · GitHub

feat: add influx-cli image - #554

Open
alespour wants to merge 6 commits into
influxdata:masterfrom
bonitoo-io:feat/influx-cli
Open

feat: add influx-cli image#554
alespour wants to merge 6 commits into
influxdata:masterfrom
bonitoo-io:feat/influx-cli

Conversation

alespour commented Nov 24, 2021
edited
Loading

Copy link
Copy Markdown
Contributor

This PR adds influx-cli image. The image is intended for CLI-based tasks like backup etc. Its size is "only" ~44 MB, compared to 226 MB of influxdb. Dockerfile content was ripped out of influxdb image.

Default command is version.

alespour marked this pull request as ready for review December 1, 2021 11:48

timhallinflux left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I think we need a License file too. This is MIT licensed. We can grab the file from the core InfluxDB portion of this repo.

Comment thread influx-cli/2.2/alpine/Dockerfile Outdated

timhallinflux left a comment
edited
Loading

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I believe this is also missing a manifest.json file.

{
  "name": "influxdb-cli",
  "maintainers": [
    "Jonathan A. Sternberg <jonathan@influxdata.com> (@jsternberg)",
    "Bucky Schwarz <bucky@influxdata.com> (@hoorayimhelping)",
    "Brandon Pfeifer  <bpfeifer@influxdata.com> (@bnpfeife)"
  ],
  "versions": ["2.2"],
  "architectures": [
    "amd64",
    "arm32v7",
    "arm64v8"
  ],
  "variants": [
    {"name": "alpine"}
  ]
}

Comment thread influx-cli/LICENSE Outdated

samhld left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

LGTM

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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


Back | FazBrowse Home | New Git URL