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

update operations doc for remigrating in QA #3213

Merged
merged 1 commit into from
Apr 25, 2018

Conversation

Glignos
Copy link
Contributor

@Glignos Glignos commented Feb 26, 2018

Signed-off-by: Glignos glignos93@gmail.com

Description

Related Issue

Motivation and Context

Checklist:

  • I have all the information that I need (if not, move to RFC and look for it).
  • I linked the related issue(s) in the corresponding commit logs.
  • I wrote good commit log messages.
  • My code follows the code style of this project.
  • I've added any new docs if API/utils methods were added.
  • I have updated the existing documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@ghost ghost assigned Glignos Feb 26, 2018
@ghost ghost added the Review: WIP label Feb 26, 2018

.. code-block:: shell

$ ssh screen
Copy link
Contributor

Choose a reason for hiding this comment

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

you are supposed to be already in the machine here, so you might want to remove the ssh there, and probably show in the prompt that you are in the remote machine, something like:

# This will also reconnect no a running session if any
build1$ screen -dR

@Glignos Glignos force-pushed the qa_doc branch 2 times, most recently from f14c3f2 to 6d6ae7f Compare February 26, 2018 12:41

.. code-block:: shell

$ workon inspire
Copy link
Contributor

Choose a reason for hiding this comment

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

Here you are missing the build1 thingie too :) (and below)

@Glignos Glignos force-pushed the qa_doc branch 5 times, most recently from 4e875c5 to 25aa399 Compare February 27, 2018 14:52
@Glignos Glignos requested a review from drjova February 27, 2018 14:53
@Glignos
Copy link
Contributor Author

Glignos commented Feb 27, 2018

@drjova Give it a check? :)

drjova
drjova previously approved these changes Mar 20, 2018
@drjova
Copy link
Contributor

drjova commented Mar 20, 2018

ci merge please

@Glignos
Copy link
Contributor Author

Glignos commented Mar 20, 2018

ci merge please


.. code-block:: shell

build1$ inspirehep migrator populate -f filename
Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

I should probably take over, it has changed quite a lot since @Glignos did it.

@drjova drjova dismissed their stale review April 5, 2018 09:52

Deprecation of the documented command

@ghost ghost assigned michamos Apr 5, 2018
Setting up the environment
~~~~~~~~~~~~~~~~~~~~~~~~~~

1. First of all establish a Kerberos authentication (this can be helpfull:
Copy link
Contributor

@drjova drjova Apr 5, 2018

Choose a reason for hiding this comment

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

s/helpfull/helpful

@michamos
Copy link
Contributor

ci merge please

@michamos
Copy link
Contributor

ci merge please

@david-caro
Copy link
Contributor

my bad... one sec

@david-caro
Copy link
Contributor

ci merge please

1 similar comment
@david-caro
Copy link
Contributor

ci merge please

Co-authored-by: Glignos <glignos93@gmail.com>
Signed-off-by: Micha Moskovic <michamos@gmail.com>
@ghost ghost assigned inspire-bot Apr 25, 2018
@inspire-bot inspire-bot merged commit 55499c8 into inspirehep:master Apr 25, 2018
inspire-bot pushed a commit that referenced this pull request Apr 25, 2018
turtle321 pushed a commit that referenced this pull request May 2, 2018
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.

5 participants