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.

Sign up for GitHub

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

Jump to bottom

cfstore plugin requirements #1

Closed
shroffk opened this issue Feb 24, 2016 · 3 comments
Closed

cfstore plugin requirements #1

shroffk opened this issue Feb 24, 2016 · 3 comments
Assignees
@mdavidsaver
Labels
enhancement

Comments

@shroffk
Copy link
Collaborator

shroffk commented Feb 24, 2016

We have been working on developing a plugin for recsync which pipes the data to channelfinder.

Currently the features supported are

  1. which ioc startup, all the records are added to channelfinder along with properties iocName, hostName, time, and ovStatus
  2. upon ioc reboot, the "time"

Missing
When an IOC shuts down the information available in the transaction object consists of only the host ip (as part of the TR.src). Since there might be more than one ioc running on a particular host, the IOCNAME information is necessary to update channelfinder correctly.
@shroffk shroffk changed the title cfstore plugin cfstore plugin requirements Feb 24, 2016
@mdavidsaver
Copy link
Collaborator

When an IOC shuts down the information available in the transaction object consists of only the host ip

Both IP address and port number are included, which together are unique.

@mdavidsaver mdavidsaver self-assigned this Feb 24, 2016
@shroffk
Copy link
Collaborator Author

shroffk commented Feb 24, 2016

Currently cfstore dose not record the port, instead it uses the iocname+hostname.
Maybe the port should be stored as a property too.

@mdavidsaver
Copy link
Collaborator

Keeping "ip:port" seems reasonable

shroffk added a commit that referenced this issue May 10, 2016
@shroffk
Merge pull request #1 from mskinner5278/master
f016ae5 
included cfstore.py
shroffk added a commit that referenced this issue May 10, 2016
@shroffk
Merge pull request #1 from mskinner5278/master
305e6f0 
Pushing previous changes to plugin.
shroffk added a commit that referenced this issue May 10, 2016
@shroffk
temporary fix for issue #1 and #2
7edbb4a
mdavidsaver added a commit that referenced this issue Jun 16, 2016
@mdavidsaver
Merge remote-tracking branch 'origin/cf-store'
dd45874 
* origin/cf-store:
  Fix missing cfstore processing factory
  Fix optional dependency on Channel Finder client library.
  cfstore is optional.  soft fail when CF client library not present
  search order for CF owner
  Remove debug comments, fix some whitespace, and remove unused stub methods.
  Fixing the mistakenly committed test code.
  Cleaning up the pull request to have minimum impact on prior defaults
  #2 fixing update logic so that pre existing properties are not destroyed
  temporary fix for issue #1 and #2
  Update cfstore.py
  Adding a simple hack to ensure the plugin doesn't break with exits  Committer: Kunal Shroff<shroffk@bnl.gov>
  updating for cfstore Committer: Kunal Shroff<shroffk@bnl.gov>
  Pushing previous changes to plugin.
  fixed server print
  updated cf-store for channelfinder3.0
  included cfstore.py
@shroffk shroffk closed this as completed Sep 6, 2016
@shroffk shroffk mentioned this issue Sep 6, 2016
Closed
shroffk pushed a commit that referenced this issue Oct 21, 2022
@kingspride
Merge pull request #1 from ChannelFinder/master
13c3da4 
take recent changes
shroffk pushed a commit that referenced this issue Mar 14, 2023
@tynanford
Merge pull request #1 from simon-ess/single_var_at_a_time
bd23c50 
Switch to using one function call per environment variable
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mdavidsaver mdavidsaver

Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@shroffk @mdavidsaver