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

Add current ip address, long hostname and short hostname to /etc/hosts #2688

Merged
merged 1 commit into from
Apr 14, 2021

Conversation

lhotari
Copy link
Member

@lhotari lhotari commented Apr 14, 2021

Motivation

Fixes reverse DNS lookup for current hostname. This is broken in GitHub Actions currently.
See apache/pulsar#10232 and apache/pulsar#10233 for more details.

Changes

Add the current IP address, long hostname and short hostname record to /etc/hosts file at the beginning of each GitHub Actions workflow.

- Fixes reverse DNS lookup for current hostname
Copy link
Contributor

@dlg99 dlg99 left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you!

@merlimat merlimat added this to the 4.14.0 milestone Apr 14, 2021
@merlimat merlimat merged commit 9256dba into apache:master Apr 14, 2021
@dlg99 dlg99 mentioned this pull request Apr 14, 2021
hsaputra pushed a commit that referenced this pull request Apr 19, 2021
### Motivation

After deleting many ledgers, seeking to the end of the RocksDB metadata can take a long time and trigger timeouts upstream. Address this by improving the seek logic as well as compacting out tombstones in situations where we've just deleted many entries.  This affects the entry location index and the ledger metadata index.


Reviewers: Andrey Yegorov, Prashant Kumar

This closes #2686 from mauricebarnum/rocksdb-tombstones and squashes the following commits:

c029968 [Maurice Barnum] Partially compact RocksDB after removing deleted ledgers
0be68a1 [Maurice Barnum] KeyValueStorageRocksDB bounded key iterator: use RocksDB upper bound
0c5ef8f [Maurice Barnum] KeyValueStorageRocksDB.getFloor() - reimplement to avoid two seeks
114cc73 [Maurice Barnum] update rocksdb to 6.16.4
f489a05 [Surinder Singh] Fix test cases using gradle build (#2689)
9256dba [Lari Hotari] Add current ip address, long hostname and short hostname to /etc/hosts (#2688)
f1a1f12 [Surinder Singh] staging and setting up vote for release candidates (#2681)
2dd4afe [Prashant Kumar] ISSUE-2640: BP-43: Gradle integration with RAT plugin (#2683)
79768fe [Jack Vanlightly] ISSUE #2615: Fix for invalid ensemble issue during ledger recovery
646e590 [Prashant Kumar] ISSUE #2640: BP-43 integrate gradle javadoc plugin
d70153f [hangc0276] Update documentation with default value for openLedgerRereplicationGracePeriod config option
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.

3 participants