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

Outage tasks for 2023-09-14 #843

Closed
8 tasks done
kltm opened this issue Aug 23, 2023 · 10 comments
Closed
8 tasks done

Outage tasks for 2023-09-14 #843

kltm opened this issue Aug 23, 2023 · 10 comments

Comments

@kltm
Copy link
Member

kltm commented Aug 23, 2023

@kltm
Copy link
Member Author

kltm commented Sep 13, 2023

@vanaukenk migration mock run nom complete.

@vanaukenk
Copy link

@kltm
I've double-checked the diff and all looks consistent with recent ontology changes.
The only models affected are MGI gene-centric models.
Thx.

@suzialeksander
Copy link

@vanaukenk are you aware we are trying to sneak the YeastPathways models in with state = development? Just in case you see a ton of new models or something, note that this is expected- or please correct me if I have my timings off @dustine32

@dustine32
Copy link
Contributor

Correct @suzialeksander and thank you for mentioning this here! I'm currently trying to see about fixing geneontology/pathways2GO#276 before the load to Prod (it looks like just commenting out a single line but I wanna test some models locally first). If I'm unable to get a fixed set of models by tomorrow, should we hold off on the prod load until the next maintenance window? Or should we just push what we have?

@suzialeksander
Copy link

@dustine32 as long as they're state=dev, I'm happy to push it anyway. Curation-wise, removing that edge is fairly minor to the other changes needed.

@kltm
Copy link
Member Author

kltm commented Sep 13, 2023

Just to clarify my understanding from poking around a little and the vibe:

  • this is for a handful of models (sjcarbon@moiraine:~/local/src/git/noctua-models/models[yeast-pathways-20230811]$:) ls | grep -i "yeastpathways" | wc -l ->220)
  • any further changes to models are minor
  • any further changes will be done manually in the interface by @suzialeksander, w/o a reload later on
  • all models are dev and will accordingly not appear in public interfaces

Does this sound right?

@vanaukenk
Copy link

Thank you @suzialeksander @dustine32 @kltm

I wasn't aware that the yeast pathways models were going onto Noctua production for this outage. Keep us posted if anything changes.

@dustine32
Copy link
Contributor

@kltm Yeah, this only affects 220 models and they'll go in as state=development.

@vanaukenk
Copy link

@suzialeksander

Are you planning to do some QC checks on the yeast pathway models after tonight's outage? If so, do you want me to hold off on sending out the all clear email until you are finished?

@suzialeksander
Copy link

@vanaukenk I was planning on checking some of these, but I don't think there's any reason to hold off on the general email if it's just these models that have any issues. We plan on curating each one manually so tonight's checks will not be particularly in-depth.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

4 participants