Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

closed ports of elastic docker #1364

Closed

Conversation

krishnaglodha
Copy link
Member

Related Issue / Discussion

PR against issue #1363

Additional Information

Ports are closed now, although I see undefined in the popup , should I create issue and fix this ?

Contributions and Licensing

(as per https://github.com/geopython/pygeoapi/blob/master/CONTRIBUTING.md#contributions-and-licensing)

  • I'd like to contribute Close the elastic ports on the elastic docker example #1363 to pygeoapi. I confirm that my contributions to pygeoapi will be compatible with the pygeoapi license guidelines at the time of contribution.
  • I have already previously agreed to the pygeoapi Contributions and Licensing Guidelines

@krishnaglodha krishnaglodha changed the title closed ports of elk closed ports of elastic docker Oct 4, 2023
@PascalLike
Copy link
Contributor

Thank you Krishna! But for the month of October, it is probably better to leave these hacktoberfest issues for people who are approaching pygeoapi for the first time. These are very suitable activities to get closer to the code base as ice breakers 🙏

Copy link
Contributor

@doublebyte1 doublebyte1 left a comment

Choose a reason for hiding this comment

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

Thank you for contributing @krishnaglodha! 👍🏽

It want to address two issues, it would be better to issue two different PR, each one with the code that addresses specifically that issue :-) . You can read more about this in the contribution guidelines.

@@ -60,8 +60,8 @@ services:
container_name: elastic

ports:
- 9300:9300
Copy link
Contributor

Choose a reason for hiding this comment

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

@krishnaglodha : this is not closing the port. It is assigning the port to a random port in the host.

@@ -86,41 +86,6 @@ metadata:
role: pointOfContact

resources:
obs:
Copy link
Contributor

Choose a reason for hiding this comment

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

@krishnaglodha Thank you for this, but this is actually addressing a different issue https://github.com/geopython/pygeoapi/issues/1362 . Please don't address it in this PR.

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks I'll separate PR for both issues

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks I'll separate PR for both issues

Copy link
Contributor

Choose a reason for hiding this comment

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

@krishnaglodha Please don't do anything else and read the other comments from @PascalLike and myself.

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes will not work on it

@doublebyte1
Copy link
Contributor

Thank you Krishna! But for the month of October, it is probably better to leave these hacktoberfest issues for people who are approaching pygeoapi for the first time. These are very suitable activities to get closer to the code base as ice breakers 🙏

@krishnaglodha I agree :-) If you want to help during the #hacktoberfest, you could create issues, similar to the ones I did: very simple and self contained, for instance fixing a bug on the documentation, or something in the configuration. That would be really helpful! 🥇

@doublebyte1 doublebyte1 closed this Oct 5, 2023
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.

None yet

3 participants