fix order of operations during multilan init #650
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Currently, the IP Mapping feature is broken due to calling the csipmap_init() function late after the map config was already read. This causes mooseFS to not find any mappings during runtime
Motivation and Context
Issue #649
Description
within multilan_init() I moved the call to csipmap_init() to the beginning of the function rather than the end, since IPMAP loading happens during multilan_reload().
How Has This Been Tested?
Tested on my own setup with a single mfsmaster that is on a VPN network with the chunkserver and the client. the client is also in direct LAN with the chunkserver. The fix was compiled and deployed and I tried streaming a video file from my existing MFS mount. Before the fix, the connection would be made via VPN even though the mfsipmap.cfg file was configured to use LAN IP. After the fix the stream was consumed from LAN as expected. the issue is described in detail in the issue ticket that I opened
This shouldn't affect any other part of code, the csipmap_init() function is pretty simple and benign, only setting global structures and values to NULL.
Types of changes
Checklist: